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

[Bugfix] Uncover cmd len warn #6536

Merged
merged 16 commits into from
Jul 9, 2024
Merged

[Bugfix] Uncover cmd len warn #6536

merged 16 commits into from
Jul 9, 2024

Conversation

hjoaquim
Copy link
Contributor

  1. Why? (1-3 sentences or a bullet point list):

    • Commands needs to be displayed differently than their original name if it's too big, and the user needs to be aware of this.
  2. What? (1-3 sentences or a bullet point list):

    • The warning was already there but under dev configuration flags, simply removing.
  3. Impact (1-2 sentences or a bullet point list):

    • NA
  4. Testing Done:

    • Navigate to any many where this happens, example economy or economy/survey
  5. Reviewer Notes (optional):

image

@github-actions github-actions bot added bug Fix bug platform OpenBB Platform v4 PRs for v4 labels Jun 24, 2024
@hjoaquim hjoaquim added the cli OpenBB Platform CLI label Jun 24, 2024
@hjoaquim hjoaquim requested a review from IgorWounds June 24, 2024 07:49
@hjoaquim hjoaquim added this pull request to the merge queue Jul 9, 2024
Merged via the queue into develop with commit 82fa129 Jul 9, 2024
14 checks passed
@IgorWounds IgorWounds deleted the bugfix/uncover-cmd-len-warn branch July 9, 2024 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fix bug cli OpenBB Platform CLI platform OpenBB Platform v4 PRs for v4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants