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

Optimisation du traitement de l'obsolescence des connexions #1885

Merged
merged 17 commits into from
May 17, 2024

Conversation

SebouChu
Copy link
Member

@SebouChu SebouChu commented May 17, 2024

on met en synchrone le Communication::Website::Connection#destroy_if_obsolete

Cette méthode est appelée uniquement par Communication::Website#destroy_obsolete_connections

La méthode destroy_obsolete_connections est appelée :

  • Dans le Communication::CleanWebsiteJob donc côté worker
  • Dans le Communication::Website#clean_and_rebuild_safely donc côté worker
  • Dans le Communication::Website#connect_about donc côté web (⚠️ problème)

@arnaudlevy arnaudlevy merged commit 57908d4 into main May 17, 2024
5 checks passed
@arnaudlevy arnaudlevy deleted the optimize-obsolete-connections-job branch May 17, 2024 14:37
@arnaudlevy arnaudlevy changed the title Optimize obsolete connections job Optimisation du traitement de l'obsolescence des connexions May 17, 2024
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.

None yet

3 participants