search

open-y-subprojects / openy_map

Map feature from Open Y DIstribution
GNU General Public License v3.0
1 stars 14 forks source link

fix: ensure y_branch is present and enabled #56

Closed andriokha closed 10 months ago

andriokha commented 12 months ago

It looks like openy_map started requiring y_branch at some point.

froboy commented 11 months ago

@andriokha thanks! Can you point to the error or issue that initially spurred this so we can get it properly documented?

andriokha commented 11 months ago

Hi, just a quick note I'm on vacation and won't be able to respond much.

Can you point to the error or issue that initially spurred this so we can get it properly documented?

I don't remember the precise details but it caused an issue during an upgrade to the D10 version of YMCA WS. It looks like y_branch is only used in openy_map_lb_preprocess_node__location_type__lb_teaser() so I think I would've seen a fatal about a non-existent y_branch.hours_helper service when viewing a node using that suggestion (presumably via the map).