Skip to content

US2 Final: Unresolve/resolve button#36

Merged
17 commits merged into
mainfrom
coverage-testing
Feb 11, 2026
Merged

US2 Final: Unresolve/resolve button#36
17 commits merged into
mainfrom
coverage-testing

Conversation

@PDFour4
Copy link
Copy Markdown

@PDFour4 PDFour4 commented Feb 10, 2026 •
edited by brendayyu
Loading

This PR updates the TA Question Resolution user story to maintain question resolution status for students and automatic unresolved question filtering for Admins/TA.

Frontend:

  • Unresolved button turns into a green resolved button when clicked
  • Green resolved button turns into a gray unresolved button when clicked
  • Only admins/TAs are able to toggle this button
  • Members (not admin/TAs) can view it but are not able to toggle it
  • Only available in the comments & feedback section
  • Posts are ordered with unresolved questions at the top
  • When posts are resolved, they move to the bottom for Admin/TAs view

Backend:

  • Edited appendResolveStatus to include filtering for Admins/TAs
  • Updated function call to posts.isResolved in post.tpl so students get accurate unresolved/resolved status.

How To Test:

  • Log in as a TA/Admin.
  • View Comments & Feedback section.
  • Go to a question.
  • Click "Unresolved" (it should turn green) and change to "Resolved".
  • Log in as a Student/Regular User.
  • Verify the button is visible and keeps the status of what the Admin changed it to.
  • Students do not have the unresolved/resolved filtering for the questions.

brendayyu and others added 15 commits February 7, 2026 21:16
@brendayyu
added an Unresolved button underneath each question
34367e8
@brendayyu
feat: add resolved/unresolved toggle button to posts
e629d9f 
- Add resolve button to post actions in topic template
- Implement client-side toggle functionality in resolve.js
- Button displays green checkmark when resolved, gray circle when unresolved
- Text shows 'Resolved' or 'Unresolved' state (hidden on mobile)
- Position button to the left of reply button in post toolbar
@brendayyu
fixed a syntax error
843b42c
@brendayyu
limited the resolve/unresolve function for questions/comments only, a…
23c6b82 
…nd only admins are able to toggle it
@brendayyu
removed unusual api
83662b9
@PDFour4
Feat: Backend logic for TA resolve and archive
93ae114
@PDFour4
Merge remote-tracking branch 'origin/frontend' into backend
cde0116 
Attaching the backend feature for the unresolved/resolved buttons done in the frontend.
@PDFour4
Feat: Complete TA Resolve Question (Backend + UI + Permissions)
a58777b
@bendnema
Added post-queue back from the file
6976995
@bendnema
Adjust post.tpl to fix frontend not remebering
ae6c5b7
@bendnema
Add TA resolve feature for topics in category 4
d594637
@PDFour4
Updated the view for students to see if their question is resolved or…
e46e1e7 
… not
@PDFour4
Merge branch 'backend' into coverage-testing
97ec8f3
@PDFour4
Added automatic unresolved filter for admins/ta and unresolved/resolv…
919330b 
…ed questions keep status for student view
@PDFour4
Attempted to fix plugins.yaml url issue
fa4cba1
This was linked to issues Feb 10, 2026
Front End/UI: Resolved/unresolved button at the bottom of each question #3
Closed
Backend: Organize questions in “resolved” and “unresolved” lists #18
Closed
Front End/UI: Users can see a resolved/unresolved button for each question in the “Comments & Feedback” dashboard #20
Closed
Front End/UI: Questions marked as resolved are automatically moved to the bottom ONLY for TA/admin views #15
Closed
Permissions: Student does not see archived view #6
Closed
@brendayyu brendayyu changed the title Final? US 2 US2 Final: Unresolve/resolve button Feb 10, 2026
This was linked to issues Feb 10, 2026
Permissions: TA and Admins are the only ones who can resolve discussions #9
Closed
Permissions: Frontend Permissions for Users, hid role management from UI for non-admins #13
Closed
This was unlinked from issues Feb 10, 2026
Permissions: TA and Admins are the only ones who can resolve discussions #9
Closed
Permissions: Frontend Permissions for Users, hid role management from UI for non-admins #13
Closed
This was linked to issues Feb 10, 2026
Testing: Does not change after page refresh #28
Closed
Testing: TAs/Instructors can resolve a discussion with one action #31
Closed
Testing: Only authorized users can resolve/unresolve discussions #32
Closed
@PDFour4 PDFour4 closed this pull request by merging all changes into main in 76bf730 Feb 11, 2026
@PDFour4 PDFour4 linked an issue Feb 13, 2026 that may be closed by this pull request
Permissions: Frontend Permissions for Users, hid role management from UI for non-admins #13
Closed
@brendayyu brendayyu added the good first issue Good for newcomers label Feb 28, 2026
@brendayyu brendayyu added this to the Sprint 1 milestone Feb 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment