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

[TECH] Ajout de la colonne de typage des souscriptions à une certification (PIX-1221). #8751

Merged

Conversation

Steph0
Copy link
Contributor

@Steph0 Steph0 commented Apr 25, 2024

🦄 Problème

Nous voulons faire porter la notion d’inscription d’un candidat à une certif Pix coeur de manière détachée d’une certif complémentaire. Mais aujourd’jui nous assumons que un candidat passe toujours un pix coeur.

🤖 Proposition

Préparer l'arrivée d'une souscription de type "coeur" en typant l'existant (choisi à COMPLEMENTARY)

🌈 Remarques

💯 Pour tester

  • Sur Pix certif avec certif-pro@example.net
  • Ajouter un candidat via l'import de candidats avec une complémentaire
  • Ajouter un candidat via l'import de sessions avec une complémentaire
  • Ajouter candidat via la modale avec une complémentaire

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@Steph0 Steph0 force-pushed the pix-12211-add-type-to-certification-subscriptions branch from 7c09c5d to e807de1 Compare April 26, 2024 14:35
@Steph0 Steph0 requested a review from AndreiaPena April 26, 2024 14:36
return databaseBuffer.pushInsertable({
tableName: 'certification-subscriptions',
values,
values: {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question: on renommera les complementaryCertificationSubscriptions dans le code lorsqu'on renommera la table ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ca sera probablement le meilleur moment, mais il va falloir en tout cas 😅

Comment on lines +6 to +9
await knex.schema.dropView('certification-subscriptions');
return knex.schema.createView('certification-subscriptions', function (view) {
view.as(knex('complementary-certification-subscriptions'));
});
Copy link
Member

@AndreiaPena AndreiaPena Apr 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question : Ah oui donc pour faire matcher la vue par rapport à la table on doit forcément la supprimer ? :o

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'avais lu cela

ALTER VIEW changes various auxiliary properties of a view. (If you want to modify the view's defining query, use CREATE OR REPLACE VIEW.)

Pas d'impacts cependant car les migrations sont transactionnels, il n'y aura pas de moment où la vue aura disparue réellement

@AndreiaPena AndreiaPena added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed labels Apr 29, 2024
@Steph0 Steph0 force-pushed the pix-12211-add-type-to-certification-subscriptions branch from e807de1 to c89e209 Compare May 2, 2024 15:43
@pix-service-auto-merge pix-service-auto-merge merged commit 6736c99 into dev May 2, 2024
9 of 10 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-12211-add-type-to-certification-subscriptions branch May 2, 2024 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants