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

Resolve 'BugBountyCards.js' todo. Fixes #4303 #4905

Merged
merged 2 commits into from
Jan 5, 2022

Conversation

Hakeemmidan
Copy link
Contributor

Description

Resolves the todo at src/components/BugBountyCards.js:111 on the dev branch

Before After
138608761-015cde1b-3ab4-40eb-b653-e1dbc1e5d53d Screen Shot 2021-12-27 at 4 58 50 AM

It should be the same. I couldn't spot a difference. Comment below if you do.

Related Issue

Fixes #4303

@gatsby-cloud
Copy link

gatsby-cloud bot commented Dec 27, 2021

Gatsby Cloud Build Report

ethereum-org-website-dev

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 6m

Performance

Lighthouse report

Metric Score
Performance 🔶 19
Accessibility 💚 97
Best Practices 💚 93
SEO 🔶 79

🔗 View full report

Copy link
Contributor

@minimalsm minimalsm left a comment

Choose a reason for hiding this comment

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

Hey @Hakeemmidan thanks for the PR on this and happy new year to you 😀✌️

Looks good overall, left a few comments but nothing major so feel free to disagree.

src/components/BugBountyCards.js Outdated Show resolved Hide resolved
src/components/BugBountyCards.js Outdated Show resolved Hide resolved
src/components/BugBountyCards.js Outdated Show resolved Hide resolved
src/components/BugBountyCards.js Outdated Show resolved Hide resolved
src/components/BugBountyCards.js Outdated Show resolved Hide resolved
src/components/BugBountyCards.js Outdated Show resolved Hide resolved
@Hakeemmidan
Copy link
Contributor Author

Hi @minimalsm,

Thank you for the response, and happy new year to you as well! 🎇🎈✨

I adjusted the code based on the feedback.


Please let me know if you have any questions or requests
Cheers

Copy link
Member

@corwintines corwintines left a comment

Choose a reason for hiding this comment

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

Thanks @Hakeemmidan! This is much cleaner!

@Hakeemmidan
Copy link
Contributor Author

@corwintines Thank you! 🙏 Glad to help

@minimalsm
Copy link
Contributor

Thanks again @Hakeemmidan 🚀

@minimalsm minimalsm merged commit 1f11fe7 into ethereum:dev Jan 5, 2022
@Hakeemmidan
Copy link
Contributor Author

@minimalsm Thank you! 🙏 Glad to help

@Hakeemmidan Hakeemmidan deleted the resolve-todo_fixes-4303 branch January 5, 2022 15:14
@corwintines corwintines mentioned this pull request Jan 5, 2022
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.

Resolve Todo at src/components/BugBountyCards.js:111 on dev Branch
3 participants