-
Notifications
You must be signed in to change notification settings - Fork 3
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
Collision add-on unaware of get/set methods? #79
Comments
Hm. Could you please provide a (stripped?) code to reproduce it locally?
That would significantly help.
…On Tue., 7 Apr. 2020, 06:43 Duane Johnson, ***@***.***> wrote:
I have a stamp with a method that uses get style function:
methods: {
/**
* @returns {string}
*/
get label() {
return this.state.label.target
},
...
I also have a few methods that I am trying to set up for defer style
collision so that they are all called (instead of the last one).
It appears that when I use these two features together, I get an error
that this.state is undefined. Using console.trace I believe that this
line of code is "calling" my getter:
https://github.com/stampit-org/stamp/blob/master/packages/collision/index.js#L146
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#79>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAMMEL43MATKDY7Y33HAQJLRLI5IJANCNFSM4MCSWOAA>
.
|
Oh! I see now. It calls the getter. Shite! 😀 |
Ha! Yes, that's the one. Thanks for taking a look. |
Is this something that you have time to fix? I just ran into this issue again. |
Sorry. I hadn't had a chance. It's a difficult one. Also, I have a baby
now. Taking my whole time atm.
…On Sun., 14 Jun. 2020, 04:09 Duane Johnson, ***@***.***> wrote:
Is this something that you have time to fix? I just ran into this issue
again.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#79 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAMMEL6NG5VKD3I7NKMKJWLRWO6G7ANCNFSM4MCSWOAA>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have a stamp with a method that uses
get
style function:I also have a few methods that I am trying to set up for
defer
style collision so that they are all called (instead of the last one).It appears that when I use these two features together, I get an error that
this.state
is undefined. Using console.trace I am lead believe that this line of code is "calling" my getter:https://github.com/stampit-org/stamp/blob/master/packages/collision/index.js#L146
The text was updated successfully, but these errors were encountered: