Skip to content

AO3-7293 Add unactioned and unwrangleable status filters to tag search#5588

Open
marcus8448 wants to merge 4 commits intootwcode:masterfrom
marcus8448:AO3-7293
Open

AO3-7293 Add unactioned and unwrangleable status filters to tag search#5588
marcus8448 wants to merge 4 commits intootwcode:masterfrom
marcus8448:AO3-7293

Conversation

@marcus8448
Copy link
Member

Issue

https://otwarchive.atlassian.net/browse/AO3-7293

Purpose

Adds two more wrangling status options to tag search: "Unwrangleable" and "Non-canonical and non-synonymous and not marked unwrangleable" (which are tags that have not been actioned in some way).

Credit

marcus8448 (he/him)

It doesn't make sense to search for both separately (e.g. can't be canon + unwrangleable), and status=unwrangleable should cover it.
It appears to be unused (and was not exposed to tag search).
end

def unwrangleable_filter
term_filter(:unwrangleable, bool_value(options[:unwrangleable])) unless options[:unwrangleable].nil?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am surprised this removal doesn't cause test failures, isn't this option used in tag_counts_per_category, unwrangled_query, TagWranglingsController#index and even the tag_query spec?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welp, I searched for :unwrangleable and not unwrangleable:... probably should've just searched all TagQuerys.

Tests pass because there's a typo:

expect(results).not_to include(tags[:unwrangeable])

I'll restore the filter and fix the test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants