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

Issue4/create rachael t card #41

Merged

Conversation

rachael-t
Copy link
Contributor

@rachael-t rachael-t commented Oct 14, 2021

Summary of changes

Added my card to the scrollodex

Related Issue

#4

How to test (for features) / reproduce (for bugs)

Run yarn start and the my card should load (see screenshot below)

Screenshots

Screen Shot 2021-10-14 at 3 15 10 PM

Changes include

  • Bugfix (non-breaking change that solves an issue)
  • New feature (non-breaking change that adds functionality)
  • Documentation (explains but does not affect functionality)
  • Breaking change (change that is not backwards-compatible and/or changes current functionality)

Notes

husky

@netlify
Copy link

netlify bot commented Oct 14, 2021

✔️ Deploy Preview for scrollodex ready!

🔨 Explore the source changes: 767805b

🔍 Inspect the deploy log: https://app.netlify.com/sites/scrollodex/deploys/6168f5a6bdb8140008ea14a8

😎 Browse the preview: https://deploy-preview-41--scrollodex.netlify.app/

@domsbytes domsbytes self-requested a review October 15, 2021 03:28
Copy link
Member

@domsbytes domsbytes left a comment

Choose a reason for hiding this comment

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

Updates look good!

@domsbytes
Copy link
Member

Very nicely formatted PR @rachael-t ! and I love the GIF 💯

@domsbytes domsbytes merged commit 7dca69b into women-who-software:main Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants