Closed lduarte1991 closed 8 years ago
@lduarte1991 Looks pretty good to me. I think the "include X tab" approach works fine for the different use cases, and it makes sense to leave out the anonymous mode for now due to the CATCH changes that need to be made first.
👍
@arthurian Can you take a look at this? Basically instead of having distinct modes, I used the "include X tab" as a way to turn on/off the different use cases. I left anonymous off since that requires some changes to the CATCH if we do want it to be completely anonymous. If we want to do a hack for it where we hide them after they load, we can probably do that. Otherwise, this should add two more options: 1) "Include MyNotes tab" and 2)"Include Public tab" here are the use cases I have in mind:
MyNotes only -> Personal notes without instructor guidance, i.e. reading and taking notes for yourself Instructor only -> Instructor guidance without allowing them to create their own annotation. This mode should not allow users to make their own annotations. Public only -> Not a normal use case, recommend not doing. Buuuut it could happen if for example you want to show annotations from a previous version of the course that include more than instructor ones. This should be read only as well.
MyNotes + Instructor -> Personal notes with instructor guidance. Replies to instructors are still public. MyNotes + Public -> Open forum without instructor guidance Instructor + Public -> Same as Public only but with a filter to see just the instructor annotations.
All three -> Read/Write with filter to see your annotations only and instructor annotations only, or everyone.
Take a gander and let me know if you see anything wrong/worrisome.