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

[FEATURE] Ne pas afficher le bandeau "vous semblez ne pas être en France" dans les DOM-TOMs (PIX-7713) #525

Merged

Conversation

clemlatz
Copy link
Member

@clemlatz clemlatz commented Apr 14, 2023

🦄 Problème

La bannière "vous semblez ne pas être en France" est affichée pour les utilisateurs situés dans les DOM-TOM.

🤖 Proposition

Ajouter les codes pays dans les DOM-TOM en plus de celui de la France dans les exceptions à l'affichage de la bannière.

Pays Code
la Guadeloupe GA
la Guyane GF
la Martinique MQ
la Réunion RE
Mayotte YT
la Nouvelle-Calédonie NC
la Polynésie française PF
Saint-Barthélemy BL
Saint-Martin MF
Saint-Pierre-et-Miquelon PM
les Terres australes et antarctiques françaises TF
les îles de Wallis-et-Futuna WF

🌈 Remarques

RAS

💯 Pour tester

  • Accéder à https://pix.fr/ en utilisant un VPN pour se géolocaliser dans un DOM-TOM
    • constater que la bannière est affichée.
  • Accéder à https://site-pr525.review.pix.fr/ en utilisant un VPN pour se géolocaliser dans un DOM-TOM
    • constater que la bannière n'est pas affichée.

@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 sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-site-review-pr525/environment

@clemlatz clemlatz force-pushed the pix-7713-hide-out-of-france-banner-in-dom-toms branch from a67f80a to 9ba7758 Compare April 14, 2023 14:31
@clemlatz clemlatz marked this pull request as ready for review April 14, 2023 14:59
services/should-show-out-of-france-banner.js Outdated Show resolved Hide resolved
services/should-show-out-of-france-banner.js Outdated Show resolved Hide resolved
@lego-technix
Copy link
Contributor

lego-technix commented Apr 14, 2023

Je n'ai malheureusement pas pu tester fonctionnellement car je ne dispose pas d'un VPN me permettant de me géolocaliser dans un DOM-TOM.

@clemlatz clemlatz force-pushed the pix-7713-hide-out-of-france-banner-in-dom-toms branch from 9ba7758 to 9e41a4d Compare April 17, 2023 10:05
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