-
Notifications
You must be signed in to change notification settings - Fork 46.1k
fix(frontend): Ensure agent version is fetched when active_version_id exists #11291
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
base: dev
Are you sure you want to change the base?
Conversation
|
This PR targets the Automatically setting the base branch to |
🔍 Existing Issues For ReviewYour pull request is modifying functions with the following pre-existing issues: 📄 File: autogpt_platform/frontend/src/app/(platform)/marketplace/agent/[creator]/[slug]/page.tsx
|
✅ Deploy Preview for auto-gpt-docs canceled.
|
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Comment |
|
Here's the code health analysis summary for commits Analysis Summary
|
|
Thank you for submitting this PR to fix the issue with server-side rendering and token retrieval. The changes look reasonable and well-focused on the problem described. However, before this can be approved for merging, please complete the checklist in the PR description by checking the appropriate boxes. Specifically:
The PR looks good otherwise, with a clear description of the issue being fixed and changes that appear to address the problem directly. |
|
Thank you for submitting this PR to fix the SSR issue with agent version fetching. The changes look straightforward and well-scoped. However, I noticed that you haven't checked off the items in the checklist. Please mark the following items:
You can mark the test plan as N/A since you've indicated that, but the checkboxes themselves need to be checked for the PR to be compliant with our guidelines. Otherwise, the code changes look good - ensuring the agent version is only fetched when necessary conditions are met should resolve the server-side rendering issue described in the Sentry report. |
Changes 🏗️
Fixes BUILDER-4HJ. The issue was that: Server-side rendering failed to retrieve the Supabase access token, causing authenticated API calls to omit the Authorization header.
creator_agent.active_version_idexists and the status code is 200.prefetchGetV2GetAgentByStoreIdQueryquery whencreator_agent.active_version_idexists.This fix was generated by Seer in Sentry, triggered by Craig Swift. 👁️ Run ID: 2234004
Not quite right? Click here to continue debugging with Seer.
Checklist 📋
For code changes:
Example test plan
For configuration changes:
.env.defaultis updated or already compatible with my changesdocker-compose.ymlis updated or already compatible with my changesExamples of configuration changes