intersystems / intersystems/git-source-control

Configure() Prompt Improvements

Open
#718 1 comment 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
ObjectScript
Stars
22
Forks
14
Avg merge
1d 9h
Merged PRs (30d)
5

Description

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.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.