- Notifications
You must be signed in to change notification settings - Fork113
Support for Groups in Connections File#311
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thanks for the PR, and sorry for holding this for so long.
Can you rebase from master and fix the conflicts? There were some changes in the rendering logic from the previous PR.
Done! |
Support for Groups in Connections File
This MR adds support for groups in the connections file andcloses#135.
All changes are fully backward compatible with existing connection files.
New Structure
[ {"name":"this_is_a_group","connections": [ {"url":"postgres://postgres:postgres@localhost:5432/postgres","name":"this is a nested connection" } ] }, {"url":"postgres://postgres:postgres@localhost:5432/users","name":"users" }]Implementation Notes
No additional group manipulation commands were implemented.
Grouping is handled naturally by prefixing connection names with the desired group path when using :DBUIAddConnection.
Groups could be renamed and deleted as usual connection.