You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
At the moment, when an import is done, the history table of an imported Page is not updated.
It would be useful to at least show that a Page (or similar) was last updated via import.
While we can't guarantee that a User corresponding to the original last editor will exist in the destination site, we can still add useful info without needing to get into explicit attribution.
Simply, PageLogEntry can handle a null user for system actions, so we could create a non-user-linked PageLogEntry post-import for each imported Page with a message saying that the page was last updated on this server via Wagtail-Transfer, and potentially with the verbose name and ID of the admin who triggered the import, which we will have available.
This would also be handy in creating an audit trail for showing which admins had triggered an import, and when.
(Am happy to write this up into a more coherent spec/structure, btw.)
The text was updated successfully, but these errors were encountered:
This sounds like a good plan IMO - now audit logging is in, we should definitely be hooking into it. We could potentially use the uuid of the user to try to fetch the user on the destination site if they exist, and fall back to null user if not.
This has been confusing, would be good to at least document the expected behavior with page changes via import. Sometimes I see an entry in the editor history, sometimes I don't.
At the moment, when an import is done, the history table of an imported Page is not updated.
It would be useful to at least show that a Page (or similar) was last updated via import.
While we can't guarantee that a User corresponding to the original last editor will exist in the destination site, we can still add useful info without needing to get into explicit attribution.
Simply,
PageLogEntry
can handle a nulluser
for system actions, so we could create a non-user-linkedPageLogEntry
post-import for each imported Page with a message saying that the page was last updated on this server via Wagtail-Transfer, and potentially with the verbose name and ID of the admin who triggered the import, which we will have available.This would also be handy in creating an audit trail for showing which admins had triggered an import, and when.
(Am happy to write this up into a more coherent spec/structure, btw.)
The text was updated successfully, but these errors were encountered: