Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(url): correct WOPI settings URL to prevent duplicate nextcloud #4461

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

codewithvk
Copy link
Collaborator

@codewithvk codewithvk commented Feb 9, 2025

Replaced getCallbackBaseUrl() with getBaseUrl() to ensure the correct WOPI settings URL. This prevents the issue where /nextcloud appears twice in the generated URL.

prev URL : http://localhost/nextcloud/nextcloud/index.php/apps/richdocuments/wopi/settings
now it should be: http://localhost/nextcloud/index.php/apps/richdocuments/wopi/settings

@codewithvk codewithvk force-pushed the private/codewithvk/url_issue branch 4 times, most recently from 87c828c to f235e9c Compare February 9, 2025 09:12
@codewithvk codewithvk marked this pull request as ready for review February 10, 2025 09:14
@codewithvk codewithvk force-pushed the private/codewithvk/url_issue branch from f235e9c to 565535f Compare February 10, 2025 09:15
src/helpers/url.js Outdated Show resolved Hide resolved
@codewithvk codewithvk force-pushed the private/codewithvk/url_issue branch from 565535f to 40a9df1 Compare February 10, 2025 16:17
@codewithvk codewithvk force-pushed the private/codewithvk/url_issue branch 3 times, most recently from 40a9df1 to 9f61c3a Compare February 12, 2025 12:13
@codewithvk
Copy link
Collaborator Author

CI Irrelevant of this PR, @juliusknorr @elzody Could you please review this PR?

@pedropintosilva pedropintosilva force-pushed the private/codewithvk/url_issue branch from 9f61c3a to 04d6ce7 Compare February 13, 2025 10:01
Signed-off-by: codewithvk <vivek.javiya@collabora.com>
@codewithvk codewithvk force-pushed the private/codewithvk/url_issue branch from 04d6ce7 to 49df6d4 Compare February 15, 2025 06:30
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.

2 participants