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

Polymer bind method -- should take oneTime flag? #405

Closed
jmesserly opened this issue Jan 28, 2014 · 2 comments
Closed

Polymer bind method -- should take oneTime flag? #405

jmesserly opened this issue Jan 28, 2014 · 2 comments

Comments

@jmesserly
Copy link
Contributor

In the code here:
https://github.com/Polymer/polymer-dev/blob/b1a4e7df0d3f285f4b8116fa92f3d0f1f7528a1b/src/instance/mdv.js#L34

I don't see anything that handles the oneTime flag. It's normally handled in bind:
https://github.com/Polymer/NodeBind/blob/05dca5cdc4fdb9e0388132bbf456ec13c5aa67c5/src/NodeBind.js#L143

Maybe y'all are handling this at another place? (e.g. polymerexpressions) ... if so it would be good to mention in a comment.

@rafaelw as fyi

@jmesserly
Copy link
Contributor Author

(p.s. let me know if I should file new issues on polymer-dev instead ... I wasn't sure...)

@rafaelw
Copy link
Contributor

rafaelw commented Feb 3, 2014

Yup. probably should.

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

No branches or pull requests

2 participants