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.

Sign up for GitHub

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

Jump to bottom

ci: allow authentication using preview apps #77

Closed
bonjourmauko opened this issue Oct 10, 2023 · 3 comments
Closed

ci: allow authentication using preview apps #77

bonjourmauko opened this issue Oct 10, 2023 · 3 comments
Assignees
@streino
Labels
build Tout ce qui touche au build et CI

Comments

@bonjourmauko
Copy link
Contributor

bonjourmauko commented Oct 10, 2023 •
edited
Loading

Depends on ecolabdata/ecospheres#71

Amélioration technique

Pour faciliter le test par des stakeholders non-techniques, il est important que les évolutions qui nécessitent d'une authentification puissent être testés avant d'un merge sur « main » (en vue de ecolabdata/ecospheres#72).

Note : je ne mets pas de dépendance ici car il y a de limitantes techniques importantes à considérer.

L'exemple est que l'on puisse s'authentifier quand on teste les previews :

Screenshot 2023-11-23 at 18 16 28
@bonjourmauko bonjourmauko added the question Question ou discussion requise sur le sujet label Oct 10, 2023
@bonjourmauko bonjourmauko added this to the Data bouquets milestone Oct 10, 2023
@bonjourmauko bonjourmauko moved this to Pool of options in Ecosphères x Multi Oct 10, 2023
@streino
Copy link
Contributor

streino commented Oct 10, 2023

You mean from PR deploys on netlify?
If so, doesn't seem possible on data.gouv side: no globing. We'd have to add all possible URLs...
If you have another idea, I'm all ears!

@bonjourmauko
Copy link
Contributor Author

Mmm... 🤔 with a reverse proxy? But already sounds awfully complicated.

@bonjourmauko bonjourmauko modified the milestones: V1 : Créer ou éditer un bouquet, Continuous deployment, V1 : Déploiement « version bêta », V2 : Product / Market fit Oct 26, 2023
@bonjourmauko bonjourmauko changed the title Allow authentication using preview apps ci: allow authentication using preview apps Nov 23, 2023
@streino
Copy link
Contributor

streino commented Feb 21, 2024

Ca fonctionne maintenant !
opendatateam/udata#2935

@streino streino closed this as completed Feb 21, 2024
@streino streino added build Tout ce qui touche au build et CI and removed ci question Question ou discussion requise sur le sujet labels Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@streino streino

Labels
build Tout ce qui touche au build et CI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bonjourmauko @streino