~ knotless Search/
↑↓ move↵ open⇧↵ all results

grain.social/grain-android

1 collaborator
0
1

Photos post on PDSes that still require alt

Merged chadtmiller.com asked to merge into main from grain.social/grain-android:fix/photo-alt-always
app/src/main/java/social/grain/data/upload/GalleryDraft.kt+3−1
@@ -97,7 +97,9 @@ data class GalleryDraft(
put("height", photo.height)
})
put("createdAt", createdAt)
photo.alt.trim().takeIf { it.isNotEmpty() }?.let { put("alt", it) }
// Always sent: the PDS validates against its own compiled-in
// copy of this schema, which requires alt even when it is empty.
put("alt", photo.alt.trim())
},
)
)
app/src/test/java/social/grain/GalleryUploadTest.kt+3−0
@@ -162,6 +162,9 @@ class GalleryUploadTest {
val firstPhoto = writes.first { it.jsonObject["collection"]!!.jsonPrimitive.content == "social.grain.photo" }
assertEquals("photo-p1", firstPhoto.jsonObject["rkey"]!!.jsonPrimitive.content)
assertEquals("A tree", firstPhoto.jsonObject["value"]!!.jsonObject["alt"]!!.jsonPrimitive.content)
val secondPhoto = writes.last { it.jsonObject["collection"]!!.jsonPrimitive.content == "social.grain.photo" }
assertEquals("", secondPhoto.jsonObject["value"]!!.jsonObject["alt"]!!.jsonPrimitive.content)
}
@Test