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

Fixes #669, #670 and #671 #680

Merged
merged 1 commit into from
Sep 13, 2016
Merged

Fixes #669, #670 and #671 #680

merged 1 commit into from
Sep 13, 2016

Conversation

bjohare
Copy link
Contributor

@bjohare bjohare commented Sep 13, 2016

Proposed changes in this pull request

Fixes: #669, #670, #671

  • Add audio/mp3 and audio/mpeg to MIME_LOOKUPS in default settings
  • Tighten up mime-type checking for GPX uploads
  • Add exception handling for GDALExceptions
  • Updated tests
  • Add .eslintrc which was removed in previous commit

When should this PR be merged

Immediately

Risks

None

Follow up actions

None required

@ian-ross ian-ross merged commit 1b5e565 into master Sep 13, 2016
@ian-ross ian-ross deleted the bugfix/#670 branch September 13, 2016 12:35
@@ -0,0 +1,14 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we using ES Lint?

@bjohare
Copy link
Contributor Author

bjohare commented Sep 13, 2016

I thought we were, but maybe I shouldn't have committed that?

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.

3 participants