Skip to content

Vie privée : replanification du traitement de notification et d'archivage des utilisateurs non actifs - #6184

Merged
vincentporte merged 1 commit into
masterfrom
vp/fix_jobapplication_sender_link_against_notify_archive_jobseeker
May 27, 2025
Merged

Vie privée : replanification du traitement de notification et d'archivage des utilisateurs non actifs#6184
vincentporte merged 1 commit into
masterfrom
vp/fix_jobapplication_sender_link_against_notify_archive_jobseeker

Conversation

@vincentporte

@vincentporte vincentporte commented May 22, 2025

Copy link
Copy Markdown

🤔 Pourquoi ?

Reactivation de notify_archive_users
Pre-requis : #6185

  • éviter le cas d'un user à notifier / archiver qui serait sender d'une candidature liée à un autre job_seeker avec approval

@vincentporte vincentporte self-assigned this May 22, 2025
@vincentporte vincentporte added the modifié Modifié dans le changelog. label May 22, 2025
Comment thread itou/archive/management/commands/notify_archive_users.py Outdated
@francoisfreitag

Copy link
Copy Markdown
Member

L'archivage de la candidature libèrera l'archivage de cet utilisateur.

On n’archive pas les candidatures acceptées, donc il ne sera jamais archivé avec les règles actuelles, non ?

ARCHIVABLE_JOB_APPLICATION_STATES = [
JobApplicationState.NEW,
JobApplicationState.PROCESSING,
JobApplicationState.POSTPONED,
JobApplicationState.REFUSED,
JobApplicationState.CANCELLED,
JobApplicationState.OBSOLETE,
]

@vincentporte
vincentporte force-pushed the vp/fix_jobapplication_sender_link_against_notify_archive_jobseeker branch from 559bc36 to 6720bff Compare May 22, 2025 16:00
@vincentporte

Copy link
Copy Markdown
Author

On n’archive pas les candidatures acceptées, donc il ne sera jamais archivé avec les règles actuelles, non ?

Hum, il y a une ambiguité. ARCHIVABLE_JOB_APPLICATION_STATES concernent les candidatures qui peuvent être archivées (rendues invisibles dans la liste).
Le traitement actuel notify_archive_users anonymise et détruit des données sans utiliser cet enum.

gasp

@vincentporte

Copy link
Copy Markdown
Author

patch inutile avec #6185
je limite cette PR à la replanification de la tâche cron

@vincentporte
vincentporte force-pushed the vp/fix_jobapplication_sender_link_against_notify_archive_jobseeker branch from 6720bff to 5a73d98 Compare May 22, 2025 16:57
@vincentporte vincentporte changed the title Vie privée : exclure les emetteurs de candidature si le candidat possède un PASS IAE Vie privée : replanification du traitement de notification et d'archivage des utilisateurs non actifs May 26, 2025

@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.

nit: ça aurait été plus propre de revert 0b70485

@vincentporte
vincentporte force-pushed the vp/fix_jobapplication_sender_link_against_notify_archive_jobseeker branch from 5a73d98 to 918ce9a Compare May 27, 2025 15:04
@vincentporte
vincentporte added this pull request to the merge queue May 27, 2025
Merged via the queue into master with commit ce0a19d May 27, 2025
@vincentporte
vincentporte deleted the vp/fix_jobapplication_sender_link_against_notify_archive_jobseeker branch May 27, 2025 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants