Skip glitched appearances in bulk-labeling mode

It's not as important to label glitched states, and sometimes the glitch
prevents it from being visually identifiable. Don't sweat 'em!
This commit is contained in:
Emi Matchu 2024-12-01 10:39:19 -08:00
parent 407c4b38a5
commit d9bf4f745b
2 changed files with 2 additions and 1 deletions

View file

@ -38,7 +38,7 @@ class PetStatesController < ApplicationController
# Rather than just getting the newest unlabeled pet state, prioritize the
# newest *pet type*. This better matches the user's perception of what the
# newest state is, because the Rainbow Pool UI is grouped by pet type!
unlabeled_appearance = PetState.unlabeled.newest_pet_type.newest.first
unlabeled_appearance = PetState.needs_labeling.newest_pet_type.newest.first
if unlabeled_appearance
edit_pet_type_pet_state_path(

View file

@ -20,6 +20,7 @@ class PetState < ApplicationRecord
scope :newest, -> { order(created_at: :desc) }
scope :newest_pet_type, -> { joins(:pet_type).merge(PetType.newest) }
scope :unlabeled, -> { with_pose("UNKNOWN") }
scope :needs_labeling, -> { unlabeled.where(glitched: false) }
# A simple ordering that tries to bring reliable pet states to the front.
scope :emotion_order, -> {