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

Update design_config_form.xml #8880

Merged
merged 1 commit into from
Apr 19, 2017
Merged

Update design_config_form.xml #8880

merged 1 commit into from
Apr 19, 2017

Conversation

WaPoNe
Copy link
Contributor

@WaPoNe WaPoNe commented Mar 14, 2017

Missing ico and apng extension in 'allowedExtensions' parameter

Description

Trying to upload favicon image with .ico extension, you receive this error: "We don't recognize or support this file extension type."

Manual testing scenarios

  1. ...
  2. ...

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

Missing ico and apng extension in 'allowedExtensions' parameter
@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Mar 14, 2017

CLA assistant check
All committers have signed the CLA.

@okorshenko okorshenko self-assigned this Mar 15, 2017
@okorshenko okorshenko added this to the April 2017 milestone Apr 18, 2017
@okorshenko okorshenko added the 2.1 label Apr 18, 2017
@okorshenko okorshenko changed the base branch from 2.1 to 2.1-develop April 18, 2017 00:58
@magento-team magento-team merged commit 406c817 into magento:2.1-develop Apr 19, 2017
@magento-team
Copy link
Contributor

@WaPoNe Thank you for the contribution

@WaPoNe WaPoNe mentioned this pull request Jun 22, 2017
4 tasks
magento-devops-reposync-svc pushed a commit that referenced this pull request Jun 18, 2024
Investigate the league/flysystem Composer dependencies upgrading to latest version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants