-
Notifications
You must be signed in to change notification settings - Fork 24
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
make notification icon brighter when there's a notification #258
Conversation
✅ Deploy Preview for lodestone-dashboard ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for lodestone-storybook ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
src/data/EventStream.ts
Outdated
@@ -287,6 +294,7 @@ export const useEventStream = () => { | |||
title: `Macro ${macro_pid} errored on ${uuid}: ${error_msg}`, | |||
event, | |||
type: 'add', | |||
fresh: fresh, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can replace fresh: fresh
with just fresh
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me!
sorry for the delay, next time if I forgot about a pr please re-request review or just merge it
if i don't forget too lmao |
No description provided.