-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Media: Hide edit button on private sites #17347
Conversation
It looks like this will affect VideoPress videos as well. I'm guessing the same error applies in that scenario? Also, if I've previously edited an image, I still see the "Restore Original" button although the "Edit Image" button is indeed removed. The image does restore when the button is clicked, so maybe it's fine to still show it: |
That's correct!
We want to update this eventually but it does require some endpoint work. At the moment restore, adds another copy of the original as the latest edit instead of pointing at the original image. In this case the client doesn't have enough information to detect an, edit -> restore case. |
@retrofox do you recall if something changed recently? I thought we checked for this before. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jeremeylduvall this tests pretty well for me, unless other folks had feedback feel free to
I'm not aware of any recent change, Kerry. :-( |
@jeremeylduvall did you need help landing this one? |
@gwwar no - sorry! Will get this landed first thing tomorrow morning. |
This is a temporary workaround for #13625 so users no longer see the edit button in Calypso on private sites since they can't edit a media item at the moment. It prevents confusion because clicking the edit button currently on a private site gives a weird error.
74c6a9a
to
fe40f4e
Compare
This is a temporary workaround to hide the Edit button on media items on private sites. It's related to #13625, which has received a few user reports. Right now, if you try to edit a media item in Calypso on a private site, you receive this error:
It's not clear why the edit action didn't work since you can edit media items in wp-admin.
My understanding (from this comment) is that this is a CORS limitation at the moment.
To test
You should not see an edit button on the image. This should only affect private sites on WordPress.com.
Before
After