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: duplicate activity comments #6008

Merged

Conversation

walidmujahid
Copy link
Contributor

@walidmujahid walidmujahid commented Feb 18, 2020

When commenting/replying on activities/status posts, comments can be duplicated by hitting the return key more than once Disabling the textarea for every call of post_comment would solve the duplication problem.

closes #6007

When commenting/replying on activities/status posts, comments can be duplicated by hitting the return key more than once:

https://youtu.be/Btage0nfqzY

Disabling the textarea for every call of post_comment would solve the duplication problem.

fixes gitcoinco#6007
@codecov
Copy link

codecov bot commented Feb 18, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6008      +/-   ##
==========================================
+ Coverage   28.86%   28.89%   +0.02%     
==========================================
  Files         272      272              
  Lines       23978    23978              
  Branches     3498     3498              
==========================================
+ Hits         6922     6928       +6     
+ Misses      16781    16775       -6     
  Partials      275      275
Impacted Files Coverage Δ
app/dashboard/embed.py 31.6% <0%> (+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 a8ad43f...51b68d8. Read the comment docs.

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.

Duplicate Activity comments
3 participants