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

Drops a Grants Twitter verification if the user modifies the twitter_handle_1 field #8305

Merged

Conversation

gdixon
Copy link
Contributor

@gdixon gdixon commented Jan 26, 2021

Description

https://gitcoin.co/grants/1924/gitcoin-grants-round-9-dev-fund
https://gitcoin.co/grants/1927/gitcoin-grants-official-matching-pool-fund-round-

Users are able to modify a Grants twitter handle after the verification process.

Replicated here: https://gitcoin.co/grants/1930/test-2

This pull will clear the verification if the twitter_handle_1 field is modified.

Refers/Fixes

clsoes #8303

Testing

Untested - I don't have my local instance set-up with a twitter_access_token yet, I can't see anything going wrong, but let me know if you want me to test it 👍

@gdixon gdixon force-pushed the bugfix/edit-grant-twitter-verification branch from 1ca1aca to 3d33f33 Compare January 26, 2021 01:11
@thelostone-mc
Copy link
Member

@gdixon -> could you test it out and throw in a video?
You could mark a handle as verified and record on how updating the handle -> resets the verification

@gdixon
Copy link
Contributor Author

gdixon commented Jan 26, 2021

@thelostone-mc - I've set my local instance up with some Twitter keys and everything seems to be running smoothly (with some minor alterations to the front-end)...

https://www.loom.com/share/af8222c3b2ec481e95cdc262bbb5a3ee

@thelostone-mc thelostone-mc merged commit 7dcab26 into gitcoinco:master Jan 26, 2021
@gitcoinbot
Copy link
Member

⚡️ A tip worth 0.08000 ETH (114.83 USD @ $1435.38/ETH) has been granted to @gdixon for this issue from @owocki. ⚡️

Nice work @gdixon! Your tip has automatically been deposited in the ETH address we have on file.

@lthrift lthrift mentioned this pull request Mar 10, 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.

3 participants