GithubHelp home page GithubHelp logo

Comments (1)

nus-pe-script avatar nus-pe-script commented on August 12, 2024

Team's Response

This presents a similar issue as #2079. A calculator feature can be used for calculating the changes in the rewards when multiple vouchers are used.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Having a calculator to calculate change in this application is weird

image.png

image.png

This app is to manage customer's membership details on a desktop, not a cash register;
irrelevant feature


[original: nus-cs2103-AY2223S1/pe-interim#2697] [original labels: type.FeatureFlaw severity.VeryLow]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

You can use it to calculate the leftover reward points after claiming multiple vouchers, or any other tasks that require calculation.

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: [replace this with your explanation]


โ“ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: The developers' response highlights possible uses for a calculator, but not the issue of whether this app needs the calculator. I feel this app does not need one, again because the main purpose of the app was supposed to be for "managing bubble tea shop customersโ€™ membership details". It is especially unnecessary considering calculator apps are already commonplace on computers. The current calculator is also fairly lacking in utility compared to other calculators. For example, entering 2 * 2 + 3 = returns the result of 2+3 which is 5, whereas other calculators would give 7.

The time spent on implementing the calculator feature ought to have been spent on other more useful features for this app, such as being able to sort the customer list by the different fields, or saving the command history and allowing scrolling through them using the up and down keys


โ“ Issue severity

Team chose [severity.VeryLow]
Originally [severity.Low]

  • I disagree

Reason for disagreement: This issue is not a cosmetic issue as it concerns how the target user is supposed to utilise the app. This calculator serves little benefit to the user as compared to other features that could have been implemented instead using the time spent


from pe.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    ๐Ÿ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo 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.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google โค๏ธ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.