FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add MetaDetails to MediaAttachment by abraham · Pull Request #1735 · mastodon/documentation · GitHub

Add MetaDetails to MediaAttachment - #1735

Open
abraham wants to merge 1 commit into
mastodon:mainfrom
abraham:meta-details
Open

abraham wants to merge 1 commit into
mastodon:mainfrom
abraham:meta-details

Conversation

abraham commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

No description provided.

trwnh commented Oct 4, 2025

Copy link
Copy Markdown
Member

i'm not sure about this change because the meta information is serialized from whatever media processor is processing the media attachments. this information has changed significantly before. the changes are not documented, but the structure is not strictly part of the API contract, and we don't have a custom serializer for it, nor do we have a MetaDetails entity:

https://github.com/mastodon/mastodon/blob/80c8a847409c73c4e9a086c8f398a984cf5c2137/app/serializers/rest/media_attachment_serializer.rb#L48-L50

the meta information defined in the model is like so, and anything other than that is not guaranteed:

https://github.com/mastodon/mastodon/blob/80c8a847409c73c4e9a086c8f398a984cf5c2137/app/models/media_attachment.rb#L53-L58

abraham commented Oct 18, 2025

Copy link
Copy Markdown
Contributor Author

These attributes are all nullable so there is no change to the existing implicit API contract. This just makes it easier for client developers to try and use the attributes when they are available.

This branch has not been deployed

No deployments
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL