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] Supprime la colonne partnerKey (PIX-10162 ) #7659

Merged
merged 3 commits into from
Dec 15, 2023

Conversation

aceol
Copy link
Contributor

@aceol aceol commented Dec 11, 2023

🎄 Problème

Suite à #7447 la colonne partnerKey n'est plus utilisée

🎁 Proposition

La supprimer

🧦 Remarques

Il restait quelques references inutiles, principalement dans les tests

Il reste par ailleurs des "partnerKey" dans les certification-challenges. On pourrait les supprimer mais cela nécessite un travail approfondie. A faire dans une autre PR

🎅 Pour tester

Faire le parcours classique de certif complementaire, verifier que l'affichage sur admin et sur les certificats est ok

@aceol aceol force-pushed the pix-10162-remove-partnerKey-column branch from 1a49980 to a1134e7 Compare December 11, 2023 08:47
@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 :

@aceol aceol self-assigned this Dec 11, 2023
@aceol aceol force-pushed the pix-10162-remove-partnerKey-column branch 2 times, most recently from d78cc49 to e042f11 Compare December 12, 2023 10:35
@aceol aceol added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally team-certif labels Dec 12, 2023
@aceol aceol force-pushed the pix-10162-remove-partnerKey-column branch from e042f11 to e927fb5 Compare December 12, 2023 10:59
@aceol aceol marked this pull request as ready for review December 12, 2023 11:02
@Steph0
Copy link
Contributor

Steph0 commented Dec 14, 2023

Tech func avec @mariannebost

Sur Certif déjà passée

image
image
image

Sur session créée, passage de CléA

image
image
image

@Steph0 Steph0 added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed Need PO validation for this functionally labels Dec 14, 2023
@pix-service-auto-merge pix-service-auto-merge merged commit eddfcd0 into dev Dec 15, 2023
6 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-10162-remove-partnerKey-column branch December 15, 2023 09:08
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.

7 participants