-
-
Notifications
You must be signed in to change notification settings - Fork 11
Content gets not rendered #86
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
Comments
Same issue here, looks like the matcher returns |
Hmm I think the recently-released @phloe can you please check and let us know, I can also confirm I'm getting |
So this is where we try to replace the Line 82 in c608fbf
Given that this code above it works to match the tag: Line 71 in c608fbf
... I wonder if that missing attribute is what's causing it. |
Locally, I've installed |
Yeah, that works as a workaround 👍 We'll need to get this fixed though as it's currently breaking for everyone and they might have no clue where it's coming from since there are no errors or anything... |
The error is in posthtml-match-helper? Yikes. The 1.0.2 bump was only supposed to have added TypeScript types... |
Yeah, can't understand what's causing it but |
I hope it's fixed now with posthtml-match-helper@1.0.3 New bug was match('content') // -> { tag: 'content', attrs: {} } used to be match('content') // -> { tag: 'content' } failing to match tags with no attributes - but matching if they had any. Also - empty attributes never worked (not a new bug - but fixed in 1.0.3): match('content[]') |
Thanks Rasmus, appreciate the fast response! Bumped it in #87 and tests are passing again 👍 Would you maybe consider adding some tests to |
YES! 😅 |
@cossssmin Works for me on 1.0.3 ✅ |
Sorry for the delayed response. Works as expected now! Thx a lot. |
Hi seems like that the content gets not rendered correctly? Maybe i did something wrong?
This is my module
I use my module like this:
The rendered result will be
The text was updated successfully, but these errors were encountered: