search

ezeAng / pe

0 stars 0 forks source link

From the DG, it is unclear of the units of the rate. #15

Open ezeAng opened 1 year ago

ezeAng commented 1 year ago

Could be confusing to a user if the rate unit is not specified. Different negotiations have different rates, maybe all the rates are to be converted to per hour dollar.

nus-pe-script commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

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

The term "rate" is misused

Description

(Apologies and excuse this issue if my business terminology is wrong)

The term "rates" would typically refer to the price charged per unit of time. When I first encountered this, my assumption was that revenue would calculate the revenue based on hours of an event * rate. (e.g if an event is 3 hours, and the rate is $10, then I expected the revenue to be 3 * 10 = $30).

Instead, revenue seems to treat rate as the total cost of an event.

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

Their Response to the 'Original' Bug

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

The rates of an event is not only limited to just per unit time. It can also be used as the amount of money charged for working for the particular event. For example, a freelancer can charge $500 for a photoshoot event which may span across varying amount of hours and days. Our interpretation of the rates is simply how much a freelancer can earn by taking up this event which is usually the case as freelancers ourselves.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.DocumentationBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]