-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Regression: Suggestion.source is documented as a string, currently returns list of strings #29071
Comments
I believe this was added recently in main (see mastodon/documentation#1398 for more, and #28314 for the PR itself), but yes, it does represent a breaking API change. I'd like to see this reworked to be backwards-compatible, or otherwise use an /api/v2 method prefix. |
Noted and I agree that a change like this which is visible via nightly on mastodon.social and mastodon.online should not be causing disruption -> we should look into ways to either make it backwards compatible, or move to a different prefix until a formal relase is made. Will follow up with the rest of the team here. |
…ons` back to a string The new array is now in the `sources` attribute. Fixes #29071
Steps to reproduce the problem
mastodon.social
, call/api/v2/suggestions
Suggestion
objectsExpected behaviour
Suggestion.source
should be a stringActual behaviour
Suggestion.source
is an array of stringsDetailed description
I'm not sure when this changed, but it's not documented, or backwards compatible with existing clients. My suggestion for the fix would be to keep
source
as a string, populate it with the first or most important of the possible source strings, deprecatesource
, and return the full list of sources as a newsources
property onSuggestion
.Mastodon instance
mastodon.social
Mastodon version
v4.3.0-nightly.2024-02-02
Technical details
No response
The text was updated successfully, but these errors were encountered: