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

Rework des dépendances à la sync #536

Merged
merged 5 commits into from
Oct 5, 2022
Merged

Conversation

SebouChu
Copy link
Member

@SebouChu SebouChu commented Oct 3, 2022

Pour que les objets sur le repo d'un website soit bien supprimés, il est important que les 2 points suivants soient respectés :

  • La méthode git_dependencies d'un objet doit lister tous les éléments liés à cet objet. (Par exemple, une école aura en dépendance toutes ses formations)
  • La méthode git_path d'un objet doit renvoyer nil si l'objet ne doit pas être synchronisé (Par exemple, une formation non publiée).

La boucle complète serait donc la suivante :

  • On synchronise une école
  • Elle annonce toutes ses formations en dépendances
  • Chaque dépendance (chaque formation) gère, via son git_path si elle doit être présente sur le site ou non

Copy link
Member

@arnaudlevy arnaudlevy left a comment

Choose a reason for hiding this comment

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

Il y a une vérif à enlever dans person et c'est bon je crois !

@arnaudlevy arnaudlevy merged commit 6dbf58c into master Oct 5, 2022
@arnaudlevy arnaudlevy deleted the rework-dependencies branch October 5, 2022 09:03
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

2 participants