Make sure articles are unique

Question

Make sure articles are unique

Closed this issue 9 years ago · 6 comments

bartaelterman commented 9 years ago

Option 1: Set a unique constraint on the articles url. Problem: articles scraped before 2013 don't have a url. (see also #53)
Option 2: Set a unique constraint on a combination of the fields news_journal, published_at and title

Answer 1 · 2015-07-27T13:22:23.000Z

Urls are not an option (see #53 ) so we'll have to go for option 2.

@niconoe can we add such a constraint in the models file?

Answer 2 · 2015-07-27T14:23:52.000Z

Yes, it can be implemented using unique_together, and Django will create constraint at the database level. Does that looks good?

Answer 3 · 2015-07-27T14:24:24.000Z

Excellent!

Answer 4 · 2015-07-27T14:35:52.000Z

Since URLs are still mandatory, I cannot test if articles without URL are displayed nicely on the front end. Don't close this issue until this has been tested.

Answer 5 · 2015-08-03T10:00:39.000Z

The URL is now optional, and unique_together constraint is set (everything deployed to Heroku). I guess this issue + #53 + #63 are now solved, please test!

Answer 6 · 2015-08-03T12:40:55.000Z

Frontend works: only shows newspaper name, without link if no URL is available.