-
Notifications
You must be signed in to change notification settings - Fork 31
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
Support headTags Promises #28
Comments
Hey @ianpetzer, Good question. I like the idea of understanding a promise in headTags. It gets a little complicated in the implementation, but I'll be thinking about how it might be done. As always pull requests are welcome if you want to take a crack at it. I think there are a couple of approaches you could use to get this working with the addon in it's current state:
Let me know if either of those work for you. I'm happy to elaborate more if you'd like. -Ron |
Ron, Thanks so much for getting back to me. I actually implemented the first I think the second option is a great idea, so I don't need to block the Thanks for the great ideas. On Sat, Nov 21, 2015 at 5:17 PM, ronco [email protected] wrote:
|
Great @ianpetzer! I'm glad that helped out. I'm going to keep this issue open as a possible enhancement. I like the idea of supporting promises out of the box (although the complexities might keep me from building a solution anytime soon). |
This would be incredibly helpful in fastboot |
Is there a way to use async relationships in the headTags function?
Currently if I refer to an async relationship, then i get undefined show up in the meta tag, and I can't return a promise from headTags function.
The text was updated successfully, but these errors were encountered: