polymer-dart / todo_ddc

15 stars 3 forks source link

todo_ddc doesnt run. #3

Closed andrey-zakharov closed 7 years ago

andrey-zakharov commented 7 years ago

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 commented 7 years ago

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 commented 7 years ago

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 commented 7 years ago

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

dam0vm3nt commented 7 years ago

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 :

andrey-zakharov commented 7 years ago

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 commented 7 years ago

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.