-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Bug using hash and array helpers together #18693
Comments
Hmm, thank you for reporting! Definitely seems like a regression to me. |
Has there been any update on this? I see that we've cut a new beta but it doesn't seem to have changed anything 🤔 |
Tracked down the root cause and have opened up a PR to fix upstream in the VM: glimmerjs/glimmer-vm#1022 |
thanks a million for fixing this @pzuraq 🎉 I'm assuming this will be backported into the 3.17 release branch then once it's fixed so that beta will start working again? |
It looks like this has started working now 🎉 I'm assuming the fix was included in v3.17.0-beta.5 then? Thanks everyone for working on this |
@mansona I don't think so, the only change in beta 5 is docs |
…47.9 [BUGFIX beta] Add test for #18693
(cherry picked from commit 0933925)
There seems to be a bug that was introduced in [email protected] 🤔but interestingly it doesn't seem to have been happening on ember-canary before the beta was released
Essentially it looks like using the
{{array}}
and{{hash}}
helpers together doesn't work with the error:I have produced a replication and run tests with ember try here: mansona/bug-hash-array-reproduction#1 and the build is here https://travis-ci.com/mansona/bug-hash-array-reproduction/builds/145761580
You will see that it fails on ember-beta and ember-canary.
Here is the example of the template that fails:
The table is not required, I just put it in there to make it clear what an example app might be 👍
The text was updated successfully, but these errors were encountered: