Artemis
Artemis copied to clipboard
`Communication`: Show author's role in course on posting header
Checklist
General
- [x] I tested all changes and their related features with all corresponding user types on a test server.
- [x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
- [x] I chose a title conforming to the naming conventions for pull requests.
Server
- [x] I followed the coding and design guidelines.
- [x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
- [x] I implemented the changes with a good performance and prevented too many database calls.
- [x] I documented the Java code using JavaDoc style.
Client
- [x] I followed the coding and design guidelines and ensured that the layout is responsive.
- [x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
- [x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
- [x] I documented the TypeScript code using JSDoc style.
- [x] I added multiple screenshots/screencasts of my UI changes.
- [x] I translated all newly inserted strings into English and German.
Motivation and Context
Currently, many communication tools like Zulip allow setting authorities to users. This allows others to easily understand the author's role and assess their trustworthiness. Having this feature would be beneficial in Artemis.
Description
We add an icon in front of the author name in all Postings with a corresponding tooltip. The icon reveals the user's highest authority for that course and provides information about that user's role within its' tooltip. You are very welcome to suggest changes to these explanations and especially their German translations. :)Steps for Testing
Prerequisites:
- 1 Instructor
- 1 Student
- 1 Tutor
- 1 Editor
- 1 Course with Postings enabled
- Log in to Artemis
- Navigate to Course Communication
- Make sure that the corresponding icon with the appropriate tooltip is displayed before the author name for each Posting. Editors and Tutors are assigned the "Tutor" icon.
Review Progress
Code Review
- [x] Review 1
- [x] Review 2
Manual Tests
- [x] Test 1
- [x] Test 2
Test Coverage
Class/File | Branch | Line |
---|---|---|
PostService.java | 81% | 98% |
posting-header.directive.ts | 92.3% | 100% |
Screenshots
https://user-images.githubusercontent.com/67344158/181029286-56e8d9d3-4e85-40ec-b16e-7a39c55fec47.mp4
tested on ts2: When creating a new post, the icons are not correctly shown in the preview (they are all user icons - but the icon of the Admin Test Account should of course be the instructor icon):
And if I create a new post with an tutor account, first it is also displayed with an user account. Once I refresh the page, it shows the correct icon.
Nice catch! Thanks 👍🏻
Code looks good, thanks for integrating the review.
Thank you very much for your feedback, it helped a lot!
I agree with @JohannesStoehr regarding user vs. student. Our documentation also talks about students (whereas a user is anyone with an account).
Term was updated, thanks 👍🏻
Code looks good.
One thing you could think about is changing the name of "user" to "student", which would be a bit more fitting in my opinion
It makes sense to me as well, so I have updated the term :)