Open ponceta opened 7 years ago
Another requirement discussed with @sylvainbeo : local customization scripts should also be injected in the conformity version testing scripts. Beyond naming convention, we should reserve versionning namespaces or numbers.
So we took advantage of PUM to test the customization logic, as discussed at the PSC last week
From what we see, a very simple option would be:
Another point found is that most frequent customization are value list insert / delete or update for labels / translations. Running a QWAT specific step to display those value list changes would be nice too.
@3nids @m-kuhn . Any opinion?
Let's consider some examples:
I guess most of the customizations are additions like this.
What would the local customization logic look like in this case?
I guess most of them would need nothing at all, (added tables, and fields will just be kept unless incompatible upstream changes land), for the views, it will probably also require a second script to kill the views before starting the pumpdate, for triggers as well.
For additional values I'd also say, they will be just kept without requiring migration logic (99% of the time at least). For removing values, I'd discourage people and add a disabled
flag to value lists if this is required.
@m-kuhn Miss an added schema, an added group_role, an added rule, an added type. With that we should be exhaustive?
Edit: set group_role instead of role after @haubourg's comment.
With these rules and guidelines, any organization should be able to be "PUM compliant"!
@ponceta about ROLES, I think we should only check "group" roles, ie those with NO LOGIN
option. The connection roles will be too much variant (imagine you connect a LDAP to PG, they change virtually at any moment).
Documentation is here:
https://qwat.github.io/docs/master/en/html/developer-guide/local_customizations.html
I keep this open until 1.3.3 Release and deployement in Pully.
@ponceta should we leave this issue opened?
@lbartoletti I miss Custom values in the value lists customization the https://github.com/qwat/qwat-data-model/issues/122
We agreed with @tudorbarascu and @3nids on this in 2016.
@kandre thoughts?
Can I add it in the documentation too?
And Pully is up to date on QWAT 1.3.3 (freshly updated to postgres 9.6)
These aspects should be all explained in the data manager documentation.