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

Reader: "Edit post" should NOT show on sites the user does not own #94452

Closed
DustyReagan opened this issue Sep 11, 2024 · 2 comments
Closed

Reader: "Edit post" should NOT show on sites the user does not own #94452

DustyReagan opened this issue Sep 11, 2024 · 2 comments
Assignees

Comments

@DustyReagan
Copy link
Member

DustyReagan commented Sep 11, 2024

On the /discover page, in the ellipses menu, "Edit post" should not be an option on sites the user does not own.

Image

@DustyReagan DustyReagan changed the title Reader: Edit Post should NOT show on sites the user does not own Reader: "Edit post" should NOT show on sites the user does not own Sep 11, 2024
@javierarce
Copy link

I haven't been able to reproduce this 🤔

This is what I see instead:

CleanShot 2024-09-12 at 15 29 08@2x

@eoigal eoigal self-assigned this Sep 30, 2024
@eoigal
Copy link
Contributor

eoigal commented Sep 30, 2024

I can't reproduce this with an non-A8C account.

Screenshot 2024-09-30 at 20 07 52

The Edit Post option is only shown if the user has edit_post capabilities. The endpoint that figures this out is /read/streams/discover which ultimately adds the capabilities via fbhepr%2Skers%2Sjcpbz%2Sjc%2Qpbagrag%2Syvo%2Sernqre%2Qfvgr%2Qcbfg%2Spynff.jcpbz%2Qernqre%2Qfvgr%2Qcbfg.cuc%3Se%3Qs8778p0q%26sv%3Qhcqngr_qlanzvp_cbfg_svryqf%23169-og

I confirmed by sandboxing public-api.wordpress.com that the is_super_admin is set to true when logged in with A8C account. I'm going to close this as I think its intentional and not really an issue that will affect users in production.

@eoigal eoigal closed this as completed Sep 30, 2024
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

No branches or pull requests

3 participants