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

website: Fix stories, add session decorator and button stories #1552

Merged
merged 1 commit into from Feb 15, 2023

Conversation

rjmacarthy
Copy link
Contributor

@rjmacarthy rjmacarthy commented Feb 13, 2023

This PR fixes all the error preventing stories from loading.

  • Added SessionDecorator as useSession requires a provider
  • Added RouterDecorator for to support usage of useRouter hook.
  • Added a story for Button components
  • Fixed some linter warnings

Any problems please let me know.

Copy link
Collaborator

@AbdBarho AbdBarho left a comment

Choose a reason for hiding this comment

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

Thanks

Use mock router and create router decorator

pre-commit
@github-actions
Copy link

pre-commit failed.
Please run pre-commit run --all-files locally and commit the changes.
Find more information in the repository's CONTRIBUTING.md

1 similar comment
@github-actions
Copy link

pre-commit failed.
Please run pre-commit run --all-files locally and commit the changes.
Find more information in the repository's CONTRIBUTING.md

Copy link
Collaborator

@AbdBarho AbdBarho left a comment

Choose a reason for hiding this comment

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

Thank you!

@AbdBarho AbdBarho merged commit b9ac0ac into LAION-AI:main Feb 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants