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

Corrected: Viewing the recording information of a dataset or a snapshot requires being logged in #226

Closed
wants to merge 22 commits into from

Conversation

nupurbaghel
Copy link
Contributor

This pull request is to fix the ticket "Viewing the recording information of a dataset or a snapshot requires being logged in"
https://tickets.metabrainz.org/browse/AB-290

gentlecat and others added 22 commits July 27, 2016 03:47
Pylint updates + pycodestyle
Also test cases when the job doesn't exist
In tests that don't use the test client we don't have to deal with
the flask context of the client, and therefore can just use the
regular flask_login.login() method
Rename the method and endpoint to refer to low-level, like the
existing GET methods
Catch <0 offsets
If the same request is duplicated in the query, only perform the
request once
also fix embarrassing spelling mistake. valilidate?
Make navbar more compact by moving some items into dropdowns
There are no public servers which use the old format, and this
script will remain in git history, so can be recovered if needed
This file was created during the json->jsonb schema change
but was never used. Remove to to reduce confusion
Remove unused migration scripts and data files
Essentia examples used to include essentia statically, but now link to a .so, so we need to install it
Used in oauth login with musicbrainz, broken since
newhost setup. Also reported in AB-130 but not correctly
fixed the first time around
Use an https client which checks SSL certificates
Add crossdomain header for /api/v1/low-level endpoint (AB-295)
@nupurbaghel nupurbaghel closed this Mar 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants