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.

Sign up for GitHub

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

Jump to bottom

Persist none role for outcasts #4010

Merged
merged 1 commit into from
Mar 20, 2023
Merged

Persist none role for outcasts #4010

merged 1 commit into from
Mar 20, 2023

Conversation

based-a-tron
Copy link
Contributor

none roles should be persisted for banned users. I totally forgot about this, my bad.

@based-a-tron
Persist none role for outcasts
75d75c5 
`none` roles *should* be persisted for banned users. I totally forgot about this, my bad. I'm shocked nobody else noticed it.
@coveralls
Copy link

Coverage Status

Coverage: 33.154% (-0.02%) from 33.17% when pulling 75d75c5 on based-a-tron:bugfix into 04b431f on processone:master.

@badlop badlop added this to the ejabberd 23.xx milestone Mar 20, 2023
@badlop
Copy link
Member

badlop commented Mar 20, 2023

That PR follows 4d3875f and a8121cd related to issue #3330, and were included in ejabberd 22.10.

@badlop badlop merged commit f953dc3 into processone:master Mar 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
ejabberd 23.04
Development

Successfully merging this pull request may close these issues.
3 participants
@based-a-tron @coveralls @badlop