Skip to content

feat(frontend): Onboarding Updates 3 #9916

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

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

Conversation

kcze
Copy link
Contributor

@kcze kcze commented May 7, 2025

A collection of UX update and bug fixes for onboarding and wallet.

Changes πŸ—οΈ

  • Show spinner loading indicator when onboarding button is clicked
  • Use getLibraryAgentByStoreListingVersionID instead of addMarketplaceAgentToLibrary on congrats screen
  • Fix Not enough segments issue: don't fetch onboarding when user is logged out
  • Minor updates
    • Fill some missing deps in deps arrays
    • Spinner component, styles updates
    • Use useMemo/useCallback
  • Show error toast when onboarding agent fails to run:
Screenshot 2025-05-06 at 5 09 01β€―PM

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:
    • Onboarding can be completed
    • Failing agent shows toast
    • Wallet can be opened and works properly (tasks, confetti)
    • Dependency arrays don't cause infinite loops

Copy link

netlify bot commented May 7, 2025

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

Name Link
πŸ”¨ Latest commit 465bf21
πŸ” Latest deploy log https://app.netlify.com/sites/auto-gpt-docs-dev/deploys/681c6d9a6dc30500088451c3

@github-actions github-actions bot added platform/frontend AutoGPT Platform - Front end platform/backend AutoGPT Platform - Back end labels May 7, 2025
Copy link

deepsource-io bot commented May 7, 2025

Here's the code health analysis summary for commits 104928c..465bf21. View details on DeepSourceΒ β†—.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScriptβœ…Β Success
❗ 30 occurences introduced
🎯 27 occurences resolved
View CheckΒ β†—
DeepSource Python LogoPythonβœ…Β SuccessView CheckΒ β†—

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

@github-actions github-actions bot added the size/l label May 7, 2025
Copy link

netlify bot commented May 7, 2025

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

Name Link
πŸ”¨ Latest commit 465bf21
πŸ” Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/681c6d9aa055db000828a563
😎 Deploy Preview https://deploy-preview-9916--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.

@kcze kcze requested review from ntindle and majdyz May 7, 2025 12:12
@kcze kcze marked this pull request as ready for review May 7, 2025 12:12
@kcze kcze requested a review from a team as a code owner May 7, 2025 12:12
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: πŸ†• Needs initial review
Status: No status
Development

Successfully merging this pull request may close these issues.

1 participant