Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

General: Allow instructors to navigate from course management to student view #7597

Merged
merged 3 commits into from
Nov 17, 2023

Conversation

MaximilianAnzinger
Copy link
Collaborator

@MaximilianAnzinger MaximilianAnzinger commented Nov 14, 2023

Checklist

General

Client

  • Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data.
  • I followed the coding and design guidelines.
  • 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.
  • I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
  • I documented the TypeScript code using JSDoc style.
  • I added multiple screenshots/screencasts of my UI changes.
  • I translated all newly inserted strings into English and German.

Motivation and Context

Instructors have no easy way to navigate from course management back to the student view.
Resolves #7596 .

Description

Adds same button we use to navigate from the student view to management to the management view to navigate back.

Steps for Testing

Prerequisites:

  • 1 Instructor
  • 1 Student
  • 1 Course
  1. Log in to Artemis as Instructor
  2. Navigate to Course Management of your Course
  3. Click the Student view button to navigate back to the student view
  4. Make sure you can now see the Manage button again
  5. Repeat this for subpages (e.g. Statistics, Lectures, etc.). You should always get redirected to the course main page or the main management page respectively.
  6. Log in to Artemis as Student
  7. Navigate to the Course
  8. Make sure you can't see the Manage button

Testserver States

Note

These badges show the state of the test servers.
Green = Currently available, Red = Currently locked







Review Progress

Performance Review

  • I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance

Code Review

  • Code Review 1
  • Code Review 2

Manual Tests

  • Test 1
  • Test 2

Test Coverage

Client

Class/File Line Coverage Confirmation (assert/expect)
header-course.component.ts 100%

Screenshots

Screenshot 2023-11-14 at 20 21 04

@github-actions github-actions bot added tests client Pull requests that update TypeScript code. (Added Automatically!) labels Nov 14, 2023
@MaximilianAnzinger MaximilianAnzinger marked this pull request as ready for review November 14, 2023 19:35
@MaximilianAnzinger MaximilianAnzinger requested a review from a team as a code owner November 14, 2023 19:35
AntonGeTUM
AntonGeTUM previously approved these changes Nov 15, 2023
Copy link

@AntonGeTUM AntonGeTUM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Manually tested, works as expected

xHadie
xHadie previously approved these changes Nov 15, 2023
Copy link

@xHadie xHadie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Manually tested on TS3_Legacy
Works as expected!

Gusti2010
Gusti2010 previously approved these changes Nov 15, 2023
Copy link

@Gusti2010 Gusti2010 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Manually tested on TS11_Legacy. Everything works as expected!

Copy link

@JanaNF JanaNF left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Manual tested on legacy_ts2. Worked as expected.

Copy link
Contributor

@Strohgelaender Strohgelaender left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code

@github-actions github-actions bot added the deployment-error Added by deployment workflows if an error occured label Nov 15, 2023
@LenaKahle LenaKahle removed the deployment-error Added by deployment workflows if an error occured label Nov 15, 2023
@LenaKahle LenaKahle temporarily deployed to artemis-test5.artemis.cit.tum.de November 15, 2023 13:41 — with GitHub Actions Inactive
@egekurt123
Copy link
Contributor

Manually tested on artemis-test-4: Everything works as expected when acting as a student, however when I go to the communication or messages subpage as an instructor it automatically puts me into the student "view" (I can change back to instructor view however with the manage button) I don't know if this is wanted? Screenshot 2023-11-15 at 14 27 29

This problem also occurs in other versions. There is already an issue ticket for that.

@Ceiridge
Copy link

Tested on test5 and it works as expected, except that it puts me back to another subpage when I click on Student view

Copy link
Contributor

@JohannesStoehr JohannesStoehr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good to me

Copy link

@georgyia georgyia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on two services manually (ts1, ts3), tested with artemis_test_user_11 and artemis_test_user_12. There is student view button.

Снимок экрана 2023-11-15 в 19 50 33 Снимок экрана 2023-11-15 в 19 50 41

@MaximilianAnzinger
Copy link
Collaborator Author

Manually tested on artemis-test-4: Everything works as expected when acting as a student, however when I go to the communication or messages subpage as an instructor it automatically puts me into the student "view" (I can change back to instructor view however with the manage button) I don't know if this is wanted? Screenshot 2023-11-15 at 14 27 29

This problem also occurs in other versions. There is already an issue ticket for that.

Is this really an issue? I always thought that was intentional. I mean what would you expect to see differently on the communication view?

@MaximilianAnzinger
Copy link
Collaborator Author

Tested on two services manually (ts1, ts3), tested with artemis_test_user_11 and artemis_test_user_12. There is student view button.

Снимок экрана 2023-11-15 в 19 50 33 Снимок экрана 2023-11-15 в 19 50 41

Are you sure you used the correct test servers (legacy vs gh)? It seems pretty unlikely that it's working on so many servers bot not on these.

Copy link
Contributor

@sarpsahinalp sarpsahinalp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems to be working fine. But when pressed on Communication or Messages it seems to turning back to student view no matter what I do. It seems this issue is already mentioned on another issue.

@MaximilianAnzinger
Copy link
Collaborator Author

MaximilianAnzinger commented Nov 17, 2023

It seems to be working fine. But when pressed on Communication or Messages it seems to turning back to student view no matter what I do. It seems this issue is already mentioned on another issue.

Afaik this is intentional. Maybe @lennart-keller can say more about this.
When clicking on those tabs you essentially navigate to the normal conversation page which is not part of the management view.

@lennart-keller
Copy link
Contributor

It seems to be working fine. But when pressed on Communication or Messages it seems to turning back to student view no matter what I do. It seems this issue is already mentioned on another issue.

Afaik this is intentional. Maybe @lennart-keller can say more about this. When clicking on those tabs you essentially navigate to the normal conversation page which is not part of the management view.

Yes, this is intentional. There are no management pages for communication and messaging.

Copy link
Contributor

@sarpsahinalp sarpsahinalp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on ts5 again seems to be working.

@krusche krusche added this to the 6.6.7 milestone Nov 17, 2023
@krusche krusche merged commit 3d8bc19 into develop Nov 17, 2023
99 of 104 checks passed
@krusche krusche deleted the feature/student-view-button branch November 17, 2023 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client Pull requests that update TypeScript code. (Added Automatically!) ready to merge small tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add link to "student course view" to "course management view"