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

fix(backend): fix get agent db function #9350

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

Conversation

Abhi1992002
Copy link
Contributor

Changes πŸ—οΈ

  • Remove FastAPI exception raises from the get_agent database function.
  • Enable the get_agent function to include the user_id in the response.
  • Implement optional graph_data handling for the /download/agents/{store_listing_version_id} endpoint.
  • Remove the user_id from the graph data response for the /download/agents/{store_listing_version_id} endpoint.

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:
  • I have manually checked the downloading agent functionality

@Abhi1992002 Abhi1992002 requested a review from a team as a code owner January 28, 2025 05:41
@Abhi1992002 Abhi1992002 requested review from ntindle and Swiftyos and removed request for a team January 28, 2025 05:41
@github-actions github-actions bot added platform/backend AutoGPT Platform - Back end size/m labels Jan 28, 2025
Copy link

netlify bot commented Jan 28, 2025

βœ… Deploy Preview for auto-gpt-docs-dev canceled.

Name Link
πŸ”¨ Latest commit 0be6ff4
πŸ” Latest deploy log https://app.netlify.com/sites/auto-gpt-docs-dev/deploys/67986e4baea5e90008ea4015

Copy link

netlify bot commented Jan 28, 2025

βœ… Deploy Preview for auto-gpt-docs ready!

Name Link
πŸ”¨ Latest commit c8b17c6
πŸ” Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/67986e24cc88c10008934714
😎 Deploy Preview https://deploy-preview-9350--auto-gpt-docs.netlify.app
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Jan 28, 2025

βœ… Deploy Preview for auto-gpt-docs ready!

Name Link
πŸ”¨ Latest commit 0be6ff4
πŸ” Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/67986e4b4b08af00085355d3
😎 Deploy Preview https://deploy-preview-9350--auto-gpt-docs.netlify.app
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -620,6 +625,8 @@ def remove_credentials(obj):
del obj["credentials"]
if "creds" in obj:
Copy link
Member

Choose a reason for hiding this comment

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

This should probably be a greater utility function that gets called :)

we can probably also strip anything that has secret = true

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conflicts Automatically applied to PRs with merge conflicts platform/backend AutoGPT Platform - Back end size/m
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants