Bind this to custom attribute converter methods#3120
Conversation
🦋 Changeset detectedLatest commit: b3ab89f The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
b5febc4 to
b015dcf
Compare
…onverter-methods-binding
There was a problem hiding this comment.
Thanks for the fix @nebarf !
I made some comments around creating fewer short-lived function objects due to Function.bind() when setting properties/attributes.
A bigger change that we should consider outside of this fix (I think) would be to do the toAttribute/fromAttribute determination in createProperty() so it's done once per class, not on every property set. cc @sorvell
… reflected property with a custom converter is set
There was a problem hiding this comment.
Awesome, thank you! I just went ahead and combined the changeset files together which is our preferred way.
Fixes #3108