Skip to content

Conversation

corwintines
Copy link
Contributor

Updates the comment for ethereum.org gitpoap for 2025

Copy link

vercel bot commented Jan 23, 2025

@corwintines is attempting to deploy a commit to the GitPOAP Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Jan 26, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gitpoap-bot ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 26, 2025 1:43pm

Copy link
Contributor

@colfax23 colfax23 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 overall, just one comment on the copy. Let me know what you think. Excited to get this out!

src/utils.ts Outdated
// Hardcode extra comments for ethereum-org-website
const ethereumOrgWebsiteComment =
'Be sure to join the [Ethereum.org discord](https://ethereum.org/discord) if you are interested in contributing further to the project or have any questions for the team.';
'Join the [ethereum.org Discord server](https://ethereum.org/discord) to explore more ways to contribute to the project. Depending on the tasks you complete, you may also unlock additional badges (OATs). Visit [ethereum.org/contributing](https://ethereum.org/contributing) to learn more.';
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd rather not advertise other badge platforms through the GitPOAP bot - do you mind changing it just to "you may also unlock additional badges" or " ... additional rewards"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can change this for sure!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Took this out now, let me know if theres anything else needed :)

Copy link
Contributor

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

@colfax23 colfax23 merged commit 55978d3 into gitpoap:main Feb 5, 2025
1 check failed
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.

2 participants