Notes on actual page:
here
If you like, I have the code for you!
Notes on actual page:
here
If you like, I have the code for you!
#992 is required for #377, which uses the new information and just adds the nested details in the current layout. (I wasn’t sure you will like the new layout.)
I have not created PR for this new layout yet. If you want, I can create new one or add it to #377.
I can understand, that #992 contains some changes you might consider too risky although I’ve taken extra care and created tests every change I’ve made. If you want, I can split into 2 PRs 1 with “easy” harmless changes and another one with the tricky parts.
Regards,
Gyula Bibernáth
This is a really awesome change! Thanks for doing all this and setting up a demo documentation website. That really helps show me what is happening.
Datical has been working on how we can process PRs more quickly, and we’d like to start with this one. I’m reviewing the pull request in more detail now. We should have some movement on it early next week.
Steve Donie
Principal Software Engineer
Liquibase Community Engagement
Datical, Inc. http://www.datical.com/
I am unclear if this proposed change is the same as your pull request #992, which I am currently reviewing.
https://github.com/liquibase/liquibase/pull/992
Steve Donie
Principal Software Engineer
Liquibase Community Engagement
Datical, Inc. http://www.datical.com/
I’ve created a new PR #407 containing the changes in the generator to generate the described new layout, that is visible @ liquibase-doc. All the elements can be styled separately in the liquibase.css
It does not have binary dependency on core changes, although some property descriptions will be visible only after core PR #1002 has been merged. Nested properties insert, update / whereParams
and executeCommand / args
will be visible after core RP #992 is has been merged.
Regards, Gyula Bibernáth