Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

Delete local DomainRuleChecker module #113

Merged
merged 1 commit into from
Dec 6, 2021

Conversation

babolivier
Copy link
Contributor

The module has now moved to https://github.com/matrix-org/synapse-domain-rule-checker

Note to ops:

To deploy this change, the module needs to be installed in addition to Synapse (from the aforementioned repo, or the PyPI project mentioned its readme). The configuration doesn't change besides a) default is renamed into can_invite_if_not_in_domain_mapping and b) the module's configuration has moved to the modules section of the configuration file (though that was already the case as of #108).

@babolivier babolivier requested a review from a team December 3, 2021 14:19
Copy link
Member

@clokep clokep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like the code was copied into the module properly. I confirmed none of these files are part of the upstream Synapse.

Let me know if there's anything more in-depth to check.

@babolivier
Copy link
Contributor Author

babolivier commented Dec 6, 2021

Let me know if there's anything more in-depth to check.

Should be fine, I think that's all the checks we need - thanks!

@babolivier babolivier merged commit 4737a7f into dinsic Dec 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants