Skip to content

Conversation

kenjis
Copy link
Member

@kenjis kenjis commented Aug 12, 2024

Description
Supersedes #999
Fixes #998

The user registration form sets the default group value for the user when a new user is registered.
But the command shield:user create does not set the default group.
I think we should fix the inconsistency as a bug.

What do you think about reading the available roles from the config file and presenting that as a required choice to the user? #999 (comment)

It would be nice, but nobody has implemented it. It can be implemented as enhancement later.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

Copy link
Collaborator

@datamweb datamweb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lonnieezell say: instead of being forced into the default and then having to go to the database to change it.

In any case, I do not see any problem in the implementation.

@kenjis
Copy link
Member Author

kenjis commented Aug 12, 2024

Oh, I have not read the comment carefully. Thanks.

And @bgeneto suggested -g option instead of showing groups and the user interaction. #999 (comment)

@datamweb
Copy link
Collaborator

suggested -g option

I like this because of less user interactions.

@kenjis
Copy link
Member Author

kenjis commented Aug 12, 2024

I sent a PR to implement -g option: #1164

@kenjis kenjis merged commit 2554481 into codeigniter4:develop Aug 20, 2024
@kenjis kenjis deleted the fix-command-user-create-default-group branch August 20, 2024 04:03
@michalsn michalsn mentioned this pull request Jul 14, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: spark cli does not assign a default group to newly created user

2 participants