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

AO3-6541 Update Works search and filters to use "Creator" instead of "Author/Artist" #4553

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

Conversation

forceofcalm
Copy link
Contributor

@forceofcalm forceofcalm commented Jun 21, 2023

Pull Request Checklist

Issue

This PR is for AO3-6541, which requested an update of the terminology used on the search forms. This update switches "Author/Artist" to "Creator".

Purpose

This PR updates instances of "Author/Artist" to "Creator" on the work search forms.

Credit

calm, they/them

["Word Count", "word_count"],
["Hits", "hits"],
["Kudos", "kudos_count"],
["Comments", "comments_count"],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wondering if I should actually do this, since we have instances of whitespace in "Date Posted", "Date Updated", etc. so it wouldn't be consistent on each? I'm a Ruby noob, welcome to other thoughts here.

spec/models/search/work_query_spec.rb Outdated Show resolved Hide resolved
spec/models/search/work_search_form_spec.rb Outdated Show resolved Hide resolved
spec/models/search/work_search_form_spec.rb Outdated Show resolved Hide resolved
@@ -150,7 +150,7 @@ def meta
# just means Calibre's GUI treats it as one name, e.g. "testy, testy2" is
# like "Fangirl, Suzy Q", for searching and sorting.
authors: download.authors,
sortable_authors: work.authors_to_sort_on,
sortable_authors: work.creator_to_sort_on,
Copy link

Choose a reason for hiding this comment

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

Layout/HashAlignment: Align the keys of a hash literal if they span more than one line.

@forceofcalm forceofcalm marked this pull request as draft June 27, 2023 02:41
@forceofcalm forceofcalm marked this pull request as ready for review March 1, 2024 15:01
Copy link
Contributor

@weeklies weeklies left a comment

Choose a reason for hiding this comment

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

I believe the issue is asking only for a frontend change. If you undo the backend changes made and keep the visual edits, this should make the tests pass.

@brianjaustin
Copy link
Member

Hi @forceofcalm, what @weeklies said is correct

I believe the issue is asking only for a frontend change.

We'd prefer to keep the changes small to avoid unintended consequences (like potentially breaking searches people have bookmarked). There's precedent for this with history (previously named readings) or username/login/user name (all of which are used in various places)

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.

5 participants