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

feat: Pluralize and format tags #947

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

sweepies
Copy link
Contributor

This PR mainlines a couple of tagging rules I added to my personal instance:

  • Tags should be converted to Title Case.
  • If applicable, tags should be plural. For example, 'recipes' instead of 'recipe'.

These have worked great for me.

@kamtschatka
Copy link
Contributor

there are people who have the exact opposite and want lower case tags only. You can add those rules locally with the custom tagging rules, so this is where they should probably stay.

@sweepies
Copy link
Contributor Author

sweepies commented Jan 30, 2025

It's opinionated indeed but I think we need a default of some sort. The default behaviour should not open up potential for four different tags of the same subject. Too messy IMO.

i.e. recipe/recipes/Recipe/Recipes

Maybe putting these in the system prompt isn't the solution. Perhaps a better place for them would be a couple pre-filled but editable fields?

@MohamedBassem
Copy link
Collaborator

I think changing those rules now can be a bit disruptive. How about we start a new docs page where people can share some of their custom prompts?

@MohamedBassem
Copy link
Collaborator

We can even potentially bake this custom rules library in hoarder itself. A modal that opens with one row per rule, and a button to "Add" it to your rules.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants