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

Liu 277 - Changing colour of expired and deleted drops #188

Merged
merged 2 commits into from
Aug 15, 2022
Merged

Conversation

pritchardn
Copy link
Collaborator

This MR changes the colours of expired and deleted drops to a dark grey and black with white font respectively.
Additionally, this PR reduces the default garbage collection period to 30 seconds (down from 100).

Attached is an imaging showing deleted, expired and cancelled drops for reference.

DeletedExpiredCancelled

Text of deleted drops is now light, so they can be read.
Colour of expired drops are now a dark grey.
@pritchardn pritchardn requested a review from awicenec July 8, 2022 03:29
Copy link
Contributor

@awicenec awicenec left a comment

Choose a reason for hiding this comment

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

All good. We could have changed the default cleanup to 10s even, but eventually we will probably implement a fully dynamic cleanup as soon as a drop has been fully consumed.

@awicenec awicenec merged commit c8b2df5 into master Aug 15, 2022
@awicenec awicenec deleted the liu-277 branch August 15, 2022 16:27
awicenec added a commit that referenced this pull request Oct 10, 2024
Liu 277 - Changing colour of expired and deleted drops
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.

2 participants