(Translated by https://www.hiragana.jp/)
API Particulier: remove responsable de traitement, add contact metier by skelz0r · Pull Request #327 · etalab/data_pass · GitHub
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

API Particulier: remove responsable de traitement, add contact metier #327

Open
wants to merge 18 commits into
base: develop
Choose a base branch
from
Open

API Particulier: remove responsable de traitement, add contact metier #327

wants to merge 18 commits into from

Conversation

skelz0r
Copy link
Member

@skelz0r skelz0r commented Aug 2, 2024 •
edited
Loading

Screenshot 2024-09-06 at 18 45 15

A quoi ressemblera le texte de l'événement de mise à jour global --^

Modifiable ici: https://github.com/etalab/data_pass/blob/api-particulier/remove-respo-traitement/db/migrate/20240906162647_create_mass_update_for_api_particulier_on_contact_metier.rb#L22

Screenshot 2024-09-19 at 19 20 38

La modal affichée aux demandeurs --^

@skelz0r skelz0r self-assigned this Aug 2, 2024
@skelz0r skelz0r force-pushed the api-particulier/remove-respo-traitement branch 2 times, most recently from 68db164 to 20171ac Compare August 2, 2024 16:35
@skelz0r skelz0r force-pushed the api-particulier/remove-respo-traitement branch from 20171ac to 995f7be Compare August 5, 2024 12:57
@skelz0r skelz0r marked this pull request as draft August 5, 2024 15:01
@skelz0r
Copy link
Member Author

skelz0r commented Aug 5, 2024

Je passe en draft car ce n'est pas clair ce qui doit être fait ici. A priori il faut introduire un contact métier pour remplacer, mais il manque des infos sur les données historiques.

Plus d'infos ici: https://mattermost.incubateur.net/betagouv/pl/jur4gxfp9jysty6johgd4bcsxo

@skelz0r
Copy link
Member Author

skelz0r commented Aug 12, 2024

On migre le responsable de traitement en contact métier.

@skelz0r skelz0r changed the title API Particulier: remove responsable de traitement API Particulier: remove responsable de traitement, add contact metier Aug 12, 2024
@skelz0r skelz0r force-pushed the api-particulier/remove-respo-traitement branch 2 times, most recently from bd9715e to ca94817 Compare August 12, 2024 15:29
@skelz0r skelz0r marked this pull request as ready for review August 12, 2024 15:30
@skelz0r
Copy link
Member Author

skelz0r commented Aug 12, 2024

Le commit important ca94817

@skelz0r
Copy link
Member Author

skelz0r commented Aug 13, 2024

Je me demande si on ne devrait pas faire un event d'explication aussi pour l'historique, wdyt @JeSuisUnCaillou ?

@JeSuisUnCaillou
Copy link
Contributor

Je me demande si on ne devrait pas faire un event d'explication aussi pour l'historique, wdyt @JeSuisUnCaillou ?

Un event qui décrit le transfert des infos d'un responsable_traitement vers un contact_metier c'est ça ?
Bof, j'en vois pas tellement l'utilité. T'imagines qu'on en aurait quel usage éventuel ?

JeSuisUnCaillou
JeSuisUnCaillou approved these changes Aug 13, 2024
View reviewed changes
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.

pour moi c'est go

@skelz0r
Copy link
Member Author

skelz0r commented Aug 13, 2024

Je me demande si on ne devrait pas faire un event d'explication aussi pour l'historique, wdyt @JeSuisUnCaillou ?

Un event qui décrit le transfert des infos d'un responsable_traitement vers un contact_metier c'est ça ? Bof, j'en vois pas tellement l'utilité. T'imagines qu'on en aurait quel usage éventuel ?

Historiser les changements. Ça donnerait du contexte pour l'instructeur, et des potentiels explications pour le demandeur (quand on affichera l'historique (pourquoi c'est pas fait d'ailleurs ..?))

@skelz0r
Copy link
Member Author

skelz0r commented Aug 13, 2024

Un event genre mass_update qui est lié à une AuthorizationDefinition

@JeSuisUnCaillou
Copy link
Contributor

Qui va se rendre compte de la différence entre contact métier et responsable traitement ? Pour moi c'est du zèle, mais enjoy doing it :D

@skelz0r
Copy link
Member Author

skelz0r commented Aug 13, 2024

Qui va se rendre compte de la différence entre contact métier et responsable traitement ? Pour moi c'est du zèle, mais enjoy doing it :D

En règle générale je suis d'accord, sauf que là on parle d'habilitations juridiques entre plusieurs administrations, je ne suis pas sûr qu'on puisse faire ça en mode yolo non plus.

@JeSuisUnCaillou
Copy link
Contributor

Qui va se rendre compte de la différence entre contact métier et responsable traitement ? Pour moi c'est du zèle, mais enjoy doing it :D

En règle générale je suis d'accord, sauf que là on parle d'habilitations juridiques entre plusieurs administrations, je ne suis pas sûr qu'on puisse faire ça en mode yolo non plus.

