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: update post when comment is marked as answer; reload topics on open [BD-38] [TNL-9646] [TNL-9703] #104
Conversation
Thanks for the pull request, @xitij2000! I've created BLENDED-1163 to keep track of it in Jira. More details are on the BD-38 project page. When this pull request is ready, tag your edX technical lead. |
Codecov Report
@@ Coverage Diff @@
## master #104 +/- ##
==========================================
+ Coverage 85.41% 86.34% +0.92%
==========================================
Files 100 100
Lines 1838 1845 +7
Branches 451 452 +1
==========================================
+ Hits 1570 1593 +23
+ Misses 253 238 -15
+ Partials 15 14 -1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
- I tested this: (checked video of the expected behaviour)
- I read through the code
- I checked for accessibility issues
- Includes documentation
- I made sure any change in configuration variables is reflected in the corresponding client's
configuration-secure
repository.
When a comment is marked as an answer, refresh the post it's in so the answered tag appears. When navigating to the post tab, refresh the topic list.
ddf8cda
to
59d0932
Compare
@xitij2000 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
… according to figma (openedx#104) Co-authored-by: MehakNasir <mehaknasir@A006-00194.local>
When a comment is marked as an answer, refresh the post it's in so the answered tag appears.
When navigating to the post tab, refresh the topic list.
Peek.2022-03-25.15-43.mp4