whitescent / Mastify

A Mastodon client built with Jetpack Compose [WIP]
https://play.google.com/store/apps/details?id=com.github.whitescent.mastify
GNU General Public License v3.0
155 stars 8 forks source link

fix(api): improve firefish support #102

Closed lazzzis closed 2 months ago

lazzzis commented 3 months ago

Fix following error: Screenshot_20240327_174415

Most of the APIs are identical, except the meta field.

Mastodon API:

meta: {
  original: {
    width: 720,
    height: 256
  }
}

Firefish API

meta: {
  width: 720,
  height: 256
}
naskya commented 2 months ago

Firefish maintainer here.

Thank you for catching the issue, and we’re sorry for bothering you all with the incompatibility. Firefish’s Mastodon API support is currently very spotty. We don’t think Mastodon client developers should be bothered by that, and we’re going to fix this issue on our side. (it’s totally up to Mastify maintainers to accept this request as a temporary workaround, though)

whitescent commented 2 months ago

@lazzzis I'm still thinking about serialization processing that is compatible with different instances. from a long-term perspective, some instances have different backend APIs, so it may be too early to only make compatibility with Firefish's timeline api.

lazzzis commented 2 months ago

SG! Glad to hear this will be fixed in the server side instead. Close this now that the maintainer has already been working on serialization processing.