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

app-network-status-behavior:refreshNetworkStatus trying to set readonly property: online. #58

Open
1 of 8 tasks
dmix007 opened this issue Jul 16, 2016 · 2 comments
Open
1 of 8 tasks

Comments

@dmix007
Copy link

dmix007 commented Jul 16, 2016

Description

app-storage is trying to the a read-only property. Results in a console error at load time using Polymer^1.6.

Console Error: Uncaught TypeError: this._setOnline is not a function
Source & Line #: app-network-status-behavior:68

More Detail:
Uncaught TypeError: this._setOnline is not a functionrefreshNetworkStatus @ app-network-status-behavior.html:68attached @ app-network-status-behavior.html:53_invokeBehavior @ polymer-micro.html:434_doBehavior @ polymer-micro.html:427(anonymous function) @ polymer-micro.html:211_makeReady @ polymer-micro.html:109(anonymous function) @ polymer-micro.html:115

Expected outcome

No console errors or warnings.

Actual outcome

Console warning about there not being a setter for a read-only variable (online).

Live Demo

N/A

Steps to reproduce

Browsers Affected

  • Chrome
  • Firefox
  • Safari 9
  • Safari 8
  • Safari 7
  • Edge
  • IE 11
  • IE 10
@fairsayan
Copy link

fairsayan commented Jan 3, 2017

Temporary solution at https://github.com/fairsayan/app-storage waiting for the fix.
I just removed the readOnly attribute from online property and set directly.
Install: bower install https://github.com/fairsayan/app-storage.git#03c3b72131fddacc00dc2cf9c4fc32e401370e5d --save

PS: to me wasn't just a warn, it was an error that broke the js flow.

@arkihillel
Copy link

Still broken

Thanks for the link to the working version fairsayan, works as expected

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

3 participants