quotable icon indicating copy to clipboard operation
quotable copied to clipboard

Get Quotes by topic / genre

Open shankarmadeshvaran opened this issue 2 years ago • 9 comments

Feature

For Each Quotes, add genre as parameter and we can filter based on it. I can see that tag is also used for same purpose but not exactly the same.

shankarmadeshvaran avatar Jun 24 '22 07:06 shankarmadeshvaran

This is exactly what the tags feature was intended for. It probably should have been called "topics" or "categories" instead of tags. I would be in favor of renaming this parameter "topics".

However, the real problem is: for this feature to be useful, we need to add meaningful topics to all of the existing quotes. I spent quite a bit of time working on this. But there are still ~ 1,200 quotes that do not have topics/tags.

If anyone is interested in working on that -- adding topics to the existing quotes let me know.

lukePeavey avatar Jun 25 '22 00:06 lukePeavey

This is exactly what the tags feature was intended for. It probably should have been called "topics" or "categories" instead of tags. I would be in favor of renaming this parameter "topics".

However, the real problem is: for this feature to be useful, we need to add meaningful topics to all of the existing quotes. I spent quite a bit of time working on this. But there are still ~ 1,200 quotes that do not have topics/tags.

If anyone is interested in working on that -- adding topics to the existing quotes let me know.

Sign me up 🥳

Anish-Chanda avatar Jun 25 '22 03:06 Anish-Chanda

@Gal1leo03

If anyone is interested in working on that (adding topics to the existing quotes) let me know.

Sign me up 🥳

Great! The data is in a public repository, anyone is welcome to contribute. It consists of JSON files that contain the source data for each database collection (quotes, authors, tags). To contribute:

  1. Fork and clone the data repository
  2. Create a new feature branch (ie adding-tags)
  3. Make changes to the data files
  4. Open a pull request to submit changes.

The only way to add tags to existing quotes is by manually editing the data/quotes.json file. Each quote has a tags property. So you would go though and manually add new tags to each quote. This is a pretty slow process, but any help would be appreciated!

  • Feel free to create new topics/tags
  • the tags should be kebab-case
  • You don't need to add new tags to the tags.json collection. That will get updated automatically.
  • To find quotes that don't have topics/tags yet, you can search for the following values in data/quotes.json.
"tags": []
"tags": ["famous-quotes"]

lukePeavey avatar Jun 25 '22 05:06 lukePeavey

Is this still open? Do you need me to get on in this one? @lukePeavey

raxraj avatar Jul 04 '22 11:07 raxraj

I've created a pull request after changing the tags of around 300 quotes, My exams were going on so my schedule was pretty filled up, so I am really really sorry for the delay :) @lukePeavey

Anish-Chanda avatar Jul 04 '22 13:07 Anish-Chanda

@raxraj which part did you want to work on, adding tags to quotes or renaming the tags param?

lukePeavey avatar Jul 04 '22 14:07 lukePeavey

@Gal1leo03 no need to apologize! Thanks for working on that. I'll merge your PR today.

lukePeavey avatar Jul 04 '22 14:07 lukePeavey

@lukePeavey Has anything happened related to this issue?

a2937 avatar Oct 28 '23 00:10 a2937

@a2937

The api already uses "tags" to categorize and filter quotes. The tags feature is essentially the same concept as "topics" or "categories". So I don't think we need to add a new feature for this. If you want to add /edit tags on existing quotes you can do so via the data repository

lukePeavey avatar Oct 28 '23 13:10 lukePeavey