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

Add configurable list of tags to ignore #58

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

G1org1owo
Copy link
Contributor

Should close #48

This should be pretty easy to implement, I'm drafting this mainly because of style issues.
As of now I'm recycling the TagDetail model but I'm only using the first name, it'd be nice to allow for aliases when ignoring but it could get a bit cluttered and I'm pretty sure it wouldn't be possible with simple bindings; on the other hand, it might make more sense to create a separate model (or use simple strings) for the names to be saved in the config.

Also I kinda suck at front-end so if this ends up going through I'm gonna ask you to smooth the rough edges 🙏

@G1org1owo G1org1owo marked this pull request as ready for review July 11, 2024 19:35
@G1org1owo
Copy link
Contributor Author

I can't seem to get a simple string array to work with DataTable bindings so I think I'll just leave the model as it is for now

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.

Feature: Configurable list of tags to always ignore (e.g. tagme, highres)
1 participant