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

the url in MetroDemo doesn't work #229

Closed
lhongzhong opened this issue Oct 12, 2012 · 4 comments
Closed

the url in MetroDemo doesn't work #229

lhongzhong opened this issue Oct 12, 2012 · 4 comments

Comments

@lhongzhong
Copy link

this link doesn't work: "http://ws.audioscrobbler.com/2.0/?method=chart.gethypedartists&api_key=b25b959554ed76058ac220b7b2e0a026&format=json"

I got this error: "The test API key has been deprecated. To use the Last.fm API you'll need to get an API account."

@ghost
Copy link

ghost commented Oct 13, 2012

LastFM have changed their API (they're turning off that APIKey for a few hours/day), as such yes, the demo app is broken from the data side.

When I've got some time this will be replaced with a local data source so it doesn't need to query the web.

edit: sidenote, MVCMusicStore contains some sample POCOs that I'll use.

@lhongzhong
Copy link
Author

I would like to help if I can - change it to load local data source. Need to know the guideline and the basic idea you have.

On Oct 13, 2012, at 14:08, VikingCode notifications@github.com wrote:

LastFM have changed their API (they're turning off that APIKey for a few hours/day), as such yes, the demo app is broken from the data side.

When I've got some time this will be replaced with a local data source so it doesn't need to query the web.


Reply to this email directly or view it on GitHub.

@thoemmi
Copy link
Collaborator

thoemmi commented Oct 13, 2012

It works with my API key, so it's not an API issue. It's just that your key seems to be deprecated.

@ghost
Copy link

ghost commented Oct 13, 2012

@thoemmi the key being used is the 'test api' key, or specifically "you don't need to sign up to use the API", they're deprecating that.
Given the demo doesn't want/need any sort of access, that was fine at the time. It was never meant to be a long term solution.

Moving to a static/local data source is better anyway, as it reduces loading times, makes it easier to test, etc.

@ghost ghost closed this as completed in 1a6eac8 Oct 18, 2012
ziykon pushed a commit to ziykon/MahApps.Metro that referenced this issue May 23, 2013
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants