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

Fix/permissions migration #255

Merged
merged 6 commits into from
Nov 9, 2023
Merged

Fix/permissions migration #255

merged 6 commits into from
Nov 9, 2023

Conversation

amandine-sahl
Copy link
Contributor

@amandine-sahl amandine-sahl commented Oct 26, 2023

cf #249

@amandine-sahl amandine-sahl force-pushed the fix/permissions_migration branch from 7d38afd to 65e935f Compare October 26, 2023 15:03
@camillemonchicourt
Copy link
Member

Ceux qui ont déjà fait la MAJ auront pas récupéré les bonnes permissions ?

@amandine-sahl
Copy link
Contributor Author

amandine-sahl commented Oct 27, 2023

Non, c'est un fix de la PR #250 qui n'est pas encore dans une release.
Donc pas de soucis pour les utilisateurs.

@jacquesfize jacquesfize merged commit 74bf498 into develop Nov 9, 2023
6 checks passed
@jacquesfize
Copy link
Contributor

Avec @Pierre-Narcisi, nous faisons les tests sur monitoring pour la maj SQLA 1.4. Je vois que dans cette PR qu'un object_code a été ajouté aux check_cruved_scope (

@check_cruved_scope("E", object_code="MONITORINGS_MODULES")
) des routes dans monitorings.py. Une seule route n'a pas été modifié : list/<string:module_code>/<object_type>, est-ce que c'est normal ? Car celle-ci pose problème, en indiquant que l'utilisateur n'a pas les permissions nécessaires (ce qui est le cas).

@camillemonchicourt camillemonchicourt deleted the fix/permissions_migration branch August 1, 2024 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants