Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[front] mcp(remote): create remote server model #11409

Merged
merged 9 commits into from
Mar 19, 2025

Conversation

adrsimon
Copy link
Contributor

@adrsimon adrsimon commented Mar 17, 2025

Description

Create the remote server models
Contributes to https://github.com/dust-tt/tasks/issues/2368.

Tests

Migrated locally, works.

Risk

Supposed to be no risks

Deploy Plan

Block front
Migrate front
Deploy front
Unblock front.

@adrsimon adrsimon requested a review from Fraggle March 17, 2025 14:39
@adrsimon adrsimon self-assigned this Mar 17, 2025
Copy link

github-actions bot commented Mar 17, 2025

Fails
🚫 Please include a detailed Deploy Plan section in your PR description.
Warnings
⚠️

Files in **/lib/models/ have been modified and the PR has the migration-ack label. Don't forget to run the migration from prodbox.

Generated by 🚫 dangerJS against d34de65

Copy link
Contributor

@Fraggle Fraggle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks !
Made few comments but otherwise looks good.

@adrsimon adrsimon added the migration-ack 📁 Label to acknowledge that a migration is required. label Mar 19, 2025
@adrsimon adrsimon merged commit 70c47ce into main Mar 19, 2025
8 of 9 checks passed
@adrsimon adrsimon deleted the adrien/mcp/remote-server-model branch March 19, 2025 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
migration-ack 📁 Label to acknowledge that a migration is required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants