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

Mein Profil Request /LegalRepresentatives und ApprenticeshipContracts/Current ausführen Rolle Student - 1PT #727

Open
schefbi opened this issue Aug 6, 2024 · 7 comments
Assignees
Labels
module-PersonProfil Feature fürs Modul "Personenprofil"

Comments

@schefbi
Copy link
Contributor

schefbi commented Aug 6, 2024

my-profile: Die Requests GET /LegalRepresentatives und ApprenticeshipContracts/Current nur ausführen, wenn der Benutzende über die Rolle StudentRole im Token verfügt. Sonst gibt es einen 403 Error Response.

Image

Beispiel: GymBivo, [email protected] (nur AuthenticatedApplicantRole & DefaultRole) / s1 (StudentRole)

@schefbi schefbi moved this to Sprint-Backlog in Gravatscha Aug 6, 2024
@schefbi schefbi added module-PersonProfil Feature fürs Modul "Personenprofil" Front-End labels Aug 6, 2024
@fbufbi fbufbi removed the Front-End label Aug 6, 2024
@hupf
Copy link
Collaborator

hupf commented Aug 7, 2024

@schefbi Ich nehme an im Fall wo die Rolle fehlt zeigen wir analog wie bei der Stellvertretung den Text «Sie haben keinen Zugriff auf Ihr Profil» innerhalb von «Mein Profil» an, oder?

@fbufbi
Copy link
Collaborator

fbufbi commented Aug 7, 2024

@schefbi Ich nehme an im Fall wo die Rolle fehlt zeigen wir analog wie bei der Stellvertretung den Text «Sie haben keinen Zugriff auf Ihr Profil» innerhalb von «Mein Profil» an, oder?

@hupf Das Problem ist, dass es bei Personen ohne spezifische Rolle wegen den zwei in der Beschreibung aufgeführten Requests zu einem 403 kommt:
image

Werden diese beiden Requests nur bei Personen mit StudentRole abgesetzt, kommen die Fehler bei Personen ohne spezifische Rolle nicht mehr. Diese haben neu lesenden Zugriff auf ihr Profil (Bugfix SLH).

@hupf
Copy link
Collaborator

hupf commented Aug 12, 2024

@fbufbi Okay, ich interpretiere aus deiner Antwort, dass es also darum geht, die entsprechenden Daten ohne StudentRole nicht zu laden und es soll keine Fehlermeldung angezeigt werden.

@Amasit Amasit changed the title Mein Profil Request /LegalRepresentatives und ApprenticeshipContracts/Current ausführen Rolle Student Mein Profil Request /LegalRepresentatives und ApprenticeshipContracts/Current ausführen Rolle Student - 1PT Sep 5, 2024
@hupf hupf moved this from Sprint-Backlog to In progress in Gravatscha Dec 9, 2024
@hupf hupf self-assigned this Dec 9, 2024
@hupf
Copy link
Collaborator

hupf commented Dec 9, 2024

@fbufbi Mit welchem User sollte ich dieses Problem reproduzieren können? Ich habe mit allen uns bekannten Users getestet und es kommt immer ein 200 zurück (allenfalls mit leerer Liste), auch wenn die Rolle StudentRole nicht vorhanden ist.

Kann es sein, dass dieses Problem gar nicht mehr existiert?

@hupf hupf moved this from In progress to Review in Gravatscha Dec 9, 2024
@fbufbi
Copy link
Collaborator

fbufbi commented Dec 9, 2024

@fbufbi Mit welchem User sollte ich dieses Problem reproduzieren können? Ich habe mit allen uns bekannten Users getestet und es kommt immer ein 200 zurück (allenfalls mit leerer Liste), auch wenn die Rolle StudentRole nicht vorhanden ist.

Kann es sein, dass dieses Problem gar nicht mehr existiert?

@hupf Der User [email protected] ist verloren gegangen - ich habe ihn wieder erstellt und mit diesem User kommt der 403 bei den oben angegebenen Requests, wenn man auf "Mein Profil" geht.

@hupf
Copy link
Collaborator

hupf commented Dec 9, 2024

@fbufbi Danke, dann macht es Sinn den Change zu machen, PR ist erstellt.

@hupf hupf moved this from Review to To test in Gravatscha Dec 11, 2024
@hupf hupf assigned fbufbi and unassigned hupf Dec 11, 2024
@fbufbi
Copy link
Collaborator

fbufbi commented Dec 11, 2024

Test OK

@fbufbi fbufbi moved this from To test to To document in Gravatscha Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module-PersonProfil Feature fürs Modul "Personenprofil"
Projects
Status: To document
Development

No branches or pull requests

3 participants