-
Notifications
You must be signed in to change notification settings - Fork 6
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
Allow exporting of user feedback #270
Comments
This came up again recently, so started taking a look at it - it isn't super straightforward to rescind a restriction (exporting) based on a user type, and there are a few potential cases we would want to protect against. |
@dd32 the idea that came to mind was using it has the disadvantage of showing the global export sub menu item whilst on this page (you get a restricted screen when clicking on it), but couldn't figure out an alternative.
|
@pkevan We can probably just allow Anything that they can see in the admin UI should be able to be exported; I don't have the historical knowledge of why we don't have export available, but I'm assuming it was for PII information or that it allowed access to the raw post_content of something that some were not allowed to access. That being said, I'm fairly sure Jetpack has it's own Export functionality for feedback? Or is that covered by this cap too? |
It does, but Jetpack only allows users with the |
Yes, the caps and code here covers the functionality within Jetpack. Thinking about this some more, i'll probably put this in the code within mu-plugins where the caps are restricted initially to avoid any potential confusion in the future. |
Yes, plus probably exporting user data, which you would get if granted export access. |
Unsure if this matters, but for the data we currently strip out manually, there isn't really any way to remove this. |
By default on the wp.org network exporting is limited to super admin users only.
We should allow Administrators that right, in a limited way, probably just for user feedback.
Whether we do this via
map_meta_cap
or changes to the Jetpack plugin, that would enable exporting of data for marketing purposes.The text was updated successfully, but these errors were encountered: