Skip to content

Demandeur d’emploi : Suppression du champ de l'aide ATA - #5312

Merged
rsebille merged 1 commit into
masterfrom
rsebille/jobseeker-remove-ata-fields
Jan 6, 2025
Merged

Demandeur d’emploi : Suppression du champ de l'aide ATA#5312
rsebille merged 1 commit into
masterfrom
rsebille/jobseeker-remove-ata-fields

Conversation

@rsebille

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Ce champ n'a jamais été utilisé, le champ dans le formulaire a été enlevé avant la mise en prod : a7fb1cb

SELECT ata_allocation_since, COUNT(*) FROM users_jobseekerprofile GROUP BY 1;
 ata_allocation_since |  count  
----------------------+---------
                      | 1161215

🍰 Comment ?

Je fait un RemoveField() "bourrin" car un peu (beaucoup) la flemme de faire une dance SeparateDatabaseAndState() mais vu qu'on touche au JobSeekerProfile() ça serais peut-être mieux que de passer la PR hors heures d'affluences.

@rsebille rsebille added the supprimé Supprimé dans le changelog. label Dec 26, 2024
@rsebille rsebille self-assigned this Dec 26, 2024

@xavfernandez xavfernandez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Moi je suis toujours plutôt pour que les mises en prod se fassent sans coupure (:handshake: SeparateDatabaseAndState) 🤷‍♂️
Mais si tu préfères merger quand il n'y a personne ça me va aussi.

@rsebille
rsebille force-pushed the rsebille/jobseeker-remove-ata-fields branch from d060632 to 0e685b5 Compare January 6, 2025 09:39
This field is not displayed nor filled since a7fb1cb.
@rsebille
rsebille force-pushed the rsebille/jobseeker-remove-ata-fields branch from 0e685b5 to 0d07c53 Compare January 6, 2025 09:49
@rsebille
rsebille added this pull request to the merge queue Jan 6, 2025
Merged via the queue into master with commit b47fe4a Jan 6, 2025
@rsebille
rsebille deleted the rsebille/jobseeker-remove-ata-fields branch January 6, 2025 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

supprimé Supprimé dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants