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

Submitting Work Twice #2118

Closed
vs77bb opened this issue Aug 29, 2018 · 5 comments
Closed

Submitting Work Twice #2118

vs77bb opened this issue Aug 29, 2018 · 5 comments
Assignees

Comments

@vs77bb
Copy link
Contributor

vs77bb commented Aug 29, 2018

Describe the bug
As a bounty hunter, I shouldn't see the Submit Work button if I've already submitted work.

To Reproduce
Have seen it in the past and as referenced by Arjun Nemani (Gitcoiner) here.

image

Expected behavior
[comment]: # (A clear and concise description of what you expected to happen.)
Submit Work button not available for the bounty hunter who already submitted work.

@thelostone-mc thelostone-mc self-assigned this Aug 29, 2018
@thelostone-mc thelostone-mc added the bug This is something that isn't working as intended. label Aug 29, 2018
@PixelantDesign
Copy link
Contributor

I think we should monitor to see if other users are confused by this. Technically it isn't broken or a bug. The activity list does let the user know that he has already submitted work.

@PixelantDesign PixelantDesign removed the bug This is something that isn't working as intended. label Aug 29, 2018
@thelostone-mc
Copy link
Member

This does ensure that newbies submit their work once!
Multiple submissions means I spend more gas which as a contributor I would not want to.
Users may get confused and may click on submit work if they don't scroll down to see the activity stream 🤔

cc @vs77bb @SaptakS

@vs77bb
Copy link
Contributor Author

vs77bb commented Aug 30, 2018

If they've already submitted work, the user can only lose money by clicking submit work again. Thus, removing that button (and the possibility of user error) seems like a win!

@thelostone-mc
Copy link
Member

@vs77bb merged 👍

@vs77bb
Copy link
Contributor Author

vs77bb commented Aug 31, 2018

@thelostone-mc Great work!

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

No branches or pull requests

3 participants