-
Notifications
You must be signed in to change notification settings - Fork 421
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
Handle remove_domain in mod_inbox #3103
Conversation
Move host_type type into mongooseim module
Codecov Report
@@ Coverage Diff @@
## master #3103 +/- ##
==========================================
- Coverage 79.06% 79.04% -0.02%
==========================================
Files 386 386
Lines 31830 31836 +6
==========================================
Hits 25165 25165
- Misses 6665 6671 +6
Continue to review full report at Codecov.
|
@@ -108,6 +108,30 @@ The command can be used instead: | |||
And `mam` is used to run `mam_SUITE` suite only. | |||
|
|||
|
|||
## Debugging big tests database |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
absolutely love it.
it's needed probably twice a year but always time consuming to recall how to do it. thanks a lot
src/inbox/mod_inbox.erl
Outdated
@@ -267,6 +271,12 @@ remove_user(Acc, User, Server) -> | |||
mod_inbox_utils:clear_inbox(User, Server), | |||
Acc. | |||
|
|||
-spec remove_domain(mongoose_acc:t(), mongooseim:host_type(), jid:lserver()) -> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's not mongoose_acc:t()
, MB we need some special type to highlight that Acc is not used!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, changed.
@@ -15,6 +15,8 @@ | |||
%%============================================================================== | |||
-module(mongooseim). | |||
|
|||
-type host_type() :: binary(). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just one minor comment regarding Acc type. But it can be changed later in the following PRs
This PR addresses "tenant removal epic".
Proposed changes include: