osem icon indicating copy to clipboard operation
osem copied to clipboard

Users can not be confirmed

Open Ana06 opened this issue 5 years ago • 4 comments

users#toggle_confirmation threw this 0 times

undefined method `confirm' for #<User:0x0000000005f6d2b8>
Did you mean?  confirmed?

https://events.opensuse.org/admin/users/12016/toggle_confirmation?user[to_confirm]=true"

See this exception on Errbit

Ana06 avatar Apr 26 '19 11:04 Ana06

Hmmm you cannot confirm a user if you are using ICHAIN - would it make any sense to do so? Why do people want to confirm users while using ICHAIN?

This is caused by the lack of confirmable in Devise when using ICHAIN https://github.com/openSUSE/osem/blob/master/app/models/user.rb#L48

differentreality avatar Apr 29 '19 18:04 differentreality

@differentreality

Hmmm you cannot confirm a user if you are using ICHAIN - would it make any sense to do so? Why do people want to confirm users while using ICHAIN?

You are right, no. But there are already confirmed and unconfirmed users and this raises an exception.

Ana06 avatar Apr 30 '19 09:04 Ana06

The confirmed users exist since before ICHAIN was implemented in OSEM, when users actually created their accounts in OSEM, as such OSEM would confirm their email address.

So what are we expecting to be done for this? Not showing the confirm button in the admin panel?

differentreality avatar Apr 30 '19 10:04 differentreality

@differentreality

So what are we expecting to be done for this? Not showing the confirm button in the admin panel?

Not showing the button and that all the users are confirmed. We have some unconfirmed users in events.opensuse.org

Ana06 avatar May 02 '19 11:05 Ana06