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

todo_ddc doesnt run. #3

Closed
andrey-zakharov opened this issue Apr 19, 2017 · 6 comments
Closed

todo_ddc doesnt run. #3

andrey-zakharov opened this issue Apr 19, 2017 · 6 comments

Comments

@andrey-zakharov
Copy link
Contributor

andrey-zakharov commented Apr 19, 2017

Hi, Vittorio!

I have updated polimerize_elements in a hope of fixing some errors in console while running todo_ddc,
but with latest generated files (
I've still got:

Uncaught TypeError: Cannot read property 'addEventListener' of null
    at HTMLElement.attached (paper-input-container.html:509)
    at HTMLElement.attached (class.html:210)
    at HTMLElement.connectedCallback (legacy-element-mixin.html:72)
    at HTMLElement._attachDom (element-mixin.html:653)
    at HTMLElement._readyClients (element-mixin.html:630)
    at HTMLElement._flushClients (property-effects.html:1576)
    at HTMLElement._propertiesChanged (property-effects.html:1731)
    at HTMLElement._flushProperties (property-accessors.html:443)
    at HTMLElement._flushProperties (property-effects.html:1629)
    at HTMLElement.ready (property-effects.html:1663)`
attached @ paper-input-container.html:509
attached @ class.html:210
connectedCallback @ legacy-element-mixin.html:72
_attachDom @ element-mixin.html:653
_readyClients @ element-mixin.html:630
_flushClients @ property-effects.html:1576
_propertiesChanged @ property-effects.html:1731
_flushProperties @ property-accessors.html:443
_flushProperties @ property-effects.html:1629
ready @ property-effects.html:1663
ready @ legacy-element-mixin.html:135
ready @ class.html:201
ready @ class.html:201
ready @ class.html:201
ready @ class.html:201
ready @ class.html:201
_flushProperties @ property-effects.html:1627
connectedCallback @ element-mixin.html:607
connectedCallback @ legacy-element-mixin.html:70
_attachDom @ element-mixin.html:653
_readyClients @ element-mixin.html:630
_flushClients @ property-effects.html:1576
_propertiesChanged @ property-effects.html:1731
_flushProperties @ property-accessors.html:443
_flushProperties @ property-effects.html:1629
ready @ property-effects.html:1663
ready @ legacy-element-mixin.html:135
ready @ class.html:201
_flushProperties @ property-effects.html:1627
connectedCallback @ element-mixin.html:607
connectedCallback @ legacy-element-mixin.html:70
connectedCallback @ polymer-redux.js:121
polymerize @ polymerize.js:130
(anonymous) @ todo_main.mod.html:43
scope.require @ imd.html:6
(anonymous) @ todo_main.mod.html:42

looks like input missing in paper decorator or something else.

I've took sources from this but hack it to use
https://github.com/andrey-zakharov/polymerize_elements

which contains latest generated files for polyner 2.0.0.rc6.

Could you please, point me to direction where to dig in.
Thanx.

@dam0vm3nt
Copy link
Collaborator

I've tagged your PR and updated todo_ddc, updated bower_resolutions.yaml to rc6 and now it works for me.

Can you check if this works for you too ?

@andrey-zakharov
Copy link
Contributor Author

Thats my commands session output:

$ git pull -ff
Обновление 62c10e8..abdc4b5
Fast-forward
 .gitignore                   | 4 +++-
 bower_resolutions.yaml       | 4 ++--
 pubspec.lock                 | 6 +++---
 pubspec.yaml                 | 2 +-
 todo_main/lib/todo_main.dart | 2 ++
 todo_main/lib/todo_main.html | 2 ++
 todo_main/pubspec.lock       | 4 ++--
 todo_renderer/pubspec.lock   | 4 ++--
 web/index.html               | 2 +-
 9 files changed, 18 insertions(+), 12 deletions(-)

$ pub get && polymerize init && bazel build default
Resolving dependencies... (1.8s)
Warning: You are using these overridden dependencies:
! polymer_elements 2.0.0-preview.1+1 from git https://github.com/polymer-dart/polymerize_elements.git at 3fa9ca
Got dependencies!
..............
INFO: Found 1 target...
Target //:default up-to-date:
  bazel-bin/bower.json
  bazel-bin/todo_ddc.js
  bazel-bin/todo_ddc.mod.html
  bazel-bin/todo_ddc.bower
INFO: Elapsed time: 12.556s, Critical Path: 3.93s

I've open localhost:8000 and refresh it by shift+f5, and ctrl+f5.
Thats what I've got in chrome's console

GET http://localhost:8000/bower_components/web-animations-js/web-animations-next-lite.min.js 
polymerize.js:42 Override JS : myTodo.status
polymerize.js:47 undefined
polymerize.js:42 Override JS : editing
polymerize.js:47 undefined
polymerize.js:42 Override JS : myTodo.text
polymerize.js:47 undefined
polymerize.js:42 Override JS : askConfirm
polymerize.js:47 undefined
polymerize.js:42 Override JS : newText
polymerize.js:47 undefined
polymerize.js:42 Override JS : canAdd
polymerize.js:47 undefined
polymerize.js:42 Override JS : todos
polymerize.js:47 undefined
polymerize.js:42 Override JS : todo
polymerize.js:47 undefined
paper-input-container.html:509 Uncaught TypeError: Cannot read property 'addEventListener' of null
    at HTMLElement.attached (paper-input-container.html:509)
    at HTMLElement.attached (class.html:210)
    at HTMLElement.connectedCallback (legacy-element-mixin.html:72)
    at HTMLElement._attachDom (element-mixin.html:653)
    at HTMLElement._readyClients (element-mixin.html:630)
    at HTMLElement._flushClients (property-effects.html:1576)
    at HTMLElement._propertiesChanged (property-effects.html:1731)
    at HTMLElement._flushProperties (property-accessors.html:443)
    at HTMLElement._flushProperties (property-effects.html:1629)
    at HTMLElement.ready (property-effects.html:1663)

Also, here is a http server log:
https://pastebin.com/FZCRg1Ei

@andrey-zakharov
Copy link
Contributor Author

pub upgrade (polymer_element 0.7.0+1 (was 0.7.0)) and rebuild of everything does not fix issue.

@dam0vm3nt
Copy link
Collaborator

dam0vm3nt commented Apr 20, 2017

I've made another test and had to fix resolutions again but it is working, this is what I did :

  1. bazel clean --expunge
  2. pub get && polymerize init && bazel build default
  3. cleared cache of chrome
  4. http://localhost:8000

and it worked.

My test conf is :

  • dart 1.22.1
  • polymerize 0.7.1
  • chrome 57.0.2987.133

@andrey-zakharov
Copy link
Contributor Author

Thanks! Finally it works after all purge commands and rebuild and cache purge.
Issue could be closed, app is working well (add, remove todos), but still got
404 for
GET http://localhost:8000/bower_components/web-animations-js/web-animations-next-lite.min.js

what it could means?

@dam0vm3nt
Copy link
Collaborator

cool.

As for the missing web-animations-next-line.min.js I've noticed it too but I've always thought it was a polymer (js) / polymer_elements (js) issue due to their rc status. As they are near to the stable release and it is still happening I think it's time to inveastigate it deeply.

I'll open a new issue on this just to remember.

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