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.

Sign up for GitHub

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

Jump to bottom

UG states that WALLET_NAME for editpa is case-sensitive, but it is not. #14

Open
foothaddeus opened this issue Nov 8, 2024 · 0 comments
Open

UG states that WALLET_NAME for editpa is case-sensitive, but it is not. #14

foothaddeus opened this issue Nov 8, 2024 · 0 comments
Labels
severity.Medium type.FeatureFlaw

Comments

@foothaddeus
Copy link
Owner

foothaddeus commented Nov 8, 2024 •
edited
Loading

This was what was stated in the UG:

image.png

However, this is allowed:

Before:

image.png

After entering editpa 3 c/BTC l/daily wallet pa/14qViLJfdGaP4EeHnDyJbEGQysnCpwk3g:

image.png

This means that the feature does not work as intended. Could consider making WALLET_NAME case-insensitive for editpa command, which would fit better with the specifications for addpa as well since it is case-insensitive for that command:

image.png
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
severity.Medium type.FeatureFlaw
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@foothaddeus