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

Improve overall UX #1804

Merged
merged 5 commits into from
May 12, 2022
Merged

Improve overall UX #1804

merged 5 commits into from
May 12, 2022

Conversation

DidierRLopes
Copy link
Collaborator

Do not display vREPLACE_ME. Don't kill me @piiq but I have OCD and couldn't not add a protection against that.
Also added documentation on the first line so users can see more documentation
Screenshot 2022-05-12 at 01 58 01

Website added in bottom right of panel:
Screenshot 2022-05-12 at 01 21 34

About has documentation and website
Screenshot 2022-05-12 at 01 59 03

Exit message is uniform and says:
Screenshot 2022-05-12 at 01 59 30

@DidierRLopes DidierRLopes added docs Code documentation enhancement Enhancement labels May 12, 2022
@DidierRLopes DidierRLopes requested a review from jmaslek May 12, 2022 00:59
@andrewkenreich
Copy link
Contributor

we need version somewhere still pls :)

Troubleshooting this would be very annoying.

how about in the bottom right "OpenBB Terminal v1.2.1 ( https://openbb.co)"

@DidierRLopes
Copy link
Collaborator Author

v1.2.1

The version will appear if it exists when logging on the terminal in any case.

But I like your suggestion actually, will do.

@DidierRLopes DidierRLopes merged commit a2ecbe9 into main May 12, 2022
@piiq piiq deleted the improve-ux branch May 12, 2022 04:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Code documentation enhancement Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants