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

Allow using the friends plugin as an Editor #121

Merged
merged 11 commits into from
Jan 4, 2023
Merged

Conversation

akirk
Copy link
Owner

@akirk akirk commented Nov 15, 2022

Fixes #120.

Before this PR, an Administrator account was necessary to properly run the Friends plugin. This now enables Editors to run it. A requirement is that this user is declared as the main user by an admin:

Administrators

They can modify who is the main user and some blog level settings such as Comments and Post Formats.
Screenshot 2022-11-15 at 10 13 16

Main user view

They can see that they are the main user and can change most settings.

Screenshot 2023-01-03 at 17 59 22

They can also see which settings they are missing:
Screenshot 2023-01-03 at 17 48 34

Other user view

They cannot change settings.
Screenshot 2022-11-15 at 10 13 39

Unfriend

A new flow has been introduced to unfriend users:

Screenshot 2023-01-03 at 17 23 22

Friend list admin

This has been reimplemented since you cannot access it as a non-admin:
Screenshot 2023-01-03 at 17 36 12

After deletion a confirmation message is show. This screenshot also shows the link that a user with admin priviledges will see: they can also manage users on the "old" users page:
Screenshot 2023-01-03 at 17 35 33

@akirk akirk marked this pull request as ready for review January 3, 2023 17:00
@akirk akirk changed the title Allow using the friends plugin with a lower priviledge Allow using the friends plugin as an Editor Jan 3, 2023
@akirk
Copy link
Owner Author

akirk commented Jan 3, 2023

@alecmuffett this should now be ready for a beta test.

@akirk
Copy link
Owner Author

akirk commented Jan 4, 2023

I'm going to merge this already since I want to test this with other features and am confident that this is working well.

@akirk akirk merged commit 608827d into main Jan 4, 2023
@akirk akirk deleted the require-lower-priviledge branch January 4, 2023 08:11
@akirk akirk restored the require-lower-priviledge branch September 16, 2023 07:48
@akirk akirk deleted the require-lower-priviledge branch September 18, 2023 18:53
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.

Evaluate which REQUIRED_ROLE is really necessary
1 participant