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

Create bounty form #113

Merged
merged 5 commits into from
Jun 7, 2024
Merged

Create bounty form #113

merged 5 commits into from
Jun 7, 2024

Conversation

claudioantonio
Copy link
Contributor

This PR is to call a google form when users click on Create bounty button, given that currently we are not able to allow users to create their bounties permissionlessly.
@guidanoli I will create it as draft to wait until you merge the one related to form validation, because your PR has a file on /utils to add the URLs. The google form url should be there also.

@claudioantonio claudioantonio added the a-frontend Area: front-end label Jun 6, 2024
@claudioantonio claudioantonio added this to the Testnet 2/2 milestone Jun 6, 2024
@claudioantonio claudioantonio requested a review from guidanoli June 6, 2024 20:12
@claudioantonio claudioantonio self-assigned this Jun 6, 2024
Copy link

vercel bot commented Jun 6, 2024

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

Name Status Preview Comments Updated (UTC)
bug-buster ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 7, 2024 3:42pm

* Move external links to a single file
* Validate form fields in `/bounty/create`
* Validate form fields in `/bounty/<id>/sponsor`
* Validate form fields in `/bounty/<id>/exploit`
@claudioantonio claudioantonio marked this pull request as ready for review June 7, 2024 13:43
@claudioantonio
Copy link
Contributor Author

Ready for review, @guidanoli ! 😉

@guidanoli
Copy link
Contributor

This needs a rebase.

Copy link
Contributor

@guidanoli guidanoli left a comment

Choose a reason for hiding this comment

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

Please run pnpm prettier.

Copy link
Contributor

@guidanoli guidanoli left a comment

Choose a reason for hiding this comment

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

LGTM

@claudioantonio claudioantonio merged commit c788a75 into main Jun 7, 2024
2 checks passed
@claudioantonio claudioantonio deleted the feature/create-bounty-form branch June 7, 2024 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-frontend Area: front-end
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants