Skip to content
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

Add support for updating media description and focus point when editing statuses #3215

Merged
merged 2 commits into from Feb 14, 2023

Conversation

Tak
Copy link
Collaborator

@Tak Tak commented Jan 25, 2023

Completes #3211

We could also add support for viewing them in the edit history view, but I think that will maybe not be super useful since that view is mainly for seeing the history of other people's statuses

Also fixes #3273

@mcclure
Copy link
Collaborator

mcclure commented Jan 26, 2023

This is currently not even exposed on Mastodon Web. Does this feature exist in the API currently?

I would LOVE this if it's possible because currently if I forget a caption I can't add one after-the-fact.

@Tak
Copy link
Collaborator Author

Tak commented Jan 26, 2023

Yeah, it's in 4.1.0rc1 (see the mastodon PR linked from the linked issue) - I updated my glitch server to a corresponding version to work on this

@sl1txdvd
Copy link

I tried this with my Mastodon 4.1.0-rc1 instance and it seems to work. However it shows a message saying there was an error when saving the description ("Fehler beim Speichern der Beschreibung"). As far as I can tell there is no actual error.

@Tak
Copy link
Collaborator Author

Tak commented Jan 28, 2023

That message means that the server reported an error when Tusky sent the update.
Can you describe what you did to get the error?
I've tested:

  • Updating the description on an image
  • Updating the focus point on an image
  • Updating both description and focus point on an image
  • Updating the description on a video

@sl1txdvd
Copy link

-Start new post, enter text, add image, press "Toot"
-Edit post:
--"Set caption", enter text, press "ok": error says "Failed to set caption"
-"Set focus point", move circle, press "ok": error says "Failed to set focus point"
-Press "Toot"
-Reload post: caption and focus point are the way I set them despite the error messages

Maybe I will find something in the server logs.

@Tak
Copy link
Collaborator Author

Tak commented Jan 29, 2023

Ohh, the message is when setting them locally, that gives me something to look into

@Tak
Copy link
Collaborator Author

Tak commented Feb 1, 2023

Ok, Tusky was attempting to update the media attributes via the normal path as soon as they were accepted, but the server doesn't like that when the post toot has already been published tooted.
I've added a fix, thanks!

@sl1txdvd
Copy link

sl1txdvd commented Feb 1, 2023

Works nicely now, thanks!

@mcclure
Copy link
Collaborator

mcclure commented Feb 12, 2023

Tested 49d5a89 on mastodon.social now that it is on v4.1.0rc3

Can confirm it fixes #3273 and fixes the inappropriate "Edit Image" offer. Both focus change and caption change work.

@mcclure
Copy link
Collaborator

mcclure commented Feb 12, 2023

…even weirder, Tusky 49d5a89 is able to edit focus point on images even though Mastodon Web itself cannot

@connyduck connyduck merged commit 395e21c into tuskyapp:develop Feb 14, 2023
@Tak Tak mentioned this pull request Mar 8, 2023
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.

Can't edit caption during "delete and re-draft" anymore
4 participants