-
Notifications
You must be signed in to change notification settings - Fork 126
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
Fragments support #167
Fragments support #167
Conversation
Really looking forward to this. Wish I had the time to help you out with more tests. Maybe this weekend. |
I think we're close to fully support them. Need more manual/automated testing I think. Especially the case with several top fragments level. Maybe others. |
What's the status for this? |
I personally haven't had time to finish the PR. I'm sure it's close though,
if you want to take a look.
…On Sat, Mar 23, 2019, 11:55 AM Peter Strömberg ***@***.***> wrote:
What's the status for this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#167 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAEt3q7MKvmfblXBoS2NdQ63xKZMhpZxks5vZgg8gaJpZM4UW65R>
.
|
Hello 👋 New maintainer here. We'll most likely pull in Sablono into the repo some time in the future (see #198), so this will be addressed later for both JS and JVM targets. The plan is to have |
|
This time for real. My simple implementation seems to work well (at least it passes the test).
This needs more tests maybe, examples and Readme update at least.