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

Addition of user commands #313

Closed
nico8948 opened this issue Jul 20, 2023 · 4 comments · Fixed by #316
Closed

Addition of user commands #313

nico8948 opened this issue Jul 20, 2023 · 4 comments · Fixed by #316
Assignees

Comments

@nico8948
Copy link
Contributor

nico8948 commented Jul 20, 2023

I don't how to comment the project, but i've added some user commands, like in brush.
i needed these options myself..

see my fork
https://github.com/nico8948/bee

i don't know if it's usefull

Commands:

  • user-create
  • user-cancel
  • user-block
  • user-unblock
  • user-add-role
  • user-remove-role
@yorkshire-pudding yorkshire-pudding changed the title added some user commands in a fork Addition of user commands Jul 20, 2023
@yorkshire-pudding

This comment was marked as resolved.

@yorkshire-pudding
Copy link
Collaborator

Hi @nico8948 - thanks for your patience with this.

I've committed all the minor typos, wording improvements and phpcs corrections. What is left is:

  • removing the code that adds a role if it doesn't match - see my comment here
  • Adding tests. Please let me know if you would prefer if I added these or whether you are happy to have a go.

@nico8948
Copy link
Contributor Author

nico8948 commented Jul 29, 2023 via email

@yorkshire-pudding
Copy link
Collaborator

@nico8948 🎉 - great work in getting to a state with all tests passing. I'm going to download this version, give it a thorough review and test, and hopefully we can get this merged

yorkshire-pudding added a commit to yorkshire-pudding/bee that referenced this issue Jul 30, 2024
bugfolder pushed a commit to bugfolder/bee that referenced this issue Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants