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

Facilitate editing of all YAML settings in the site.yaml config file #38

Closed
mikegcox opened this issue Aug 7, 2015 · 7 comments
Closed

Comments

@mikegcox
Copy link

mikegcox commented Aug 7, 2015

Currently editing the site.yaml file is limited to default data. It would be good to be able to edit all the YAML in that file as it is a good place to store site settings that a website content administrator would need to make changes to periodically.

@rhukster
Copy link
Member

rhukster commented Aug 7, 2015

Thanks mike...

@rhukster
Copy link
Member

Things that need to happen here:

  1. user/blueprints/ merged with system/blueprints. Currently user/blueprints are picked up but completley override he ones from system. So you must copy all blueprints from system to user or it breaks things.

  2. Supporting @extends would be good, currently this sends things into an infinite loop.

I'm going to ask matias about these things which are really needed before we can implement this functionality.

@rhukster rhukster assigned mahagr and unassigned rhukster Aug 12, 2015
@rhukster
Copy link
Member

Also we might be able to provide the 'extra' fields in a textarea rather than in hidden json encoded fields as they are now.

@mahagr
Copy link
Member

mahagr commented Aug 12, 2015

I've been thinking something like this for both blueprints and twig; to be able to say extends parent. We do need similar functionality also in Gantry 5 as I really hate copying the whole files because of the base file cannot be seen.

@mahagr
Copy link
Member

mahagr commented Aug 20, 2015

Fix for 1) is in: getgrav/grav@4a59b8a

@mahagr
Copy link
Member

mahagr commented Aug 20, 2015

Fix for 2) is in: getgrav/grav@c4cd355

Just note that it allows only single level of inheritance -- I will need to think up a way to generalize the function.

@rhukster
Copy link
Member

Very cool! Closing...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants