-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Add phx.new --live #3680
Add phx.new --live #3680
Conversation
… `phoenix_live_view`
…ngs LiveView into new phoenix app
longpoll: false | ||
longpoll: false<%= if live do %> | ||
|
||
socket "/live", Phoenix.LiveView.Socket, websocket: [connect_info: [session: @session_options]]<% end %> |
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.
socket "/live", Phoenix.LiveView.Socket, websocket: [connect_info: [session: @session_options]]<% end %> | |
socket "/live", Phoenix.LiveView.Socket, | |
websocket: [connect_info: [session: @session_options]]<% end %> |
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.
This is what I had at first, but it failed --check-formatted
and that pesky formatter wants it on ea single line :)
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.
Bump the Elixir version. Since V1.7 or V1.8 we respect the user choice in here.
This is based on the work @aforward started in #3376 (❤️) . All the setup and tests are in place so it's now a matter of determining what we want to go with naming wise and home page feature wise. I included a Modal component with the app, which is something our live CRUD generators could check for and add, but it would be nice to include it here if we can make it worthwhile. Currently I use the modal to show
hex.outdated
results, but I think we can do better so consider this a starting point to explore from live home page wise.