Skip to content

feat: route revision sync and publish#177

Merged
fufeck merged 8 commits into
masterfrom
fufeck_feat_route-revision-sync-and-publish
May 13, 2026
Merged

feat: route revision sync and publish#177
fufeck merged 8 commits into
masterfrom
fufeck_feat_route-revision-sync-and-publish

Conversation

@fufeck

@fufeck fufeck commented May 5, 2026

Copy link
Copy Markdown
Contributor

CONTEXT

  • Intégration de la route revisions/:revisionId/sync-ids-ban-publish qui synchronise la BAL de la révision avec les identifants BAN et ensuite publie une nouvelle révision

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

En faite je suis vraiment sceptique par rapport à cette republication par un autre client : Y'a pleins de trucs qui sont gérés en regardant la current-revision d'une commune et le client dans la révision :

  • Tableau de déploiement
  • Le signalement (périmètre, activation et désactivation)
  • bal-widget (statut d'une commune)
  • Stepper de création mes-adresses (statut d'une commune)
    etc...

Est-ce qu'on pourrait pas plutôt republier avec le même client mais ajouter dans extras un flag? par exemple extras: {sourceRevisionId: req.revision.id, fixedBANIdsPublication: true}

Comment thread src/modules/revision/revision.controller.ts Outdated
@fufeck

fufeck commented May 13, 2026

Copy link
Copy Markdown
Contributor Author

En faite je suis vraiment sceptique par rapport à cette republication par un autre client : Y'a pleins de trucs qui sont gérés en regardant la current-revision d'une commune et le client dans la révision :

  • Tableau de déploiement
  • Le signalement (périmètre, activation et désactivation)
  • bal-widget (statut d'une commune)
  • Stepper de création mes-adresses (statut d'une commune)
    etc...

Est-ce qu'on pourrait pas plutôt republier avec le même client mais ajouter dans extras un flag? par exemple extras: {sourceRevisionId: req.revision.id, fixedBANIdsPublication: true}

Ca roule, je vais faire en sorte que cela soit publié avec le même client, j'avais pas pensé a tout les effets de bord que tu cité

@fufeck fufeck requested a review from MaGOs92 May 13, 2026 08:18

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

Tests and code LGTM (j'ai pas trouvé le bouton sur bal-admin) mais par API ça marche bien.

Petite suggestion pour bal-formatting : Est-ce que ça serait possible de garder le même ordre des colonnes que dans le fichier source? Là quand j'essaie de faire un diff entre le fichier source et le fichier formaté y'a tout qui est en diff parce que les colonnes sont pas dans le même ordre. ça pourrait être pratique de pouvoir checker ça facilement

Comment thread src/modules/revision/revision.controller.ts Outdated
Comment thread src/modules/revision/revision.controller.ts Outdated
@fufeck fufeck merged commit edb12c3 into master May 13, 2026
3 checks passed
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