Skip to content
This repository has been archived by the owner on Aug 9, 2018. It is now read-only.

Can't work in child route #36

Open
e8517027 opened this issue Jul 27, 2016 · 5 comments
Open

Can't work in child route #36

e8517027 opened this issue Jul 27, 2016 · 5 comments

Comments

@e8517027
Copy link

I use "node server.js",then i can see the default page,But I click on the link in the city without any reaction.Why?

@vostrik
Copy link

vostrik commented Sep 14, 2016

I fount the problem.

See to 143 line in common.js: weather url is

http://api.openweathermap.org/data/2.5/weather?q=

But openweathermap.org need auth.

The right url need be like this:

http://api.openweathermap.org/data/2.5/forecast/weather?APPID=YOUR_API_KEY&q=

@apparentlymart
Copy link
Collaborator

Hi @vostrik. Thanks for the pointer.

The weather demo was written a long time ago, and at that point this API did not require authentication. I guess we will need to do some work here to add authentication support.

@vostrik
Copy link

vostrik commented Sep 14, 2016

Ok. Can you give me advice?

I test perfomance and render 10k id in ng-repeat by node server, then by client. Client is faster.
Can we render heavy directives on backend?

@apparentlymart
Copy link
Collaborator

jsdom is noticeably slower than native DOM implementations in desktop browsers. In our application using angularjs-server we had to do a lot of profiling and performance improvements to get acceptable performance.

This library certainly isn't a "magic solution" to rendering AngularJS pages on the server. If you're planning to apply it to an already-existing application then you should expect that some adjustments will be required to get suitable behavior and performance.

I'm happy to answer further questions, but let's start a new issue so we don't spam the subscribers of this one...

@vostrik
Copy link

vostrik commented Sep 14, 2016

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants