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.

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

[chectl] [ux] Reorder/relabel successful deployment messages #18122

Closed
nickboldt opened this issue Oct 15, 2020 · 1 comment
Closed

[chectl] [ux] Reorder/relabel successful deployment messages #18122

nickboldt opened this issue Oct 15, 2020 · 1 comment
Labels
area/chectl Issues related to chectl, the CLI of Che area/ux Issues related to User Experience Design kind/enhancement A feature request - must adhere to the feature request template. severity/P1 Has a major impact to usage or development of the system.
Milestone

Comments

@nickboldt
Copy link
Contributor

nickboldt commented Oct 15, 2020

Is your enhancement related to a problem? Please describe.

Current deployment success message is great, but could be reordered a little for better grouping of content.

As of che-incubator/chectl#913 we have:

  ✔ Show important messages
    ✔ [ACTION REQUIRED] Please add Che self-signed CA certificate into your browser: /home/tolusha/cheCA.crt.
   Documentation how to add a CA certificate into a browser: https://www.eclipse.org/che/docs/che-7/overview/importing-certificates-to-browsers/
    ✔ Eclipse Che 7.20.0-SNAPSHOT has successfully installed.
    ✔ Users can access the Dashboard at   : https://che-che.192.168.99.100.nip.io
    ✔ Plugins can be viewed at            : https://plugin-registry-che.192.168.99.100.nip.io/v3
    ✔ Devfiles can be viewed at           : https://devfile-registry-che.192.168.99.100.nip.io
    ✔ Identity provider can be accessed at: https://keycloak-che.192.168.99.100.nip.io/auth
    ✔ Product documentation is at         : https://www.eclipse.org/che/docs/
    ✔ Eclipse Che admin credentials       : "admin:admin". You will be asked to change default Che admin password on the first login.
    ✔ Identity Provider credentials       : "admin:qzwHgv9ZkHdP"

I'd suggest we reorder/relabel that a bit. See below.

Describe the solution you'd like

  ✔ Show important messages
    ✔ Eclipse Che 7.20.0-SNAPSHOT has been successfully deployed.
    ✔ Documentation            : https://www.eclipse.org/che/docs/

    ✔ Users Dashboard          : https://che-che.192.168.99.100.nip.io
    ✔ Admin user login         : "admin:admin". NOTE: must change after first login.

    ✔ Plug-in Registry         : https://plugin-registry-che.192.168.99.100.nip.io/v3
    ✔ Devfile Registry         : https://devfile-registry-che.192.168.99.100.nip.io

    ✔ Identity Provider URL    : https://keycloak-che.192.168.99.100.nip.io/auth
    ✔ Identity Provider login  : "admin:qzwHgv9ZkHdP"

    ✔ [ACTION REQUIRED] Please add self-signed CA certificate into your browser: /home/tolusha/cheCA.crt
   See https://www.eclipse.org/che/docs/che-7/overview/importing-certificates-to-browsers/ for instructions.
@nickboldt nickboldt added the kind/enhancement A feature request - must adhere to the feature request template. label Oct 15, 2020
@nickboldt nickboldt changed the title [chectl] [ux] Reorder successful deployment messages [chectl] [ux] Reorder/relabel successful deployment messages Oct 15, 2020
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Oct 15, 2020
@nickboldt nickboldt added area/chectl Issues related to chectl, the CLI of Che area/ux Issues related to User Experience Design team/deploy labels Oct 15, 2020
@tolusha tolusha added severity/P1 Has a major impact to usage or development of the system. and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. team/deploy labels Oct 19, 2020
@tolusha
Copy link
Contributor

tolusha commented Oct 19, 2020

@nickboldt
I can print empty lines like this

  ✔ Eclipse Che 7.21.0-SNAPSHOT has been successfully deployed.
    ✔ Documentation             : https://www.eclipse.org/che/docs/
    ✔ 
    ✔ Users Dashboard           : https://che-che.192.168.99.100.nip.io
    ✔ Admin user login          : "admin:admin". NOTE: must change after first login.
    ✔ 
    ✔ Plug-in Registry          : https://plugin-registry-che.192.168.99.100.nip.io/v3
    ✔ Devfile Registry          : https://devfile-registry-che.192.168.99.100.nip.io
    ✔ 
    ✔ Identity Provider URL     : https://keycloak-che.192.168.99.100.nip.io/auth

@tolusha tolusha closed this as completed Oct 20, 2020
@tolusha tolusha added this to the 7.21 milestone Oct 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/chectl Issues related to chectl, the CLI of Che area/ux Issues related to User Experience Design kind/enhancement A feature request - must adhere to the feature request template. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

3 participants