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

feat(platform): Schedule specific agent version #9444

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

kcze
Copy link
Contributor

@kcze kcze commented Feb 7, 2025

Scheduling always takes the newest version of an agent.

Changes 🏗️

This PR allows to schedule any graph version by adding number input to schedule popup. Number is automatically set to the newest version when agent is chosen.

Screenshot 2025-02-07 at 5 05 56 PM
  • Update API, so it accepts graph version
  • Update schedule pop up, so it lets user input version number
  • Open and schedule correct agent
  • Add Version column to the schedules table

Checklist 📋

For code changes:

  • I have clearly listed my changes in the PR description
  • I have made a test plan
  • I have tested my changes according to the test plan:
    • Can schedule version between 1 and max version
    • Reject incorrect version
    • Table shows proper version
    • Removing schedule works

@kcze kcze requested a review from a team as a code owner February 7, 2025 16:07
@kcze kcze requested review from Pwuts and aarushik93 and removed request for a team February 7, 2025 16:07
@github-actions github-actions bot added platform/frontend AutoGPT Platform - Front end platform/backend AutoGPT Platform - Back end labels Feb 7, 2025
Copy link

netlify bot commented Feb 7, 2025

Deploy Preview for auto-gpt-docs-dev canceled.

Name Link
🔨 Latest commit 989b92b
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs-dev/deploys/67a62fd9b936b80008d23300

@github-actions github-actions bot added the size/m label Feb 7, 2025
@kcze kcze requested review from majdyz and removed request for Pwuts February 7, 2025 16:08
Copy link

netlify bot commented Feb 7, 2025

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 989b92b
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/67a62fd92fa026000841c3fa

Copy link

deepsource-io bot commented Feb 7, 2025

Here's the code health analysis summary for commits 1a4ba53..989b92b. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ Success
❗ 71 occurences introduced
🎯 7 occurences resolved
View Check ↗
DeepSource Python LogoPython✅ Success
❗ 8 occurences introduced
🎯 8 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@kcze kcze enabled auto-merge February 7, 2025 16:36
@@ -198,6 +240,7 @@ export const SchedulesTable = ({
{agents.find((a) => a.id === schedule.graph_id)?.name ||
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we just append the version here? e.g: {graph_id}#V{graph_version} instead of a new column ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform/backend AutoGPT Platform - Back end platform/frontend AutoGPT Platform - Front end size/m
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants