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

Valeurs TO.BE.REMOVED apparaissent quand on vide un champ #488

Closed
LGAGELIN opened this issue Nov 8, 2017 · 1 comment
Closed

Valeurs TO.BE.REMOVED apparaissent quand on vide un champ #488

LGAGELIN opened this issue Nov 8, 2017 · 1 comment
Labels
bug
Milestone
Editlog

Comments

@LGAGELIN
Copy link
Collaborator

LGAGELIN commented Nov 8, 2017 •
edited
Loading

Observé sur la release du 6 nov en PPRD :

Quand on vide un champ dans une fiche, la valeur TO.BE.REMOVED apparaît parfois. Ce n'est pas systématique. Voir l'historique de la fiche de​ Yi-Chun CHEN (rattaché à l'OSC).

Étapes pour reproduire :

  • Remplir le champ Bureau occupé à Sciences Po, sauvegarder => la création apparaît bien dans l'historique
  • Puis vider le champ, sauvegarder => TO.BE.REMOVED apparaît dans le champ et dans la "valeur après" dans l'historique

image
image
@LGAGELIN LGAGELIN added the bug label Nov 8, 2017
@LGAGELIN LGAGELIN added this to the Editlog milestone Nov 8, 2017
@LGAGELIN LGAGELIN changed the title Valeurs TO.BE.REMOVED apparaissent quand on vide un champ Valeurs TO.BE.REMOVED apparaissent dans l'historique quand on vide un champ Nov 8, 2017
@LGAGELIN LGAGELIN changed the title Valeurs TO.BE.REMOVED apparaissent dans l'historique quand on vide un champ Valeurs TO.BE.REMOVED apparaissent quand on vide un champ Nov 8, 2017
@paulgirard
Copy link
Collaborator

Pour reproduire.
Ajouter une valeur à bureau dans contact.
Sauvegarder
Supprimer cette valeur.
Sauvegarder.
Regarder l'historique

@paulgirard paulgirard mentioned this issue Nov 21, 2017
Closed
paulgirard added a commit that referenced this issue Nov 21, 2017
@paulgirard
removing TO.BE.REMOVED in deep objects in rest-utils.js fixed #488 #487
ac4774a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
Editlog
Development

No branches or pull requests
2 participants
@paulgirard @LGAGELIN