Follow up for #1648930: Introduce configuration schema and use for translation
Problem/Motivation
The file location and names in #1648930: Introduce configuration schema and use for translation work but discussing many ways regarding the specific file locations and names is distracting that critical issue.
The discussion of the file location and naming convention needs a dedicated and non-rushed place to polish it and come up with something really nice.
This can happen after feature freeze.
Proposed resolution
I'll copy the relevant comments from the originating issue into the first few comments on this new issue, and then I'll summarize the approaches here in the issue summary.
Remaining tasks
asap
- copy the relevant suggestions from original issue
- summarize them in the issue summary
after feature freeze or at least when people want a break from feature freeze tasks
- discuss pros and cons
- decide on a approach to pursue
- mock up examples
- implement code changes
User interface changes
no ui changes.
API changes
no api changes anticipated. (just implementation of location of files)
Related
#1851498: Polish file format of configuration metadata (for translation)
Comments
Comment #1
webchickPostponing this until the CMI i18n patch is in; no sense in holding up other features in the meantime.
Comment #2
YesCT CreditAttribution: YesCT commented1. meta location and names
Here are the current locations, and names, of the meta files that the patch adds from #1648930-247: Introduce configuration schema and use for translation. Just documenting it. Not lobbying for it.
Generated this list with:
$ grep yml config_metadata-1648930-247.patch | grep +++ | cut -f2 -d" " | sed 'sxb/xx'
2. an (*the*) include in a meta
We have one (or so grep says) example with an include:
3. config names and locations
Here are the config names and locations (at least at the time of this posting)
Comment #3
YesCT CreditAttribution: YesCT commentedI'm a little less clear on location, so I didn't add much history. Please feel free to add any history you think is relevant, or just add new ideas.
The new include format I think helped reduce some concerns about implying merging based on location.
222.
names and % patterns
#1648930-222: Introduce configuration schema and use for translation by @Jose Reyero
237.
#1648930-237: Introduce configuration schema and use for translation
@effulgentsia
238.
#1648930-238: Introduce configuration schema and use for translation
@sun
Comment #4
Jose Reyero CreditAttribution: Jose Reyero commentedOn top of the location of the generic metadata files (that I really don't mind that much since any folder would work) we have the specific issue of metadata for plugins, that may need to be addressed differently maybe:
- Adding it as annotations into the plugin classes (which I'm not a big fan of)
- Placing it side by side with the plugin classes, in the same folder the class is, for which we may need to define some specific namespace in the configuration names (like 'plugin.*') to be able to resolve it to the right folder.
- Having one or more levels of subdirectories in the main metadata folder for each module. Example:
This is an example of how views metadata looks like when trying to describe properties for plugins (not complete yet): http://drupalcode.org/sandbox/reyero/1635230.git/tree/8facc18e7fd8fd09d9...
Comment #5
Gábor HojtsyWith the form generation concept discarded in #1648930: Introduce configuration schema and use for translation actual work is happening in #1861640: Provide config metadata solely for translation with the express goal of having a format/placement that people can agree on out of the gate. No need for followups.