...
- Appearance / availability / functionality of post actions and labels based on user role
- reporting (flagging) - always available to all users
- Students' reported status should only indicate whether or not they individually reported the post
- TAs / Mods / Admins see the reported status when any user has reported the post
- When TAs / Mods / Admins un-report reported posts, the changed status should take effect for all users
- voting - always available to all user types on threads and responses (not comments)
- pinning - only changeable by TAs / Mods / Admins
- closing - only changeable by TAs / Mods / Admins
- endorsing - only for discussion responses - only changeable by TAs / Mods / Admins
- marking as answer - only for question responses - only changeable by thread author and TAs / Mods / Admins
- following - available to all user types at thread level only
- editing and deleting - always available to TAs / Mods / Admins or the content author
- reporting (flagging) - always available to all users
- Correct behavior for Discussion threads vs Question threads
- endorsement rules differ for authors - they can only endorse (mark answers) on question threads
- responses marked as question answers are always clearly designated as such (appear in a separate section, include answer marker and date info)
- answered and unanswered questions are always clearly distinguished from discussion threads in the sidebar
- New sidebar thread list filters
- Flagged thread filter is only available to TAs / Mods / Admins
- Unread thread filter accurately filters unread threads for the presently active user
- Filters correctly interact with category selection (discussion categories, "posts i'm following" filter)
- searches
- Tabs vs. inline: thread display should look and behave the same in both tab and inline mode
- AJAX - post actions (voting, pinning, following, reporting, closing) should be exercised to ensure they continue to function correctly.
- Errors can be deliberately triggered by killing the comments service and/or the LMS
- Closed threads: we should ensure we haven't regressed behavior with regard to closed threads.
- example: on recent inspection of our branch the "add a comment" box currently appears on closed threads, which may or may not be a regression.
Sandbox Info
http://forums.m.sandbox.edx.org/
in addition to the standard provisioned user accounts, there's a few extra set up for convenience
- jsa+ta@edx.org / edx (a community TA enrolled in edx 101)
- jsa+moderator@edx.org / edx (a moderator enrolled in edx 101)
- also, staff@example.com has been granted the discussion admin role in edx 101
Code Version
https://github.com/edx/edx-platform/tree/gprice/forum-action-cleanup / commit: afd9aa35e4dffe391b6915d386fe5a5fe46174f5
We have deployed this sandbox from the tip of a working branch because we have a few more minor changes to make but we don't want to block testing from starting.
In general, here is what we can expect to change in a subsequent update:
- a handful of accessibility-related adjustments (making new controls available via keyboard / screen reader) - not expected to change present appearance or mouse interactions
- a bunch of javascript test fixes (which should be entirely invisible)
- a merge / rebase of this branch (not expected to impact functionality at all)
We will keep this section updated depending on any discussed / agreed-upon plans to redeploy.