-
Notifications
You must be signed in to change notification settings - Fork 423
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
Show information when a new Livebook version is available #1121
Conversation
Co-authored-by: José Valim <jose.valim@dashbit.co>
022fe34
to
fbf79a1
Compare
lib/livebook_web/live/home_live.ex
Outdated
</span> | ||
</span> | ||
<div class="px-2 py-2 bg-red-200 text-gray-900 text-sm text-center"> | ||
Less than 30 MB of memory left, consider adding more resources to |
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.
You don't want to keep the icon? :D It looked cool. Your call!
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.
The new styles look great! |
Co-authored-by: José Valim <jose.valim@dashbit.co>
Co-authored-by: José Valim <jose.valim@dashbit.co>
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.
Just one tiny comment about also handling DOWN messages and ship it!
Closes #1109.
notify_updates.mp4