Show better message to students not enrolled on courses

Aalto-LeTech / aplus-courses

A+ Courses is a plugin for IntelliJ IDEA, used in programming courses at Aalto University

https://plugins.jetbrains.com/plugin/13634-a-courses

MIT License

16 stars 7 forks source link

Show better message to students not enrolled on courses #1133

Open jaakkonakaza opened 3 weeks ago

jaakkonakaza commented 3 weeks ago

Currently courses that do not have a public /api/v2/courses/$id endpoint show the message correctly but since it is public for O1, a generic network error message is shown.

/api/v2/users/me could be checked for enrollment, but since it does not show the course for TAs they would not be able to access the course. After this issue on aplus is fixed we can use the users/me endpoint

oseppala commented 1 day ago

Another possible case is that the A+ token has been changed. The message could also say "... or you have created a new A+ token"