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

Added information about the missing settings to serve Magento app from /pub directory #7038

Merged
merged 4 commits into from
Apr 21, 2020

Conversation

thiaramus
Copy link
Contributor

Added information about the missing settings and steps when the web server is configured to serve the Magento app from the /pub directory.

Purpose of this pull request

There are multiple resolved issues that suggest these settings to be added to the configuration file env.php in order for the Magento app to properly work when being served from the /pub directory instead of /. I think it is necessary to add this information to DevDocs for future references.

Affected DevDocs pages

Links to Magento source code

  • app/etc/env.php

…erver is being configured to serve the Magento app from the `/pub` directory.
@magento-engcom-team
Copy link

@thiaramus thank you for contributing. Please accept Community Contributors team invitation here to gain extended permissions for this repository.

@devops-devdocs
Copy link
Collaborator

An admin must run tests on this PR before it can be merged.

@nuzil
Copy link
Contributor

nuzil commented Apr 15, 2020

Thanks @thiaramus , its definitely make sense

…erver is being configured to serve the Magento app from the `/pub` directory.
@thiaramus thiaramus requested a review from dobooth April 20, 2020 20:29
@dobooth
Copy link
Contributor

dobooth commented Apr 21, 2020

Thanks for the updates, @thiaramus. Can you please update your branch here and ping me when you do? Thanks.

@dobooth
Copy link
Contributor

dobooth commented Apr 21, 2020

running tests

…erver is being configured to serve the Magento app from the `/pub` directory.
@dobooth
Copy link
Contributor

dobooth commented Apr 21, 2020

running tests

@dobooth dobooth merged commit c6f49c5 into magento:master Apr 21, 2020
@ghost
Copy link

ghost commented Apr 21, 2020

Hi @thiaramus, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

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.

5 participants