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 👍 / 👎.

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

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