From 6a0afb330b8f1144820fee939087d9140d6f82a9 Mon Sep 17 00:00:00 2001 From: Emi Matchu Date: Tue, 27 Feb 2024 18:16:23 -0800 Subject: [PATCH] Add warning for "Baby Body Paint" bugs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit I *think* what I'm observing is that: 1. The zone restrictions are different between these items. 2. The zone restrictions *change* when reloading the page sometimes. (I assume from remodeling?) 3. The items look very buggy on many pets, because many appearances seem to expect different zone restrictions than the item actually has. I think what this means is: 1. TNT has finally unbound restricted zones from the item level, and allowed different appearances to have different restrictions. Neat! 2. The API still serves it the same way, as a field on the item. So I think this means we need to update our schema to reflect the fact that an item's `zones_restrict` field isn't *really* a property of the item; it's a property of the combination of the item and the current body ID. My gut take here is that maybe this means it's time for the Large Refactor that I've kinda been interested in for a while, but been avoiding because of Impress 2020 compatibility issues: instead of a `body_id` field on assets, and having them directly belong to items, make an `ItemAppearance` record (closer to how 2020's GQL API modeled it, I was looking ahead to this possibility!) that's keyed on item and body ID, and assets belong to *that*. Then, we could move the zones restriction field onto the `ItemAppearance` record instead. And then it doesn't really matter to us how TNT models it internally; whatever we saw is what we use. (Again, I looked ahead to this in the 2020 app, and tried to use the `restrictedZones` field on `ItemAppearance` when possible—even though it secretly just reads directly from the `Item`!) …but that's a pretty big departure from how things are modeled now, and isn't something we can just throw together—especially coordinating it across both apps. I was getting close to being able to shut off 2020 from a *front-facing* perspective (but still keeping a lot of the GQL endpoints open for the wardrobe-2020 frontend), but I don't think we're very close to being able to try to target turning off 2020's *backend* as a prereq to this; or at least, if we do, we should expect that to take a while. (Counting now, there's still 9 GQL queries—not as many as I expected tbh, but still quite a few.) So idk how to sequence this! But for now, let's put out a warning, and start setting expectations. --- app/views/items/show.html.haml | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/app/views/items/show.html.haml b/app/views/items/show.html.haml index 6a011b70..87468289 100644 --- a/app/views/items/show.html.haml +++ b/app/views/items/show.html.haml @@ -6,6 +6,13 @@ current_user_lists: @current_user_lists, current_user_quantities: @current_user_quantities} +- if @item.name.include? "Baby Body Paint" + %p.warning + The Baby Body Paint items seem to have new zone restriction rules that our + system doesn't support yet, whuh oh! This might require major changes to + how we handle zones. Until then, these items will be very buggy, + sorry! + #outfit-preview-root{'data-item-id': @item.id} - unless @contributors_with_counts.empty?