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.

Sign up for GitHub

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

Jump to bottom

Permettre le nettoyage des propriétés inutilisées #803

Closed
thomaslhostis opened this issue Oct 30, 2019 · 3 comments · Fixed by #815
Closed

Permettre le nettoyage des propriétés inutilisées #803

thomaslhostis opened this issue Oct 30, 2019 · 3 comments · Fixed by #815
Labels
api-change user request

Comments

@thomaslhostis
Copy link
Collaborator

thomaslhostis commented Oct 30, 2019 •
edited
Loading

Cette fonctionnalité est présente dans le frontend. L'idée est de la rapatrier dans le backend et de proposer le nettoyage au niveau module mais aussi au niveau de la plateforme.

La rendre disponible dans l'API permettrait de l'inclure à la sharedLib => sncf-connect-tech/hesperides-jenkins-lib#36

Issue front pour le nettoyage au niveau de la plateforme : sncf-connect-tech/hesperides-gui#316
This was referenced Oct 30, 2019
Closed
Closed
@Lucas-C
Copy link
Member

Lucas-C commented Nov 7, 2019

LADR à prévoir pour définir la nouvelle ressource REST

thomaslhostis added a commit that referenced this issue Dec 3, 2019
@thomaslhostis
Définition de l'ADR pour le nettoyage des propriétés intulisées (#803)
594d712
thomaslhostis added a commit that referenced this issue Dec 3, 2019
@thomaslhostis
Définition de l'ADR pour le nettoyage des propriétés intulisées (#803)
7e9c15f
@thomaslhostis
Copy link
Collaborator Author

thomaslhostis commented Dec 3, 2019 •
edited
Loading

LADR : https://github.com/voyages-sncf-technologies/hesperides/blob/master/documentation/lightweight-architecture-decision-records/clean_unused_properties.md

baaroth pushed a commit to baaroth/hesperides that referenced this issue Dec 4, 2019
behavior tests for sncf-connect-tech#803
4580107
baaroth pushed a commit to baaroth/hesperides that referenced this issue Dec 4, 2019
fix sncf-connect-tech#803 via exclusion + (internal) save
4b40ebf
baaroth pushed a commit to baaroth/hesperides that referenced this issue Dec 5, 2019
fix sncf-connect-tech#803 via exclusion + (internal) save
2196e19 
merge the (now) 2 exclusion algorithms where applicable
@baaroth baaroth mentioned this issue Dec 5, 2019
Merged
@thomaslhostis
Copy link
Collaborator Author

thomaslhostis commented Dec 6, 2019 •
edited
Loading

Vérifier que les propriétés suivantes ne sont pas supprimées :

  • Propriétés présentes dans les templates des technos
  • Propriétés absentes du modèle mais référencées par d'autres propriétés (de module ou d'instance)

baaroth pushed a commit to baaroth/hesperides that referenced this issue Dec 6, 2019
improve fix sncf-connect-tech#803: indirect properties
023cd03 
some properties exists only on value-side (no model defines them) but other model-defined properties use them => they MUST NOT disappear!
@thomaslhostis thomaslhostis changed the title Permettre le nettoyage des propriétés supprimées Permettre le nettoyage des propriétés inutilisées Dec 9, 2019
thomaslhostis pushed a commit that referenced this issue Dec 9, 2019
@baaroth @thomaslhostis
Fix #803 : Permettre le nettoyage des propriétés inutilisées (#815)
3010a7f
@Lucas-C Lucas-C mentioned this issue Feb 12, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
api-change user request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue 803 : Permettre le nettoyage des propriétés inutilisées baaroth/hesperides
2 participants
@Lucas-C @thomaslhostis