theman-oj10 / pe

0 stars 0 forks source link

Allows For Duplicate articles #4

Open theman-oj10 opened 2 months ago

theman-oj10 commented 2 months ago

The app allows for duplicate articles with all the fields identical. While I agree that articles of the same name should be allowed, having articles of the same name and date not be allowed so that searching for a specific article is easy.

nus-se-script commented 2 months ago

Team's Response

This is a duplicate as only drafts are allowed to be duplicates. Hence this is the same issue.

The 'Original' Bug

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

Duplicate drafts

Screenshot 2024-04-19 at 4.35.10 PM.png

I understand that it was stated that the HEADLINE should be unique unless it is a draft however the app allows multiple duplicate drafts all with the same attributes. I think that if there is no reason as to why multiple drafts can be duplicates or each other it should share the same restriction as the other article types


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

Their Response to the 'Original' Bug

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

Users are intended to be able to keep multiple drafts of the same name. The intention behind this is that a user might be writing a few articles of a similar nature for different oulets (e.g. a tech product review for different outlets) and they are able to use their preferred headline for all these drafts. Only when the actual article is published do they need to differentiate their article headlines.

Due to the expectation that not all articles drafted get published as a freelancer, users should not be required to come up with a unique article headline each time they begin a draft. If they wish to keep a particularly punchy or impactful headline and use it for multiple drafts, there should be no imposed restriction on that until the article is published and needs a unique headline to distinguish itself.

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]