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): Redirect to builder after saving agent to library and disable save button for owned agents #9225

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from

Conversation

kcze
Copy link
Contributor

@kcze kcze commented Jan 8, 2025

Resolves:

Changes 🏗️

  • Add GET /library/agents/{store_listing_version_id} endpoint (and python function) that returns specific agent metadata to check if user owns it
  • Modify POST /library/agents/{store_listing_version_id} so it returns added agent (need agent id for redirection to the builder)
  • Redirect to builder when agent is added to the library
  • Disabled save to library button when agent is owned by the user
  • Show toast and disable button while saving or downloading
  • Remove commented out AgentDetailContent.tsx and its actions.ts

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:
    • Logged-in user owned agent displays "Already in Library"
    • Logged-in user already added agent displays "Already in Library"; not possible currently
    • Logged-in user not added agent displays "Add to Library"
    • Adding agent saves it and opens builder
    • Logged out displays "Download Agent as File"
    • Agent downloads

For configuration changes:

  • .env.example is updated or already compatible with my changes
  • docker-compose.yml is updated or already compatible with my changes
  • I have included a list of my configuration changes in the PR description (under Changes)
Examples of configuration changes
  • Changing ports
  • Adding new services that need to communicate with each other
  • Secrets or environment variable changes
  • New or infrastructure changes such as databases

@github-actions github-actions bot added platform/frontend AutoGPT Platform - Front end platform/backend AutoGPT Platform - Back end size/l labels Jan 8, 2025
Copy link

netlify bot commented Jan 8, 2025

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

Name Link
🔨 Latest commit 5ee23cc
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs-dev/deploys/67828735aaf77f0008ab9da3

Copy link

netlify bot commented Jan 8, 2025

Deploy Preview for auto-gpt-docs failed.

Name Link
🔨 Latest commit 5ee23cc
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/67828735e4a003000858fccf

@kcze kcze marked this pull request as ready for review January 9, 2025 15:39
@kcze kcze requested a review from a team as a code owner January 9, 2025 15:39
@kcze kcze requested review from Pwuts, Bentlybro, Swiftyos and majdyz and removed request for a team, Pwuts and Bentlybro January 9, 2025 15:39
@kcze kcze changed the title feat(platform): Dynamic button disabling and redirect to builder on agent listing feat(platform): Redirect to builder after saving agent to library and disable save button for owned agents Jan 9, 2025
@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jan 10, 2025
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Jan 11, 2025
Copy link
Contributor

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

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/l
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

1 participant