chinwag/chinwagsocial
chinwag/chinwagsocial
2
0
Fork 0
Code Issues Pull requests Releases 4 Wiki Activity

Move direct serializer usage out of admin view partial (#29028)

Browse source
This commit is contained in:
Matt Jankowski 2024-02-06 08:18:37 -05:00 committed by GitHub
parent 2d6ab44556
commit 69e61fff38
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
app/helpers/react_component_helper.rb
View file

@ -15,9 +15,20 @@ module ReactComponentHelper
div_tag_with_data(data)
end
def serialized_media_attachments(media_attachments)
media_attachments.map { |attachment| serialized_attachment(attachment) }
end
private
def div_tag_with_data(data)
content_tag(:div, nil, data: data)
end
def serialized_attachment(attachment)
ActiveModelSerializers::SerializableResource.new(
attachment,
serializer: REST::MediaAttachmentSerializer
).as_json
end
end

2
app/views/admin/reports/_media_attachments.html.haml
View file

@ -12,6 +12,6 @@
= react_component :media_gallery,
height: 343,
lang: status.language,
media: status.ordered_media_attachments.map { |a| ActiveModelSerializers::SerializableResource.new(a, serializer: REST::MediaAttachmentSerializer).as_json },
media: serialized_media_attachments(status.ordered_media_attachments),
sensitive: status.sensitive?,
visible: false