-
Notifications
You must be signed in to change notification settings - Fork 70
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
Fix refresh of dynamic templates #1786
Conversation
Ahhhh I see how it works now! Doesn't this mean that changes to the non |
Good catch. Probably. I had this at compile-time if you recall. It may be possible to duplicate the module attributes and get both... |
I am having a hard time reproducing that it would not refresh :D |
I've give this a whirl tomorrow |
I tried adding both and that seemed to work. But then I couldn't get it to not refresh when I removed them. May need a |
I found https://hexdocs.pm/mix/1.14/Mix.Tasks.Compile.Elixir.html#module-__mix_recompile__-0 (from https://elixirforum.com/t/external-resource-for-folder-how-does-embed-templates-do-it/55823), which shows we can define I tested it locally and it fixes it for me! |
Not to say your fix didn't work, although granted I haven't checked it yet. |
Double it! |
No description provided.