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

Merge of resources from app + package #159

Open
Levivb opened this issue Sep 30, 2019 · 3 comments
Open

Merge of resources from app + package #159

Levivb opened this issue Sep 30, 2019 · 3 comments

Comments

@Levivb
Copy link

Levivb commented Sep 30, 2019

Hey,

This package support the possibility to customize the resources in config/health/resources. Upon doing so, the resources in the packages are skipped completely. It's quite a shame to copy-paste all resources just because one resource needs a bit of customization.

Would it be possible to merge the resources in the config directory and the resources in the package? That way, most code is maintained in the package and only that what needs to be changed is available in developer projects

Let me know what you think :)

@kost999
Copy link

kost999 commented Sep 30, 2019

It is very useful!
http://joxi.ru/12MgyOqflRVnMr
And can be realised right here by merging files arrays

@antonioribeiro
Copy link
Owner

@Levivb I completely agree with you, when I first built this, I had the impression people would have to customize most of the resources.

Can you provide a Pull Request to do this merge?

@antonioribeiro
Copy link
Owner

I have a question: if we don't copy all those files from the package dir, how will copy and related to them when they need to modify something in the resource?

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

3 participants