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

Fix button not disabled #7716

Merged
merged 3 commits into from
Oct 27, 2020

Conversation

walidmujahid
Copy link
Contributor

Description

Disables the join buttons on hackathon onboard after after first click to prevent double clicking. On failure, button is renabled.

Refers/Fixes

fixes #7290 (comment)

@walidmujahid
Copy link
Contributor Author

walidmujahid commented Oct 20, 2020

@octavioamu my apologies for the delay. Here is the fix for #7290 (comment)

@codecov
Copy link

codecov bot commented Oct 25, 2020

Codecov Report

Merging #7716 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7716      +/-   ##
==========================================
+ Coverage   25.42%   25.44%   +0.01%     
==========================================
  Files         328      328              
  Lines       33044    33044              
  Branches     4919     4919              
==========================================
+ Hits         8401     8407       +6     
+ Misses      24356    24350       -6     
  Partials      287      287              
Impacted Files Coverage Δ
app/dashboard/embed.py 31.60% <0.00%> (+3.44%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 204c0cd...f7d99f8. Read the comment docs.

@thelostone-mc thelostone-mc merged commit 4085305 into gitcoinco:master Oct 27, 2020
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