-
Notifications
You must be signed in to change notification settings - Fork 104
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
'emailConfirmation' => 'false', #124
Comments
What do you mean makes no difference - what doesn't work exactly? |
Just tried it on my dev machine and it's working fine. Gonna need more info from you. |
Hi, thanks for looking at this. 'modules' => [
|
Yes, it's supposed to. Your config looks correct, so I'm not really sure what's going on. What classes have you extended/functions did you override? Just to make sure, did you set the user component properly? // config/web.php
'components' => [
'user' => [
'class' => 'amnah\yii2\user\components\User',
],
] |
Yes I have the user component set properly. The only functions I have overridden were to change the subject of emails sent. I have just commented out the lines in my config file so it doesn't use my version of those classes and I still have the problem. |
Ok then I have no idea. You'll have to walk through the code and try to see what's happening, I can't do much but speculate from here as I cannot reproduce this. The first part is setting the user attributes, which sets the https://github.com/amnah/yii2-user/blob/master/models/User.php#L345 Then it checks what that status is, and it sends an email if needed. ``DefaultController::afterRegister()` https://github.com/amnah/yii2-user/blob/master/controllers/DefaultController.php#L252 Can you walk through/print_r and see what's happening? |
When I change this setting it makes no difference. I want users to be able to sign up and login to my site straight away.
/**
* @var bool If true, users will have to confirm their email address after registering (= email activation)
*/
public $emailConfirmation = true;
The text was updated successfully, but these errors were encountered: