Skip to content

multi-ingress: Document to disable public account creation - #125

Merged
supersven merged 2 commits into
mainfrom
sventennie/multi-ingress-team-settings-docs
Oct 7, 2026
Merged

supersven merged 2 commits into
mainfrom
sventennie/multi-ingress-team-settings-docs

Conversation

@supersven

Copy link
Copy Markdown
Contributor

Team settings is not multi-ingress ready, so this won't work anyways. Also, allowing everyone to create accounts is certainly not what on-prems want.

Change type

  • Documentation change
  • Build pipeline change
  • Submodule update
  • Deployment change

Basic information

  • THIS CHANGE REQUIRES A WIRE-DOCS RELEASE NOW

Testing

  • I ran/applied the changes myself, in a test environment.

Tracking

  • I mentioned this PR in Jira, OR I mentioned the Jira ticket in this PR.
  • I mentioned this PR in one of the issues attached to one of our repositories.

@supersven
supersven requested review from a team as code owners August 13, 2026 09:39
@supersven
supersven force-pushed the sventennie/multi-ingress-team-settings-docs branch from 118d7af to d40290d Compare September 24, 2026 14:05
@supersven
supersven force-pushed the sventennie/multi-ingress-team-settings-docs branch from d40290d to 16bc883 Compare October 7, 2026 07:23
Comment thread src/how-to/install/multi-ingress.md Outdated
@supersven
supersven force-pushed the sventennie/multi-ingress-team-settings-docs branch 2 times, most recently from 9b5d951 to 426dd62 Compare October 7, 2026 09:14
Team settings is not multi-ingress ready, so this won't work anyways.
Also, allowing everyone to create accounts is certainly not what
on-prems want.
@supersven
supersven force-pushed the sventennie/multi-ingress-team-settings-docs branch from 426dd62 to 935a622 Compare October 7, 2026 13:08
@supersven
supersven requested a balanced review from Copilot October 7, 2026 13:11

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

Only an optional grammar correction remains; no blocking issues were identified.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
What changed in this PR

Documents how to disable public account creation in multi-ingress deployments.

Changes:

  • Adds optional Brig restrictions and team-owner provisioning guidance.
  • Disables registration in the webapp example and explains team settings’ multi-ingress limitation.
File Description
src/​how-to/​install/​multi-ingress.md Adds account-creation restrictions and updates webapp configuration guidance.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/how-to/install/multi-ingress.md Outdated
Fix typo and clarify usage of team settings app.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@supersven
supersven requested a review from mohitrajain October 7, 2026 13:17

@mohitrajain mohitrajain left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@supersven
supersven merged commit cf74c5f into main Oct 7, 2026
2 checks passed
@supersven
supersven deleted the sventennie/multi-ingress-team-settings-docs branch October 7, 2026 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants