Skip to content
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

Failing test for {{yield}} inside a component that uses the element helper #1

Closed
wants to merge 1 commit into from

Conversation

cibernox
Copy link
Collaborator

I finally got time to check if this helper fixed my needs and I found one case where it doesn't.
Seems that there is a bug when used inside a component and yielding inside the generated element.

@cibernox
Copy link
Collaborator Author

After taking a second look, I found that this only fails if the yielded component receives splattributes, which is even weirder TBH.

@chancancode
Copy link
Member

Thanks for the report! This is probably a glimmer bug 🤔 I’ll investigate

@cibernox
Copy link
Collaborator Author

@chancancode any update on this?

@chancancode
Copy link
Member

Sorry! I'll check this weekend!

@chancancode
Copy link
Member

See glimmerjs/glimmer-vm#892

@cibernox
Copy link
Collaborator Author

cibernox commented Jan 18, 2019

@chancancode shouldn't we wait to close this until new versions of the glimmer-vm are in use by ember? Until then the issue persists.

@chancancode chancancode reopened this Jan 18, 2019
@chancancode
Copy link
Member

It just didn't turn out to be related to this add-on at all, and it's an upstream bug, but sure, I can leave this open for now.

@cibernox
Copy link
Collaborator Author

@chancancode This can be closed now, Ember 3.4 and 3.7 have been updated.

@wagenet wagenet closed this Jan 23, 2019
simonihmig pushed a commit that referenced this pull request Feb 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants