Skip to content

Add Dialog Interface #4

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Conversation

muse-sisay
Copy link

@muse-sisay muse-sisay commented Jun 14, 2021

This PR introduces dialog for interactive input from sys-admin.

Screen Shot 2021-06-15 at 10 05 29 PM

This is still a work in progress.

TODO

  • Check for dialog
  • Get New user's info. [ Full Name, room number , work phone, home phone ,other ]
  • New user's password
  • Get Git info
  • Time Zone selection ( should check if it exists)
  • Have a list of services to install
  • Description

Dialog's to add

Add dialog's to enter the following information

  • ssh key
  • hostname
  • Sendgrid API
  • Backup and Project folder path
  • Mail detail

Screen Shot 2021-06-15 at 10 05 44 PM

@muse-sisay muse-sisay marked this pull request as ready for review June 15, 2021 16:52
@muse-sisay muse-sisay marked this pull request as draft June 15, 2021 16:54
@engineervix
Copy link
Owner

Hi @muse-sisay, this looks good so far! Although I kinda think having a dialog interface with so many options will slow down the setup process because now you have to start making selections.

One of the key things to deal is eliminating the prompts that come mid-way through the setup process, for example,

  • during installation of postgres
  • setting up unattended upgrades

It would be good if the user has the option of deciding whether they wanna use the dialog interface or just the traditional setup as it is currently. So, maybe the script could be run like this

# this invokes the dialog interface
bash setup.sh --gui

or something in those lines.

Another option I was thinking of is providing some kind of *.ini or *.conf file where all the variables are set, and the setup script just reads from that file. But the password should still be entered by hand to avoid accidentally disclosing it, lol.

@engineervix engineervix added the enhancement New feature or request label Jul 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants