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

[PE-D][Tester E] UG states that WALLET_NAME for editpa is case-sensitive, but it is not. #257

Closed
nus-se-script opened this issue Nov 8, 2024 · 1 comment · Fixed by #277
Assignees
Labels
Milestone

Comments

@nus-se-script
Copy link

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


Labels: severity.Medium type.FeatureFlaw
original: foothaddeus/ped#14

@Airiinnn
Copy link

Valid.

Documentation bug. Duplicate of #201

@Airiinnn Airiinnn self-assigned this Nov 10, 2024
@Airiinnn Airiinnn added the type.Bug A bug label Nov 10, 2024
@Airiinnn Airiinnn added this to the v1.6 milestone Nov 10, 2024
@Airiinnn Airiinnn assigned Nicholascyx and unassigned Airiinnn Nov 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants