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

JSON not in default allowedFileExtensions config array. Yet assets allows you to select JSON as allowed file type. #3254

Closed
andrewhawkes opened this issue Aug 30, 2018 · 2 comments

Comments

@andrewhawkes
Copy link

Description

I wonder if json needs adding to the default allowedFileExtensions array as documented here as you can select JSON as an "allowed file type"?

image

Luckily, I knew about this config setting but it could be very confusing and time wasting for some users.

Steps to reproduce

  1. Add assets field with JSON as allowed file
  2. Try upload a JSON file
  3. Get an error, because JSON isn't allowed by default in the config

Additional info

  • Craft version: Craft CMS 3.0.13.2
@brandonkelly
Copy link
Member

Thanks for pointing that out! json definitely should have been in that list. Just added for the next release.

@andrewhawkes
Copy link
Author

@brandonkelly great cheers!

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

No branches or pull requests

2 participants