Comments (1)
Team's Response
No details provided by team.
The 'Original' Bug
[The team marked this bug as a duplicate of the following bug]
Command summary table can be clearer
Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.
![]()
Here,
i/NRIC
is indicated for add and view, butedit INDEX
is not indicated. Inconsistent especially between view and edit, since bothi/NRIC
andINDEX
should serve the same purpose of identifying the contact to view/edit.This makes it confusing for the user, especially since one would expect to edit the same way you view, or add a contact. Since this is not the case, it is better to be explicit with the indexing method.
[original: nus-cs2103-AY2122S2/pe-interim#1864] [original labels: severity.Low type.DocumentationBug]
Their Response to the 'Original' Bug
[This is the team's response to the above 'original' bug]
Agree that the command tables can be clearer to differentiate the use of referencing parameters
i/NRIC
andINDEX
.
Items for the Tester to Verify
❓ Issue duplicate status
Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)
- I disagree
Reason for disagreement: The original bug which was accepted was between the Patient Specific Commands: Edit
and Patient Specific Commands: View
, where view
description includes the contact-identifier, whereas edit does not.
In the bug report that was flagged as duplicate, the issue is between Patient Specific Commands: View
and Contact Specific Commands: View
. Both are similar commands by nature, yet the patient view
includes the NRIC identifier parameter, whereas contact view
does not include the NRIC identifier parameter.
Excluding this bug report would've meant that this issue would've gone unnoticed, since it does not overlap with the original bug at all.
from pe.
Related Issues (13)
- Inconsistency in commands HOT 1
- Unable to reset password HOT 1
- Password error message typo HOT 1
- Sample data incorrect HOT 1
- Delete in view HOT 1
- Input NRIC should be non-case sensitive HOT 1
- Editing patient's detail in summary page brings us back to dashboard/main view page HOT 1
- Find does not work for personal details HOT 1
- Wrong usage description HOT 1
- No list of parameters for edit HOT 1
- Command summary table can be clearer HOT 1
- View consultation diagram is hard to see HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from pe.