Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • C chouette-core
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 36
    • Merge requests 36
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Infrastructure Registry
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Administrator
  • chouette-core
  • Merge requests
  • !939

Merged
Created Oct 01, 2018 by Administrator@rootMaintainer

8157 syncs clean

  • Overview 2
  • Commits 5
  • Changes 6

Created by: zog

Bon en fait je pense que l'approche n'est pas la bonne, voici pour moi une meilleure façcon de faire:

1/ tu gardes ton concern, mais tu l'inclues dans les LineReferentialSync et StopAreaReferentialSync 2/ tu déclenches le nettoyage dans un after_create, comme ça on est surs qu'il est appelé (pas de risque de l'oublier dans un controlleur). Par contre faire attention de bien "scoper" les objets que tu supprimes (tu veux en garder X pour chaque LineReferential/StopAreaReferential, pas X en tout)

Je t'ai fait un petit refacto du concern pour pouvoir modifier le nombre d'objets qu'on peut garder. Tu peux par exemple t'en servir dans les specs pour ne pas créer 50 objets.

Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Source branch: 8157-syncs-clean