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

NT-2001:UX – Show pending reply dialog #1330

Merged
merged 17 commits into from
Jul 20, 2021

Conversation

hadia
Copy link
Contributor

@hadia hadia commented Jul 19, 2021

πŸ“² What

Show pending comment dialog in replies thread activity

πŸ€” Why

Add confirmation dialog when users have a failed post or something written in the comment composer in reply thread and try to Go back to root comments

πŸ›  How

Checking if there are pending replies handling back action with dialog

πŸ‘€ See

device-2021-07-19-215435.mp4

πŸ“‹ QA

add pending reply try back action submit reply and then try back again

Story πŸ“–

https://kickstarter.atlassian.net/browse/NT-2001

@codecov
Copy link

codecov bot commented Jul 19, 2021

Codecov Report

Merging #1330 (ee1d3df) into master (0f6eea3) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1330   +/-   ##
=========================================
  Coverage     74.12%   74.12%           
  Complexity      739      739           
=========================================
  Files           221      221           
  Lines          6666     6666           
  Branches        406      406           
=========================================
  Hits           4941     4941           
  Misses         1589     1589           
  Partials        136      136           

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 0f6eea3...ee1d3df. Read the comment docs.

@Arkariang Arkariang merged commit c5dc291 into master Jul 20, 2021
@Arkariang Arkariang deleted the fix/Show_pending_reply_dialog branch July 20, 2021 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants