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

Add --recursive option for config Import #41

Merged
merged 1 commit into from
Mar 13, 2021
Merged

Conversation

Maksold
Copy link
Contributor

@Maksold Maksold commented Feb 10, 2021

Issue

.

Proposed changes

I want to support this config files structure

magento_root
...
├── config
│   └── store
│       ├── base
│       │       ├── vendor1
│       │       │   ├── geoip.yaml
│       │       │   └── storepickup.yaml
│       │       └── vendor2
│       │           └── general.yaml
│       └── staging
│               ├── vendor1
│               │   └── recaptcha.yaml
│               └── vendor2
│                   └── general.yaml
...

For this purpose I added -r option for Import command. It allows to find config files recursively and import them.

So I use this command bin/magento config:data:import config staging -r to import my configs.

@Maksold
Copy link
Contributor Author

Maksold commented Feb 16, 2021

@therouv could you please approve my commit?

@therouv therouv merged commit f86c816 into semaio:develop Mar 13, 2021
@therouv
Copy link
Contributor

therouv commented Mar 13, 2021

@Maksold Thank you for your contribution. I've made some minor changes and added a recommended folder setup in the readme.

@Maksold
Copy link
Contributor Author

Maksold commented Mar 23, 2021

@therouv Thank you for accepting my PR! ;)

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.

2 participants