Comments (1)
Team's Response
Reason
This is an intended behaviour as we want to inform the user that they have not assigned a rating or written a comment to the show they have added. We are rejecting this issue as we feel that this is not a severity.Low bug at all since the operation of the application is not affected at all and it does not cause any inconvenience to the users. We are also downgrading the severity of this issue to severity.VeryLow due to the above reason.
Items for the Tester to Verify
❓ Issue response
Team chose [response.Rejected
]
- I disagree
Reason for disagreement: The user guide did not specify the default behaviour of an optional field. Given there are a lot of different ways to handle optional field behaviour such as populating the field with a default value, nil, NULL or undefined, this behaviour should be properly documented in the user guide as mentioned in the bug report guideline.
Since the behaviour is not documented in the user guide, I would assume that the Dev team would handle this behaviour by informing the users via other means -- the display response for this scenario. However, the user would instead receive a confusing message - New show added: All of us are dead; Status: plan-to-watch; Rating: 0; Comment: ; Tags: [Horror];
despite not using c/
and r/
in the command.
Moving forward
The optional field visual bug could be avoided in the following examples:
New show added: All of us are dead; Status: plan-to-watch; Rating: 0; Comment: No Comment; Tags: [Horror];
OR
New show added: All of us are dead; Status: plan-to-watch; Tags: [Horror];
Rating: Not specified;
Comment: Not specified;
OR
3. Remove unused prefixes
![image.png](https://raw.githubusercontent.com/clement0010/pe/master/files/21ee12ef-4178-4210-b16a-49f9b6f138cf.png)
-------------------
## :question: Issue severity
Team chose [`severity.VeryLow`]
Originally [`severity.Low`]
- [x] I disagree
**Reason for disagreement:** This is not a cosmetic bug. Justification is the same as above.
-------------------
from pe.
Related Issues (8)
- Resizing help column causes UI bug HOT 1
- Default value for show rating is not specified HOT 1
- Inaccurate input validation when adding a new show HOT 1
- Unclear instructions on the usage of parameter tables HOT 1
- Unclear usage for sorting sequence parameter HOT 1
- Multiple prefix name used in sorting sequence parameter HOT 1
- Grammar error for import and export show 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.