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(data): Review validated fields for actors and projects #29

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

0live
Copy link
Contributor

@0live 0live commented Nov 6, 2024

La propriété isValidated est passée en Trait utilisé dans Actor et Project.
Du coup j'ai modifié les getters sur ton entité projet, mais il faudra penser à mettre à jour les setters sur les opérations de post/patch comme sur acteur:
image

Au niveau client, pour les acteurs, le store récupère toujours les données auquel l'utilisateur a droit (uniquement ce qui est validé pour un user, tout pour un admin). Sur la partie actors view, je filtre pour avoir que les données validées (pour éviter d'avoir les données à valider quand on est admin), et je récupère tout ce qui est dans le store. Je te laisse voir comment tu veux faire côté projets parce que j'ai vu que t'avais stocké les données filtrées dans le store.

Il faudra peut être adapter le provider pour renvoyer aussi les données créées par l'user même si pas validé, je ne sais pas si on veut qu'un user puisse éditer du contenu en cours de validation.

J'ai aussi créé une fonction dans le store de l'app qui reload toutes les data, cette fonction est appelée quand l'user se log/out

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