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

Fix outbound_federation_restricted_to docs & note when added #16628

Merged
merged 2 commits into from Nov 13, 2023
Merged

Fix outbound_federation_restricted_to docs & note when added #16628

merged 2 commits into from Nov 13, 2023

Conversation

DMRobertson
Copy link
Contributor

@DMRobertson DMRobertson commented Nov 13, 2023 •
edited

Missed in #15773 and #15913.

@github-actions github-actions bot deployed to PR Documentation Preview November 13, 2023 13:13 Active
@DMRobertson DMRobertson marked this pull request as ready for review November 13, 2023 13:13
@DMRobertson DMRobertson requested a review from a team as a code owner November 13, 2023 13:13
@github-actions github-actions bot deployed to PR Documentation Preview November 13, 2023 13:14 Active
clokep
clokep approved these changes Nov 13, 2023
View reviewed changes
Copy link
Contributor

@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.

What formatting error is being fixed?

@DMRobertson
Copy link
Contributor Author

DMRobertson commented Nov 13, 2023 •
edited

What formatting error is being fixed?

The extra newline before --- ensures that we insert a <hr> rather than converting the previous line into a <h2>. See https://matrix-org.github.io/synapse/v1.95/usage/configuration/config_documentation.html#outbound_federation_restricted_to for the current behaviour.

@github-actions github-actions bot deployed to PR Documentation Preview November 13, 2023 14:26 Active
@DMRobertson DMRobertson merged commit fb2554b into develop Nov 13, 2023
35 checks passed
@DMRobertson DMRobertson deleted the dmr/docs branch November 13, 2023 14:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@clokep clokep clokep approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@DMRobertson @clokep