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

Feat: CTA other projects backend (with frontend merged) #1214

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

Conversation

ttdm
Copy link
Collaborator

@ttdm ttdm commented Oct 13, 2024

Feature principale :

  • Ajout d'un 3ème type d'opportunité possible

Feature technique majeure :

  • Refacto de nombreux éléments de la gestion d'opportunité dans le backend avec notamment l'ajout d'une classe OpportunityAssociatedObject qui permet une meilleure gestion des différetns payloads associés aux opportunité

Petit bonus sympa :

  • maj de la valeu rpar défaut du point d'api opporutnité de swagger qui est de nouveau utilisable

Test effectués :

  • A partir de swagger : créations d'opportunité des 3 types, vérification de la prise en compte de chaque type d'opportunité dans le compte des opportunités transmises et donc vérification de la limite de transmission à 1 opportunité/joru et par contact à PDE.

Remarque : le Theme n'est pas utilisé dans le backend. Clarifier les souhaits de l'équipe par rapport à ce champ !
close #1173
close #1066

@ttdm ttdm self-assigned this Oct 13, 2024
@oumeimaelisbihani oumeimaelisbihani marked this pull request as ready for review October 16, 2024 14:02
@ttdm ttdm requested review from dolemoine and removed request for dolemoine October 16, 2024 21:57
@oumeimaelisbihani oumeimaelisbihani removed the request for review from dolemoine October 17, 2024 08:18
@betagouv betagouv deleted a comment from ttdm Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants