search

veliovgroup / jazeee-meteor-spiderable

Fork of Meteor Spiderable with longer timeout, caching, better server handling
https://atmospherejs.com/jazeee/spiderable-longer-timeout
33 stars 9 forks source link

Crashing on angular-meteor #47

Closed alkaj closed 7 years ago

alkaj commented 7 years ago

I found this package very interesting as i can decide when to release the content for spiders, but it crashes right after the installation ends with an "unexpected token (1, 81)", could someone help? i did not found any reference to this error anywhere so i conclude that it is specific to my installation; i am using Meteor 1.4.2.3 along with Angular. thanks in advance

dr-dimitru commented 7 years ago

Hello @alkaj ,

Could you post more info about error? Full error stack, is it happens on Client or Server? How do you request "rendered" version?

alkaj commented 7 years ago

Hello @dr-dimitru, the error is thrown immediately after the installation ends as the server is not able to process startup anymore, here is the complete error stack:

=> Errors prevented startup:

   While building for web.browser:
   lib/spiderable.js:1:81: Unexpected token, expected , (1:81)

   While computing assigned variables:
   /packages/jazeee_spiderable-longer-timeout/lib/spiderable.js: Unexpected
   token, expected , (1:81)

   While building for os.linux.x86_64:
   lib/spiderable.js:1:81: Unexpected token, expected , (1:81)

   While computing assigned variables:
   /packages/jazeee_spiderable-longer-timeout/lib/spiderable.js: Unexpected
   token, expected , (1:81)

=> Your application has errors. Waiting for file change.

thanks...

dr-dimitru commented 7 years ago

Is ecmascript package installed to your app?

alkaj commented 7 years ago

No it is not, it got removed when installing the angular package, it can not be added because of conflict with the pbastowski:angular-babel package on *.js files.

dr-dimitru commented 7 years ago

@alkaj why not switch to NPM babel-runtime?

dr-dimitru commented 7 years ago

From pbastowski:angular-babel docs

The discussion here is for Meteor 1.2 only.

So, quick guess - you don't need it on Meteor@1.4.2.3

alkaj commented 7 years ago

seems like the problem discussed persist anyhow...

SimonatoLucaManuele commented 7 years ago

I've the same problem.. i'm not using angular.

SOLVED -> my ecmascript module wasn't updated at his last version

dr-dimitru commented 7 years ago

@SimonatoLucaManuele thank you for report

@alkaj is it solved on your end?

alkaj commented 7 years ago

@dr-dimitru I finally opt-out for the greenydragonny:spiderable-ui-router that actually works well with the angular ui-router.

dr-dimitru commented 7 years ago

@alkaj I assume we can close this one

Feel free to reopen it in case if the issue still persists on your end.