search

haystack / nb

11 stars 10 forks source link

Add TA role #272

Closed JumanaFM closed 1 year ago

JumanaFM commented 1 year ago

Add TA

Similar to adding an instructor, you can add TA by selecting TA. If the user already exists, remove it from the class first, then add it (All previous comments and interactions for this user are preserved and will be shown as a TA).

image

TA vs Student

TA and Student share the same privileges except for the following:

image

Escalation workflow

  1. TA can escalate a comment by clicking the reply request icon. image
  2. Instructor can see all pending requests by filtering using "Pending TA reply requests". image
  3. Instructor can resolve the request by replying to the comment.
  4. After the instructor reply, the request will be removed immediately from the pending requests.

Closes #271 Closes #37

karger commented 1 year ago

I thought we had a mechanism to switch someone between student and instructor status.  can't that just be broadened to switch between the three different types?  That's much simpler than separate remove and add.

On 2/10/2023 5:35 PM, Jumana Almahmoud wrote:

Add TA

Similar to adding an instructor, you can add TA by selecting TA. If the user already exists, remove it from the class first, then add it (All previous comments and interactions for this user are preserved and will be shown as a TA).

image https://user-images.githubusercontent.com/22763091/218207116-8a0b06d2-dd3a-4bd6-9aea-a2f29057734c.png

TA vs Student

TA and Student share the same privileges except for the following:

  • TA has a flag (icon) next to its comments and replies.

image https://user-images.githubusercontent.com/22763091/218208003-06e6647b-f5a6-4380-99e0-87f623b605f6.png

Closes #271 https://github.com/haystack/nb/issues/271 Closes #37 https://github.com/haystack/nb/issues/37

    You can view, comment on, or merge this pull request online at:

https://github.com/haystack/nb/pull/272

    Commit Summary

(10 files https://github.com/haystack/nb/pull/272/files)

— Reply to this email directly, view it on GitHub https://github.com/haystack/nb/pull/272, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIWSXS475A7W75HJCGF34DWW27EXANCNFSM6AAAAAAUYJRPBA. You are receiving this because you are subscribed to this thread.Message ID: @.***>

JumanaFM commented 1 year ago

@karger it’s on our issue list and it’s still pending https://github.com/haystack/nb/issues/43 i marked it as “UROP”, and “Good first issue”