Code Monkey home page Code Monkey logo

Comments (1)

soc-pe-bot avatar soc-pe-bot commented on July 19, 2024

Team's Response

add workout is to record workouts(mainly completed ones), with the calories burned.

add schedule is simply to plan/schedule workouts, having an idea of your workout plan in the future.

Although not the conventional usage, add workout retains the ability to record workouts in the future as some users might find some use for it. For example, some users might know they are doing a workout the next morning, they would find it useful to record a workout(with its calories) in advance, they know they will be doing in case they forget to record it.

Basically this is an implementation choice since it allows for user flexibility IF they want to record stuff in advance.

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: With the functionalities of the 2 commands overlapping, I find this to be very confusing for the user. Would it not be better to just then have only 1 workout command that can add both past and future workouts? Then have a function to mark a workout as completed, because sometimes even if users add a workout to the future he may eventually not do it (hence the calories don't count). Else if schedule workouts is meant to be for future workouts, the regular add workout command (which requires the user to input calories burned, is to record workouts that are completed so as to count calories) should not be allowed to add future workouts right? Then similarly have a function to make a workout from the list of schedule workouts as completed on the day that the workout was scheduled.


❓ Issue severity

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

  • I disagree

Reason for disagreement: The implementation is honestly quite confusing for me due to the overlapping command usage (and needing to handle 2 separate lists of workouts and schedule workouts yet without a clear distinction between the 2 commands). I believe this causes more than just a minor inconvenience that happens very rarely.


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.