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.

Sign up for GitHub

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

Jump to bottom

refactor adding database to monitoring modal window #52

Closed
6 tasks done
pashagolub opened this issue Nov 14, 2022 · 0 comments · Fixed by #64
Closed
6 tasks done

refactor adding database to monitoring modal window #52

pashagolub opened this issue Nov 14, 2022 · 0 comments · Fixed by #64
Assignees
@pashagolub @denys-holub
Labels
enhancement New feature or request webui Admin UI related issues
Milestone
Built-in WebUI

Comments

@pashagolub
Copy link
Collaborator

pashagolub commented Nov 14, 2022 •
edited by denys-holub
Loading

Right now "Add new..." modal window lists all possible connection options on one screen with vertical scrollbar. Let's refactor this windows into tabbed UI.

  • Caption should be: "Add new database to monitoring..."
  • Buttons on the bottom panel #53
    • Test connection...
    • Start monitoring
    • Cancel
  • First tab "Main"
    • Enabled
    • Unique name
    • Monitored type, e.g. simple database, pgpool, patroni, etc.
    • Group
    • Tags
  • Second tab "Connection"
    • Host
    • Port
    • DB name (inclusion/exclusion paterns)
    • User
    • Password
    • Connection Timeout
    • Statement Timeout
    • Button "Test connection..."
  • Third tab "SSL"
    • Use SSL Connection
    • SSL Client Certificate
    • SSL Private key
  • Fourth tab "Presets"
    • Preset
    • Auto create helpers
    • Master mode only?

Before:
image

After:
image
@pashagolub pashagolub added enhancement New feature or request webui Admin UI related issues labels Nov 14, 2022
@pashagolub pashagolub added this to the Built-in WebUI milestone Nov 14, 2022
@Lana8Lee Lana8Lee linked a pull request Dec 2, 2022 that will close this issue
refactor adding database to monitoring modal window, closes #52 #64
Merged
pashagolub added a commit that referenced this issue Dec 5, 2022
@pashagolub
Merge pull request #64 from cybertec-postgresql/52-refactor-adding-da…
6cda65e 
…tabase-to-monitoring-modal-window

 refactor adding database to monitoring modal window, closes #52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pashagolub pashagolub

@denys-holub denys-holub

Labels
enhancement New feature or request webui Admin UI related issues
Projects
None yet
Milestone
Built-in WebUI
Development

Successfully merging a pull request may close this issue.

refactor adding database to monitoring modal window, closes #52 cybertec-postgresql/pgwatch
2 participants
@pashagolub @denys-holub