-
Notifications
You must be signed in to change notification settings - Fork 0
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
v1.4.1-HMR for review #5
Conversation
…eview_reload Live reloading LiveViews without a hard refresh
for {topic, patterns} <- notify_patterns do | ||
if matches_any_pattern?(path, patterns) do | ||
Phoenix.PubSub.broadcast( | ||
socket.pubsub_server, | ||
to_string(topic), | ||
{:phoenix_live_reload, topic, path} | ||
) | ||
end | ||
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.
基本的には:notifyオプションの追加でnotify_patterns
からPubSubで他のappにも通知できるようになったのが主な修正のようでした。
PR見た感じ:phoenix_live_view
で利用できるように拡張したらしい
phoenixframework#131
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.
:naruhodo:
* `:debounce` - an integer in milliseconds to wait before sending | ||
live reload events to the browser. Defaults to `0`. | ||
|
||
* `:iframe_attrs` - attrs to be given to the iframe injected by |
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.
間違って前回のbranchで戻してしまってたので再度削除
確認完了、こちらのPRはcloseしてsiiibo-appの依存先をv1.4.1-HMRに変更します |
Fork & patch of Phoenix.LiveReload v1.4.1 for internal use.
We won't merge this!! Just for review.
1.4.1対応してv1.4.0 をbaseとしたPRになります、手元環境ではこちらでHMR挙動が維持されていることを確認しました