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

#221/add storybook #228

Merged
merged 29 commits into from
Jun 5, 2024
Merged

#221/add storybook #228

merged 29 commits into from
Jun 5, 2024

Conversation

camelPhonso
Copy link
Contributor

Description

**Closes #221 **

📚 This PR adds Storybook to our stack and introduces a few basic story file examples for our components along with documentation changes explaining this addition to newcomers.

🎭 Playwright has been removed as it was the original testing suite - now replaced by Cypress for E2E and Storybook for component tests

Files changed

  • .eslintrc.json, package.json & package-lock.json - edited with the installation of Storybook and dependent add-ons
  • `TESTING.md - edited with guidance on how to build and maintain our component library/tests
  • ./storybook/** - setup files for our storybook environment
  • **.stories.tsx - story files for a few of our components. These are the start of our component library and can serve as examples for future implementations
  • **.spec.ts - playwright test suites deleted as they are no longer in use
  • .gitignore - edited to ignore new storybook log and all references to playwright have been removed to keep the file from bloating
  • documentation images added to show in TESTING.md the implementation of Storybook

UI changes

No UI changes to the app but visual guides have been added to our documentation

Changes to Documentation

  • TESTING.md updated as described above

Tests

YES! 🦆 - Component stories have been added for the following components:

  • MessageCard.tsx
  • ItemCard.tsx
  • SystemMessageCard.tsx
  • ConversationCard.tsx

Copy link

netlify bot commented May 29, 2024

Deploy Preview for cool-creponne-3e1272 ready!

Name Link
🔨 Latest commit 7f724aa
🔍 Latest deploy log https://app.netlify.com/sites/cool-creponne-3e1272/deploys/666086d798d3bd0008b77c42
😎 Deploy Preview https://deploy-preview-228--cool-creponne-3e1272.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
Contributor

@nichgalzin nichgalzin left a comment

Choose a reason for hiding this comment

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

Looks great. Excited to really grow this as we continue on. Into it!

@camelPhonso camelPhonso merged commit 04703c3 into dev Jun 5, 2024
2 checks passed
@camelPhonso camelPhonso deleted the #221/add-storybook branch June 5, 2024 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

📚 Add Storybook
2 participants