-
Notifications
You must be signed in to change notification settings - Fork 467
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
CLI: Case Sensitivity Issue with "direction" Parameter in CLI #28143
Comments
valentinogiardino
added a commit
that referenced
this issue
Jun 28, 2024
valentinogiardino
added a commit
that referenced
this issue
Jun 28, 2024
github-merge-queue bot
pushed a commit
that referenced
this issue
Jun 28, 2024
### Proposed Changes * Transform the direction parameter to uppercase * Add test cases * Update CLI documentation to reflect this change ### Screenshots ![screenshot-case-insensitive-fix](https://github.com/dotCMS/core/assets/77643678/88bfad81-d990-46b4-a54b-b53433a14730) This PR fixes: #28143
Passed Internal QA
Videoissue-28143-cli-case-sensitivity-issue-with-direction-parameter.mov |
Note to QAThe P.S.: You can ask @dcolina to update the |
valentinogiardino
added a commit
that referenced
this issue
Jul 2, 2024
valentinogiardino
added a commit
that referenced
this issue
Jul 2, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Problem Statement
In the dotCMS Command Line Interface (CLI), there is an issue where the "direction" parameter is case sensitive. This inconsistency causes inconvenience for users, especially when specifying sorting direction, as they may unintentionally use incorrect capitalization and encounter unexpected behavior.
Steps to Reproduce
Acceptance Criteria
dotCMS Version
Tested on master // lastest CLI version
Proposed Objective
Quality Assurance
Proposed Priority
Priority 2 - Important
The text was updated successfully, but these errors were encountered: