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.

Sign up for GitHub

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

Jump to bottom

Installer not respecting project.yaml #3291

Closed
brandonkelly opened this issue Sep 11, 2018 · 2 comments
Closed

Installer not respecting project.yaml #3291

brandonkelly opened this issue Sep 11, 2018 · 2 comments
Assignees
@andris-sevcenko
Labels
enhancement improvements to existing features project config 🐫 features related to Project config

Comments

@brandonkelly
Copy link
Member

Description

After installation, if the useProjectConfigFile config setting is enabled, Craft should check if there’s a config/project.yaml file and if so, not overwrite it, and allow the next request to apply the changes in it.

Steps to reproduce

  1. Ensure a project.yaml file is in place and the useProjectConfigFile config setting is enabled
  2. Install Craft

Additional info

  • Craft version: 3.1.0-alpha.1
@brandonkelly brandonkelly added enhancement improvements to existing features project config 🐫 features related to Project config labels Sep 11, 2018
@brandonkelly
Copy link
Member Author

This should solve #881.

@andris-sevcenko andris-sevcenko mentioned this issue Dec 14, 2018
Closed
andris-sevcenko added a commit that referenced this issue Jan 2, 2019
@andris-sevcenko
Closed #3291
3878d16
@andris-sevcenko
Copy link
Contributor

Closed via 3878d16

@wouter-vs wouter-vs mentioned this issue Apr 9, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andris-sevcenko andris-sevcenko

Labels
enhancement improvements to existing features project config 🐫 features related to Project config
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@brandonkelly @andris-sevcenko