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

(PC-31860)[BO] feat: transfert all PRO links in BO to connect as exte… #14230

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

Meewan
Copy link
Contributor

@Meewan Meewan commented Sep 19, 2024

…nded

But de la pull request

Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-31860

Vérifications

  • J'ai écrit les tests nécessaires
  • J'ai mis à jour le fichier des plans de tests du portail pro si nécessaire
  • J'ai mis à jour la liste des routes et des titres de pages du portail pro si j'en ai rajouté/modifié ou supprimé une.
  • J'ai relu attentivement les migrations, en particulier pour éviter les locks, et je préviens les équipes Shérif et Data
  • J'ai ajouté des screenshots pour d'éventuels changements graphiques

Copy link
Contributor

@vroullier-pass vroullier-pass left a comment

Choose a reason for hiding this comment

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

ça me semble honnête, mais c'était obligatoire de décaler tout l'implémentation des liens dans le html (vs urls.py) ?

def _get_user_id_from_bank_account_id(bank_account_id: int) -> int:
if not FeatureToggle.WIP_CONNECT_AS_EXTENDED.is_active():
raise ValueError(
"L'utilisation de la version étendue de « connect as » requiert l'activation de la feature : WIP_CONNECT_AS_EXTENDED"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"L'utilisation de la version étendue de « connect as » requiert l'activation de la feature : WIP_CONNECT_AS_EXTENDED"
"L'utilisation de la version étendue de « connect as » requiert l'activation du feature flag : WIP_CONNECT_AS_EXTENDED"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

c'est le même message que les autres qui ne devrait jamais apparaitre (et qui disparaitra definitivement assez vite) donc je ne vais pas le modifier.

@Meewan
Copy link
Contributor Author

Meewan commented Sep 24, 2024

ça me semble honnête, mais c'était obligatoire de décaler tout l'implémentation des liens dans le html (vs urls.py) ?

Malheureusement oui, je ne peux pas utiliser les filtres facilement dans links.html. Je ne l'ai fait que pour build_offer_name_to_pc_pro_link et si tu regarde le code c'est un gros hack.

@prouzet prouzet force-pushed the pc-31860-rpa-connect-as-generalisation branch from f1f2ca5 to d778152 Compare September 26, 2024 16:41
@prouzet
Copy link
Contributor

prouzet commented Sep 26, 2024

Je n'ai fait qu'un rebase sur origin/master.

@Meewan Meewan force-pushed the pc-31860-rpa-connect-as-generalisation branch from d778152 to 3ad23e2 Compare September 30, 2024 10:12
@Meewan Meewan force-pushed the pc-31860-rpa-connect-as-generalisation branch from 3ad23e2 to 5169dd9 Compare September 30, 2024 11:34
@Meewan Meewan merged commit 55e4213 into master Oct 1, 2024
24 checks passed
@Meewan Meewan deleted the pc-31860-rpa-connect-as-generalisation branch October 1, 2024 07:56
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.

4 participants