delishad21 / pe

0 stars 0 forks source link

Mailing function is limited #12

Open delishad21 opened 4 months ago

delishad21 commented 4 months ago

Being only able to mail people based on their tags is a huge inconvenience to the User. In a large tech firm, there is a good chance that emails are to be sent out based on their department.

nus-pe-bot commented 4 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Cannot mail or phone department

I think this is an overlook, as we should be able to email and phone by departments as well.


[original: nus-cs2103-AY2324S2/pe-interim#498] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

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

This is not important to the current implementation. This can be added in a future implementation.

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`] - [x] I disagree **Reason for disagreement:** The team's value proposition states that the purpose of the application is to > Categorise people according to their department/origin and provide methods for efficient retrieval and manipulation of the information They have based the value of their application around the ability to categorize people by their department and to manipulate information around the categorization. This means that new functionality added into the application should **at the very least** be compatible with the department categorization. For something that should have been at the top of their priority list, it is outrageous to claim that this issue is unimportant and not within the scope of their current iteration.
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** The team has provided no elaboration for the downgrading of severity. The lack of implementation for this feature severely limits the functionality of the application. There is a higher likelihood that users would send emails to people based on their department, especially considering that one of the application's main functionality and the value proposition of the application was to categorize the people by their department. The team is also likely in agreement with the importance of this functionality as they have included it as part of their user stories (even though the feature was not even implemented). ![image.png](https://raw.githubusercontent.com/delishad21/pe/main/files/d8436e29-7b9b-4999-9f7e-801ab56df6d7.png) The level of inconvenience caused for users in missing a key functionality is at least deserving of a `severity.Medium` label