Je pense que ça se yolo bien, mais encore une fois, enjoy doing it, j'y vois pas d'inconvénient.

@skelz0r
Copy link
Member Author

skelz0r commented Aug 13, 2024

Je pense que ça va attendre le retour de Nicolas ou Vincent ici.

@skelz0r
Copy link
Member Author

skelz0r commented Aug 13, 2024

(Je repasse en draft)

@skelz0r skelz0r marked this pull request as draft August 13, 2024 09:27
@SchweisguthN
Copy link
Contributor

@skelz0r Pour moi RT et contact métier sont 2 choses différentes. Je suis pas ultra fan de passer les coordonnées du RT dans "contact métier" pour l'historique, puisque ça n'est pas un contact métier. Je serais plus pour dire qu'on n'affiche plus les coordonnées du RT sur ces habilitations, mais le problème est plutôt : est-ce que ces habilitations peuvent exister sans contact métier renseigné.

JeSuisUnCaillou
JeSuisUnCaillou reviewed Sep 20, 2024
View reviewed changes
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.

few comments

app/controllers/authorization_request_forms_controller.rb Outdated Show resolved Hide resolved
features/mise_a_jour_global_habilitation.feature Show resolved Hide resolved
@skelz0r skelz0r force-pushed the api-particulier/remove-respo-traitement branch from 7d1c56e to 3f0eb81 Compare September 20, 2024 09:35
@skelz0r
Copy link
Member Author

skelz0r commented Sep 20, 2024

@JeSuisUnCaillou c'est BULK rename c'est bon :D

JeSuisUnCaillou
JeSuisUnCaillou approved these changes Sep 20, 2024
View reviewed changes
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 bon pour moi

@skelz0r
Copy link
Member Author

skelz0r commented Sep 20, 2024

Je vais merge/deploy ça post migration d'HubEE.

@skelz0r
Enhance DeliverGDPRContactsMails tests
f0d3dce
@skelz0r
API Particulier: remove responsable de traitement
e4945bf
@skelz0r
Cucumber: introduce contact technique step
15bf9e7 
This is a really common contact for a lot of authorizations
@skelz0r
Cucumber: introduce contact métier step
3555740 
Will be introduced for API Particulier in a next commit
@skelz0r
API Particulier: introduce contact métier
1c73fb7
@skelz0r
Introduce migration for API Particulier contacts
1a47560 
Migrate keys from responsable traitement to contact metier.
This migration should be really fast thanks to mass update. It skips
ActiveRecord validations as well (thanks to SQL query)

Here a script to test in local: https://gist.github.com/skelz0r/319fba9b9160f054873142ea55db02ff
@skelz0r
Introduce StaticApplicationRecord.exists?
65e54d2 
Sugar method yeah
@skelz0r
Introduce BulkAuthorizationRequestUpdate
8f7fdb2
@skelz0r
Introduce AuthorizationRequest#bulk_updates
713c64d
@skelz0r
Introduce AuthorizationRequestEvent bulk_update event
4d52c74
@skelz0r
BulkAuthorizationRequestUpdate#reason can interpolate vars
dbfef53
@skelz0r
Create bulk update for API Particulier contact metier migration
93e287b
@skelz0r
Scout commit: move denial block to summary
91cded9 
All alerts are within this view, simplify by moving this lonely alert
@skelz0r
Scout commit
6377d8e
@skelz0r
Harden Authorization#request_as_validated
58fb26c 
Add created_at, can be use within views
@skelz0r
Make modal customisable
e8aee05 
* Can use layout multiple times
* Can trigger on load the modal
@skelz0r
Introduce BulkAuthorizationRequestUpdateNotificationRead
e3d29f9
@skelz0r
Introduce bulk update modal for applicants
ab0c561 
On the first visit post bulk update on authorization requests, the
system displays a modal to inform applicant (only, not other users)
to the changes.
@skelz0r skelz0r force-pushed the api-particulier/remove-respo-traitement branch from 3f0eb81 to ab0c561 Compare September 21, 2024 08:18
@JeSuisUnCaillou
Copy link
Contributor

Ping sur cette PR, elle devient quoi ?

@skelz0r
Copy link
Member Author

skelz0r commented Oct 21, 2024

ah oui faut la mettre en prod celle-ci

@JeSuisUnCaillou
Copy link
Contributor

ah oui faut la mettre en prod celle-ci

Ca veut dire que tu t'en charges ?

@skelz0r
Copy link
Member Author

skelz0r commented Oct 21, 2024

Oui

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JeSuisUnCaillou JeSuisUnCaillou JeSuisUnCaillou approved these changes

@jbfeldis jbfeldis Awaiting requested review from jbfeldis

@SchweisguthN SchweisguthN Awaiting requested review from SchweisguthN

@Isalafont Isalafont Awaiting requested review from Isalafont

Assignees

@skelz0r skelz0r

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@skelz0r @JeSuisUnCaillou @SchweisguthN @Isalafont @evaspae