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

[DRAFT][Permissions]Ajout des filtres sur la portée des données #263

Conversation

MathRdt
Copy link

@MathRdt MathRdt commented Dec 8, 2023

Pour faire suite à l'issue 249 et refléter le comportement que l'on a côté Occtax, il serait intéressant d'implémenter un filtre sur la portée des données pour les différents objets de monitoring (mes données, données de mon organisme, toutes les données).

Pour les besoins du CREA, je vais tenter de l'implémenter dans cette PR, à la manière d'occtax pour avoir le comportement le plus similaire possible.

Je pense que les principales difficultés seront liées à :

  • la généricité du module (on manipule différents types d'objets au sein d'une même route API)
  • l'héritage:
    • Si j'ai le droit de voir mon site, mais qu'il existe des visites qui n'ont pas été créées par moi, ai je droit de les voir ?
    • Si j'ai le droit de voir les visites, mais pas les sites, que se passe t'il ? Est ce que je peux quand même voir les sites où mes visites sont présentes ou rien du tout ?

Preneur de toute aide ou remarque sur le sujet.

@amandine-sahl
Copy link
Contributor

Bonjour,

En fait cette fonctionnalité est en cours de finalisation avec la PR #238

@MathRdt
Copy link
Author

MathRdt commented Dec 8, 2023

Au temps pour moi, bonne nouvelle alors !
Merci pour ta réactivité, et si besoin de tests utilisateurs ou autre, je suis disponible.

Je ferme donc cette PR,
Bonne journée !

@MathRdt MathRdt closed this Dec 8, 2023
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