Skip to content

BITMAG-980: Audit trail preservation info for web-GUI#34

Merged
Bohlski merged 7 commits intomasterfrom
BITMAG-980-audittrail-GUI-preservation-info
Aug 15, 2022
Merged

BITMAG-980: Audit trail preservation info for web-GUI#34
Bohlski merged 7 commits intomasterfrom
BITMAG-980-audittrail-GUI-preservation-info

Conversation

@Bohlski
Copy link
Copy Markdown
Contributor

@Bohlski Bohlski commented Aug 2, 2022

This adds a simple preservation info to the 'Audit trail' page in the web-GUI if preservation is enabled.

The preservation info is pretty much equivalent to what is shown about audit trail collection.

@Bohlski Bohlski requested review from m-atlantis and ole-v-v August 2, 2022 11:04
@@ -20,319 +20,384 @@
#L%
-->
<!DOCTYPE html>
Copy link
Copy Markdown
Contributor Author

@Bohlski Bohlski Aug 2, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Of course changes in this file ended up being a mess because of weird indentation... If you CTRL+F for 'preservation' I think the actual changes should be clearer.

On another note: ideally I would want the JavaScript moved to a separate file to not clutter the html, but as this issue is apparent in all of the html files I felt the changes might as well be done in bulk. As this would only serve to clutter the PR further I decided against doing it at this time.

Copy link
Copy Markdown
Member

@m-atlantis m-atlantis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good !

@Bohlski Bohlski merged commit 0cf17d2 into master Aug 15, 2022
@Bohlski Bohlski deleted the BITMAG-980-audittrail-GUI-preservation-info branch August 15, 2022 13:24
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.

3 participants