d->m: Late-loaded embeds might not appear? #29

Closed
opened 2024-01-13 10:38:10 +00:00 by cadence · 2 comments
Owner

If it's the first time an embed (URL preview) appears and Discord doesn't have it cached, Discord will take time to load the data, then send a MESSAGE_UPDATE for the original message to add the embed to it.

It seems like this doesn't work some or all of the time: https://matrix.to/#/!TqlyQmifxGUggEmdBN:cadence.moe/$O-Ra7iqqJG5jtT7Omxx26DCh8JzZk_888Ruhze4TgoU?via=cadence.moe&via=matrix.org&via=conduit.rory.gay

I know that this MESSAGE_UPDATE doesn't include many essential properties, so the first thing to do is make sure it isn't being excluded due to a missing properties guard.

If it's the first time an embed (URL preview) appears and Discord doesn't have it cached, Discord will take time to load the data, then send a MESSAGE_UPDATE for the original message to add the embed to it. It seems like this doesn't work some or all of the time: https://matrix.to/#/!TqlyQmifxGUggEmdBN:cadence.moe/$O-Ra7iqqJG5jtT7Omxx26DCh8JzZk_888Ruhze4TgoU?via=cadence.moe&via=matrix.org&via=conduit.rory.gay I know that this MESSAGE_UPDATE doesn't include many essential properties, so the first thing to do is make sure it isn't being excluded due to a missing properties guard.
cadence changed title from d->m: Embeds might not appear when the message is updated? to d->m: Late-loaded embeds might not appear? 2024-01-13 10:38:31 +00:00
Author
Owner

This was fixed in d01c888d02 and 23d85547f3

This was fixed in d01c888d02180a30ee8313122ac2f5744dd356ad and 23d85547f34b7998a4eddeebb3051aff7de1125c
Author
Owner

This had extra code added in c615ea1e61 and bce3d0f2c9 to reflect back generated embeds for messages sent by Matrix users.

This had extra code added in c615ea1e6162a1f193f29e52d277d5a01f9fecdd and bce3d0f2c9b622a9aa9d564ee69a5c833e70d51d to reflect back generated embeds for messages sent by Matrix users.
Sign in to join this conversation.
No Label
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cadence/out-of-your-element#29
No description provided.