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.

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

Don't show audit snackbar if all entries are all 'ok' state #9229

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

caolanm
Copy link
Contributor

@caolanm caolanm commented Jun 6, 2024

I see this on a local nextcloud where the audit dialog only has one entry which is "ok".

Change-Id: I55f12125cbc0ca4c88c9220ed114d455e84639e0

  • Resolves: #
  • Target version: master

Summary

TODO

  • ...

Checklist

  • Code is properly formatted
  • All commits have Change-Id
  • I have run tests with make check
  • I have issued make run and manually verified that everything looks okay
  • Documentation (manuals or wiki) has been updated or is not required

@caolanm caolanm requested a review from eszkadev June 6, 2024 14:57
@caolanm caolanm force-pushed the private/caolan/audit_snackbar_only_if_notok branch 3 times, most recently from 7a7535e to d583339 Compare June 7, 2024 09:27
I see this on a local nextcloud where the audit dialog only has one entry
which is "ok".

Signed-off-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Change-Id: I55f12125cbc0ca4c88c9220ed114d455e84639e0
@caolanm caolanm force-pushed the private/caolan/audit_snackbar_only_if_notok branch from d583339 to 3cc24ff Compare June 7, 2024 10:22
@caolanm caolanm requested a review from mmeeks June 7, 2024 15:25
Copy link
Contributor

@mmeeks mmeeks left a comment

Choose a reason for hiding this comment

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

Looks great to me =)

@mmeeks mmeeks merged commit 136e4c5 into master Jun 7, 2024
14 checks passed
@mmeeks mmeeks deleted the private/caolan/audit_snackbar_only_if_notok branch June 7, 2024 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants