fefrei
June 17, 2016, 4:57pm
1
I’m using incoming mail support for group messaging with staged users.
This works fine if the address the message is coming from is new. However, I am re-purposing a Discourse installation for a yearly event , and some old users are stuck with an inactive accounts they can no longer log in to.
When such a user sends a mail to Discourse, they are rejected with Email::Receiver::InactiveUserError.
Any thoughts here @zogstrip ?
zogstrip
June 20, 2016, 3:16pm
3
The InactiveUserError happens only when the user is not activated and not staged. If you want your users to be able to send emails in, you will have to either activate their account or mark them as staged.
fefrei
June 20, 2016, 6:56pm
4
Can I mark existing, inactive users as staged? Would that have any side effects?
zogstrip
June 20, 2016, 7:37pm
5
Other than enabling them to send emails in, none
fefrei
June 21, 2016, 4:59am
6
How can I do that, then? The admin UI does not offer this option.
Will they return to normal if they ever log in?
zogstrip
June 21, 2016, 8:21am
7
fefrei:
How can I do that, then?
You’ll have to use the rails console or run a SQL query.
Yes.
fefrei
June 21, 2016, 9:11am
8
zogstrip:
Yes.
Hm, it looks like this doesn’t work.
I had a non-active, staged user (that had been disabled via the interface and staged via Rails). After the user logged in (via SSO), he had to confirm has address, became active and could log in, but is still staged:
zogstrip
June 21, 2016, 9:16am
9
Hmm, it might not unstage when using SSO. Having a look.
fefrei
June 21, 2016, 9:22am
10
Thanks a lot!
In the meantime, for anyone who wants to try the same approach, here’s what I ran on the rails console (rails c inside the container):
User.where({ active: false }).update_all({ staged: true })
zogstrip
June 21, 2016, 9:24am
11
You can do slightly shorter
User.where(active: false).update_all(staged: true)
fefrei
June 21, 2016, 9:25am
12
I’ve given up understanding Ruby’s syntax a long time ago – even my version took many tries to be correct. Thanks for the shorter version
zogstrip
June 21, 2016, 9:29am
13
fefrei
June 21, 2016, 9:54am
14
After a journey into tests-passed lands, I can confirm this is fixed. Thanks a lot!
