Skip to content

Job de suppression d'université - #4110

Merged
SebouChu merged 9 commits into
mainfrom
destroy-university-job
Sep 7, 2026
Merged

Job de suppression d'université#4110
SebouChu merged 9 commits into
mainfrom
destroy-university-job

Conversation

@SebouChu

@SebouChu SebouChu commented Jun 15, 2026

Copy link
Copy Markdown
Member

Type

  • Nouvelle fonctionnalité
  • Bug
  • Ajustement
  • Rangement

Description

Destruction d'une université et de toutes ses dépendances.
Supprimer une université a évidemment une forte incidence, mais le code lui-même, non (donc moyenne).

Niveau d'incidence

  • Incidence faible 😌
  • Incidence moyenne 😲
  • Incidence forte 😱

@arnaudlevy arnaudlevy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • La logique est impeccable, mais les listes devraient être dans les modèles, pas dans un job, c'est trop important.

Comment thread app/jobs/destroy_university_job.rb Outdated
@SebouChu
SebouChu merged commit a1b2b70 into main Sep 7, 2026
8 checks passed
@SebouChu
SebouChu deleted the destroy-university-job branch September 7, 2026 13:30
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.

2 participants