-
Notifications
You must be signed in to change notification settings - Fork 55
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
Keep "last client id created" in memory and offer that as the default for commands that require client ID [CLI-91] #190
Comments
We can store it in the config file so it survives across command invocations. |
Also, we can use it to set a default value in the picker (otherwise just the Client Id, devoid of all context, may be confusing to the dev that may no realize it is the Id of the app recently created). |
yes. When I say default I meant that exactly (otherwise it would be too
much magic)
…On Wed, Mar 24, 2021 at 7:31 PM Rita Zerrizuela ***@***.***> wrote:
Also, we can use it to select a default value in the picker (otherwise
just the Client Id, devoid of all context, may be confusing to the dev that
may no realize it is the Id of the app recently created).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#190 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABK6CCXVOJSULZDDF6K463TFJR4BANCNFSM4ZYEXPVQ>
.
|
This might have to be accompanied by an |
Tracking in CLI-91. |
* apps picker: allow configurability of default app Closes #190 * Show hint with the client id * Update internal/cli/apps.go Co-authored-by: Rita Zerrizuela <[email protected]> Co-authored-by: Rita Zerrizuela <[email protected]>
Instead of having to find the client id, we can offer the last client created as the default option (just store it in a global var in memory). Anywhere where we ask for client ID we should default to that. Delighter.
The text was updated successfully, but these errors were encountered: