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

WIP- ci(frontend,design): add storybook builds to the frontend #9353

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

Conversation

ntindle
Copy link
Member

@ntindle ntindle commented Jan 28, 2025

Storybook is a tool designed for design to approve design systems

Changes 🏗️

  • Fixes a lot of storybook stories
  • Adds providers to the storybook system
  • Adds mocks for backend libraries

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:
    • test chromatic to ensure that it works and all the stories render

@ntindle ntindle requested a review from a team as a code owner January 28, 2025 14:23
Copy link

supabase bot commented Jan 28, 2025

This pull request has been ignored for the connected project bgwpwdsxblryihinutbx because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

Copy link
Contributor

This PR targets the master branch but does not come from dev or a hotfix/* branch.

Automatically setting the base branch to dev.

Copy link

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Secret Validation

Verify that the CHROMATIC_PROJECT_TOKEN secret is properly configured in the repository settings, as it is required for the Chromatic action to work

# ⚠️ Make sure to configure a `CHROMATIC_PROJECT_TOKEN` repository secret
projectToken: ${{ secrets.CHROMATIC_PROJECT_TOKEN }}

@github-actions github-actions bot removed the platform/frontend AutoGPT Platform - Front end label Jan 29, 2025
@github-actions github-actions bot added the platform/frontend AutoGPT Platform - Front end label Jan 29, 2025
Copy link

deepsource-io bot commented Jan 29, 2025

Here's the code health analysis summary for commits f6e395f..c766bd6. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ Success
❗ 10 occurences introduced
🎯 2 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 size/l and removed size/m labels Jan 29, 2025
Base automatically changed from open-2047-add-type-checking-step-to-front-end-ci to dev January 29, 2025 12:50
@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jan 29, 2025
Copy link
Contributor

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

@aarushik93
Copy link
Contributor

Could you please add a description, it will help me review

@ntindle ntindle changed the title ci(frontend,design): add storybook builds to the frontend WIP- ci(frontend,design): add storybook builds to the frontend Jan 30, 2025
Copy link
Contributor

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

@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Jan 31, 2025
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 Review effort [1-5]: 1 size/l
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

4 participants