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

Introduce public pages for authorization requests #412

Merged
merged 3 commits into from
Sep 9, 2024

Conversation

skelz0r
Copy link
Member

@skelz0r skelz0r commented Sep 6, 2024

Thanks to uuid public id it is possible to have a public page for each
authorization request.

Currently not exposed within the app, maybe later.
Copy link
Contributor

@JeSuisUnCaillou JeSuisUnCaillou left a comment

Choose a reason for hiding this comment

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

C'est super que la page existe, mais si on donne le lien nulle part ça sert à rien, non ?

@skelz0r
Copy link
Member Author

skelz0r commented Sep 9, 2024

C'est super que la page existe, mais si on donne le lien nulle part ça sert à rien, non ?

Cf les 2 liens au dessus de ta review (tu peux lire le lien matermost).

tl;dr: ça va être donner sur API Particulier pour les sous-FDs pour éviter de dev la fontionnalité de sous-rôle sur DataPass qui ne sert que dans le cas d'API Particulier.

@skelz0r
Copy link
Member Author

skelz0r commented Sep 9, 2024

(je ne code pas des trucs pour le fun hein 😅)

@JeSuisUnCaillou
Copy link
Contributor

(je ne code pas des trucs pour le fun hein 😅)

J'en doute pas, mais quand je comprend pas je demande, même si ça me paraît absurde ;)

@JeSuisUnCaillou
Copy link
Contributor

Mais même en ayant lu les discussions, je ne pige toujours pas où on va se servir de ce lien public.

Copy link
Contributor

@JeSuisUnCaillou JeSuisUnCaillou left a comment

Choose a reason for hiding this comment

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

Le code est ok, donc je valide

@JeSuisUnCaillou
Copy link
Contributor

Mais même en ayant lu les discussions, je ne pige toujours pas où on va se servir de ce lien public.

Vu qu'on n'a pas de bouton "partager publiquement cette page" ou whatever, comment quelqu'un fait pour obtenir ce lien public ?

@skelz0r
Copy link
Member Author

skelz0r commented Sep 9, 2024

Le lien sera affiché dans une question metabase sur https://particulier.api.gouv.fr/datapass (donc ici directement lu en db) + envoyé par email par API Particulier (extrait des webhooks).

Related etalab/admin_api_entreprise#1614

@Isalafont
Copy link
Contributor

Le lien sera affiché dans une question metabase sur https://particulier.api.gouv.fr/datapass (donc ici directement lu en db) + envoyé par email par API Particulier (extrait des webhooks).

Related etalab/admin_api_entreprise#1614

Merci pour le lien, effectivement je me posais la même que @JeSuisUnCaillou et avec le ticket related aide mieux à tout comprendre.

Copy link
Contributor

@Isalafont Isalafont left a comment

Choose a reason for hiding this comment

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

Ce serait bien de peut être garder la référence de la PR api part liée dans le linear également ?
Ça faciliterait la compréhension globale de cette PR depuis Linear.

@skelz0r
Copy link
Member Author

skelz0r commented Sep 9, 2024

Y'a aucun ticket linear pour le coup 😅
La meilleure ref reste le thread Mattermost.

@skelz0r skelz0r merged commit 40dfab6 into develop Sep 9, 2024
11 checks passed
@skelz0r skelz0r deleted the features/public-id branch September 9, 2024 13:43
@JeSuisUnCaillou
Copy link
Contributor

Y'a aucun ticket linear pour le coup 😅 La meilleure ref reste le thread Mattermost.

Flemme de lire toute une discussion pour avoir la description d'une feature

@skelz0r
Copy link
Member Author

skelz0r commented Sep 9, 2024

Flemme de lire toute une discussion pour avoir la description d'une feature

La description de la feature est dans features/page_publique_habilitation.feature

Vous pouvez faire un ticket si vous voulez, ça change rien pour moi c'est strictement équivalent, du moment qu'il y a des refs pour tirer les fils pour moi ça me suffit (et tbh, on rarement du contexte dans les tickets linear VS une conversation mattermost :D ).

@Isalafont
Copy link
Contributor

Les tickets linear servent également à remplir les livrables demandés par Malt et la DINUM.

@skelz0r
Copy link
Member Author

skelz0r commented Sep 10, 2024

Ah intéressant, nous on exporte github (qui est le plus accurate)

@skelz0r
Copy link
Member Author

skelz0r commented Sep 10, 2024

(après y'a pas besoin de débattre, ajoutez vos tickets si vous voulez)

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.

3 participants