Skip to content

feat(crm): check if notif target is a project#1922

Draft
Jeremy-Bojko wants to merge 1 commit intodevelopfrom
fix/crm-trace#1076
Draft

feat(crm): check if notif target is a project#1922
Jeremy-Bojko wants to merge 1 commit intodevelopfrom
fix/crm-trace#1076

Conversation

@Jeremy-Bojko
Copy link
Collaborator

Il s'agit de :

  • Une correction de bug
  • Une nouvelle fonctionnalité programmée
  • Une nouvelle fonctionnalité spontanée

Décrivez vos changements

Soit en texte, soit en images ou les deux.

=> Liens vers des éléments de Github Issue, Trello, ...

Checklist d'acceptation de revue de code

  • Aucun texte ne fait référence à du vocabulaire spécifique d'un métier
  • Mon code est auto-documenté ou la documentation a été mise à jour/créée
  • La gestion du multi-portail a été prise en compte
  • L'accessibilité a été prise en compte
  • Pre-commit est configuré et a été lancé
  • Des tests couvrant le code changé ont été ajoutés/modifiés
  • L'ensemble des tests front et back sont au vert

Demandes

  • Je souhaite un déploiement en préproduction

@alice-telescoop
Copy link
Collaborator

Je me suis rendue compte qu'il y avait aussi des règles à part dans activity_action.html :/ J'ai l'impression qu'on voudrait avoir un template unique quitte à ce que le style varie d'une page à l'autre, mais c'est sujet à problème qu'il y ait trois templates différents avec des règles d'affichage spécifiques. Je vais un peu travailler dessus dans #1076 mais je voudrais bien qu'on discuter de comment aborder ça mieux

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.

2 participants