Skip to content

Configure() Prompt Improvements #718

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

Open
isc-npetroce opened this issue Mar 7, 2025 · 1 comment
Open

Configure() Prompt Improvements #718

isc-npetroce opened this issue Mar 7, 2025 · 1 comment
Labels
good first issue Good for newcomers

Comments

@isc-npetroce
Copy link
Collaborator

While configuring Embedded Git with do ##class(SourceControl.Git.API).Configure() for the first time with @isc-pbarton , we found a few areas where the prompts could be clarified and improved:

  1. 'The system's default mode' prompt should be before the user setting for basic mode, not after. Additionally, the term 'system' here should be replaced with 'Namespace' as this sets basic/expert mode at the Namespace level.
  2. 'Branch from which Sync will merge changes, in basic mode': This prompt should not be asked if the plugin is in expert mode.
  3. 'Define a namespace-level web application allowing access to multiple git repos across separate namespaces': In reality the purpose of this is to create a web application passthrough allowing you to edit the files in the git repo directly in VSCode, rather than just editing files in the SMP. This should be clarified in the prompt.
@isc-pbarton isc-pbarton added the good first issue Good for newcomers label Mar 10, 2025
@cambot
Copy link

cambot commented Apr 18, 2025

Two more for your list:

  1. the SSH key prompt should be co-located with the repository URL prompt.
  2. Provide an example ssh oriented URL for the git clone using this project as an example.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants