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

Added my card to the scrollodex #134

Merged
merged 4 commits into from
Oct 23, 2022
Merged

Conversation

cassreinhart
Copy link
Contributor

Summary of changes

Added Cassandra Reinhart's card-- follow me on Github and connect on LinkedIn!

Related Issue

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

Screenshots

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

  • please consider including a gif that reflects your feelings about this PR
    hyped

@netlify
Copy link

netlify bot commented Oct 21, 2022

Deploy Preview for scrollodex ready!

Name Link
🔨 Latest commit 6c8c4c9
🔍 Latest deploy log https://app.netlify.com/sites/scrollodex/deploys/635595735149640008eeda6d
😎 Deploy Preview https://deploy-preview-134--scrollodex.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 settings.

Copy link
Member

@jischr jischr left a comment

Choose a reason for hiding this comment

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

great card! just a small update requested.

.gitignore Outdated Show resolved Hide resolved
I accidentally removed from the gitignore when I used the command line to ignore /venv locally
Copy link
Contributor Author

@cassreinhart cassreinhart left a comment

Choose a reason for hiding this comment

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

Let me know if my changes resolved the issue!

Copy link
Member

@jischr jischr 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!

@jischr jischr merged commit 06c2f54 into women-who-software:main Oct 23, 2022
@jenndiaz jenndiaz mentioned this pull request Oct 25, 2022
2 tasks
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