perma icon indicating copy to clipboard operation
perma copied to clipboard

History logging does not include a link's folder membership

Open phillipsm opened this issue 10 years ago • 7 comments

If a record is in an org folder and a person is a member of that org, they can move that record into any folder (including their own).

phillipsm avatar Sep 29 '15 14:09 phillipsm

Update: we want to allow anyone to move links to any of their folders, because usually that's a good thing, but we want to make sure moves are logged so we can deal with any customer service questions.

jcushman avatar Dec 05 '16 17:12 jcushman

Hmm. It's not.....

rebeccacremona avatar Jun 19 '18 20:06 rebeccacremona

When you move links around, django simple history DOES make an entry in the "change history" list, but the change in folder is not actually reflected, and reverting has no effect.

rebeccacremona avatar Jun 20 '18 12:06 rebeccacremona

Changing the title works as expected: it adds an entry to "change history", and each entry lists the correct title, and you can revert as expected.

rebeccacremona avatar Jun 20 '18 12:06 rebeccacremona

Changing a one-to-many relationship (created_by) also works normally.

rebeccacremona avatar Jun 20 '18 13:06 rebeccacremona

see https://github.com/treyhunner/django-simple-history/issues/16

rebeccacremona avatar Jun 20 '18 13:06 rebeccacremona

This is more complex than I expected; tabling for now, pending discussion.

rebeccacremona avatar Jun 20 '18 13:06 rebeccacremona