Skip to content

Commit be23a96

Browse files
authored
Skip recovery displays without photo url (#2995)
1 parent 80e5ab7 commit be23a96

File tree

3 files changed

+6
-21
lines changed

3 files changed

+6
-21
lines changed

app/components/page_block/homepage_top/component.rb

Lines changed: 3 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -5,31 +5,14 @@ class Component < ApplicationComponent
55
def initialize(recoveries_value:, organization_count:, recovery_displays:)
66
@recoveries_value = recoveries_value
77
@organization_count = organization_count
8-
@recovery_displays = recovery_displays
8+
@recovery_displays = recovery_displays.select { it.photo_url.present? }
99
end
1010

1111
private
1212

1313
def bike_tile_images
14-
[
15-
"redesign_2025/bike_tiles/bike-entry_0000.png",
16-
"redesign_2025/bike_tiles/bike-entry_0001.png",
17-
"redesign_2025/bike_tiles/bike-entry_0002.png",
18-
"redesign_2025/bike_tiles/bike-entry_0003.png",
19-
"redesign_2025/bike_tiles/bike-entry_0004.png",
20-
"redesign_2025/bike_tiles/bike-entry_0005.png",
21-
"redesign_2025/bike_tiles/bike-entry_0006.png",
22-
"redesign_2025/bike_tiles/bike-entry_0007.png",
23-
"redesign_2025/bike_tiles/bike-entry_0008.png",
24-
"redesign_2025/bike_tiles/bike-entry_0009.png",
25-
"redesign_2025/bike_tiles/bike-entry_0010.png",
26-
"redesign_2025/bike_tiles/bike-entry_0011.png",
27-
"redesign_2025/bike_tiles/bike-entry_0012.png",
28-
"redesign_2025/bike_tiles/bike-entry_0013.png",
29-
"redesign_2025/bike_tiles/bike-entry_0014.png",
30-
"redesign_2025/bike_tiles/bike-entry_0015.png",
31-
"redesign_2025/bike_tiles/bike-entry_0016.png"
32-
].map { image_url(it) }
14+
(0..16).map { it.to_s.rjust(2, "0") }
15+
.map { image_url("redesign_2025/bike_tiles/bike-entry_00#{it}.png") }
3316
end
3417

3518
def recoveries_value

app/javascript/controllers/admin/recovery_display_form_controller.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ export default class extends Controller {
2323
connect () {
2424
this.setCharacterCount()
2525

26-
if (this.toggleImageValue) {
26+
if (this.usingBikeImageTargets.length > 0) {
2727
this.toggleBikeImageForDisplay()
2828
}
2929
}

app/views/admin/recovery_displays/_form.html.erb

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -190,6 +190,7 @@
190190
</span>
191191

192192
<span
193+
class="tw:hidden"
193194
data-admin--recovery-display-form-target="notUsingBikeImage"
194195
>
195196
<%= @bike.type_titleize %>
@@ -208,6 +209,7 @@
208209
href="#"
209210
>
210211
<span
212+
class="tw:hidden"
211213
data-admin--recovery-display-form-target="notUsingBikeImage"
212214
>
213215
Use first image

0 commit comments

Comments
 (0)