Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get Quotes by topic / genre #135

Open
shankarmadeshvaran opened this issue Jun 24, 2022 · 9 comments
Open

Get Quotes by topic / genre #135

shankarmadeshvaran opened this issue Jun 24, 2022 · 9 comments
Labels
⚡ type: enhancement An issue or pull request that aims to improve an existing feature

Comments

@shankarmadeshvaran
Copy link

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 shankarmadeshvaran added the 💡 type: feature An issue or pull request relating a new feature label Jun 24, 2022
@lukePeavey
Copy link
Owner

lukePeavey commented Jun 25, 2022

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.

@Anish-Chanda
Copy link

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 🥳

@lukePeavey
Copy link
Owner

lukePeavey commented Jun 25, 2022

@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 lukePeavey changed the title Get Quoted by Genre(Eg: Motivation, Hardwork, Consistency, Money, Football, arts, etc.,) Get Quotes by topic / genre Jun 26, 2022
@lukePeavey lukePeavey added ⚡ type: enhancement An issue or pull request that aims to improve an existing feature and removed 💡 type: feature An issue or pull request relating a new feature labels Jun 27, 2022
@raxraj
Copy link
Contributor

raxraj commented Jul 4, 2022

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

@Anish-Chanda
Copy link

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

@lukePeavey
Copy link
Owner

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

@lukePeavey
Copy link
Owner

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

@a2937
Copy link

a2937 commented Oct 28, 2023

@lukePeavey Has anything happened related to this issue?

@lukePeavey
Copy link
Owner

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚡ type: enhancement An issue or pull request that aims to improve an existing feature
Projects
None yet
Development

No branches or pull requests

5 participants