-
Notifications
You must be signed in to change notification settings - Fork 14
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
Updates #152
Conversation
Run `mix format`
- Add tailwind using the Elixir tailwind dependency - Use esbuild to manage tailwind css classes see https://tailwindcss.com/docs/guides/phoenix ref: #151
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #152 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 5 5
Lines 51 51
=========================================
Hits 51 51 ☔ View full report in Codecov by Sentry. |
Because we have already a login button on the header we can make the footer a bit lighter by removing the duplicate login button
Use :text instead of :string to allow long text as message see: #149
Updates looking good so far. 👌 |
Getting the following warning when running this branch on
|
Branch auto-deployed to: https://liveview-chat-example.fly.dev/ Still working fine including auth: Don't think there's anything |
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.
Updates look great. Thanks again @SimonLab 👌
see #151 and #150