Skip to content

docs(sharing): document shareapi_public_link_base_url app config#15361

Open
camr0 wants to merge 1 commit into
nextcloud:masterfrom
camr0:patch-1
Open

docs(sharing): document shareapi_public_link_base_url app config#15361
camr0 wants to merge 1 commit into
nextcloud:masterfrom
camr0:patch-1

Conversation

@camr0

@camr0 camr0 commented Jul 22, 2026

Copy link
Copy Markdown

Documentation for nextcloud/server#62146

🖼️ Screenshots

Screenshot 2026-07-22 at 2 10 44 AM

✅ Checklist

  • I have built the documentation locally and reviewed the output
  • Screenshots are included for visual changes
  • I have not moved or renamed pages (or added a redirect if I did)
  • I have run codespell or similar and addressed any spelling issues

Documentation for nextcloud/server#62146

Signed-off-by: Ali Aslam <ali.aslam@zohomail.com> 

Signed-off-by: Ali Aslam <ali.aslam@zohomail.com>

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: eef6aa0f7c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

+ When full match is activated, do not match user email
+ Default: ``yes``

- ``core.shareapi_public_link_base_url``

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Defer documenting the draft share config

Because this PR targets the latest documentation branch, this entry would publish in the current admin manual, but the linked implementation (nextcloud/server#62146) is still Draft/Open and the key is not present in current server master. If the docs merge first, admins following this page will set shareapi_public_link_base_url on installations that ignore it and share links will still use the instance URL, so this should wait for or be gated on the server change landing.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants