-
Notifications
You must be signed in to change notification settings - Fork 10
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
refactor: Refactors earn page #173
Conversation
@Clearwood @johannbarbie please review. [Adding in case auto review request is not generated]. Apologies for the delay in the PR. |
Hi @Clearwood, I rebased the PR with the latest changes from master. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The CTAs are missing.
here you can have a look at the hackmd detailing our motivations for reworking the content.
I am missing the drive to draw developers to us. I want everyone visiting this page to be stoked to work for LeapDAO.
src/routes/earn.svelte
Outdated
ease of use and security — all in one? | ||
<br> | ||
Want to get rewarded for your efforts while contributing to an open source initiative, join us and | ||
earn with LeapDAO by completing a bounty, taking on a role, or applying |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't have a grant program anymore. :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, I'll remove the grant program.
Hi @Clearwood, I've made changes to the content. Please review and let me know if it requires any other changes. Regarding the CTAs please let me know if anything else is required. |
Sorry for the late reply, I am missing more information on who and what we are as a community at the core. @mwaa in case you have time would also enjoy your feedback. |
As per our bounty terms and missing updates I am going to challenge the bounty and close the Pull Request. |
Fixes #165
For ParseError: Attributes need to be unique #168 - I tested with yarn version
1.12.3
the build seems to be successful, please let me know if I missed anything here)