Skip to content
This repository was archived by the owner on Jul 26, 2022. It is now read-only.

Don't share data array between instances.#16

Open
mckoss wants to merge 1 commit into
googlesamples:masterfrom
mckoss:patch-3
Open

Don't share data array between instances.#16
mckoss wants to merge 1 commit into
googlesamples:masterfrom
mckoss:patch-3

Conversation

@mckoss
Copy link
Copy Markdown

@mckoss mckoss commented Jul 9, 2014

The previous code shared the data array among all the instances of the polymer element. Moot in the case of a singleton, but not the intended behavior.

@googlebot
Copy link
Copy Markdown

Thanks for the pull request.

It looks like this is your first contribution to a Google open source project, or you're using an email address we haven't seen before. Before proceeding, you'll need to sign the Contributor License Agreement (CLA) below:

https://developers.google.com/open-source/cla/individual

Once you've done that, please reply here to let us know.

@mckoss mckoss added cla: no and removed cla: yes labels Jul 9, 2014
@mckoss
Copy link
Copy Markdown
Author

mckoss commented Jul 9, 2014

CLA signed.

@PaulRashidi
Copy link
Copy Markdown
Member

@devnook Please let me know if you approve.

@googlebot
Copy link
Copy Markdown

FYI: CLA signature found. Thanks!

@mckoss mckoss added cla: yes and removed cla: no labels Jul 9, 2014
@inl1ne
Copy link
Copy Markdown

inl1ne commented Jul 17, 2014

Added internally as b/16375804

@devnook
Copy link
Copy Markdown
Contributor

devnook commented Jul 17, 2014

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants