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

Switch from deprecated API; alter album schemas #1

Merged
merged 1 commit into from Feb 24, 2013
Merged

Switch from deprecated API; alter album schemas #1

merged 1 commit into from Feb 24, 2013

Conversation

TJC
Copy link
Contributor

@TJC TJC commented Feb 21, 2013

This switches from the deprecated Freebase API to the Google API
for Freebase, version 1.
It also adjusts the MQL queries made, to cope with the way that
albums and tracks were refactored in Freebase to have tracks attached to
releases which are attached to albums.

Note that in the new API system, you are supposed to request an API key from Google, and then append a "key" parameter to every query. You should do this for the live LPD site!
It will work for a short while without the key, but as soon as you have a moderate number of hits it'll die off.

This switches from the deprecated Freebase API to the Google API
for Freebase, version 1.
It also adjusts the MQL queries made, to cope with the way that
albums and tracks were refactored in Freebase to have tracks attached to
releases which are attached to albums.
hostilefork added a commit that referenced this pull request Feb 24, 2013
Switch from deprecated API; alter album schemas
@hostilefork hostilefork merged commit 334f9f4 into hostilefork:master Feb 24, 2013
@hostilefork
Copy link
Owner

Hello TJC, great news! I had read about the coming change...but as I'm not really involved with the site any longer I'd hoped that some JavaScripting fan would fix it so I didn't have to. :-)

I will notify the site admins and tell them what to update. Thank you very much!

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

Successfully merging this pull request may close these issues.

None yet

2 participants