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

Respect custom webroot in setup form #41189

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

HolonProduction
Copy link
Contributor

Summary

Passes an url generator to the installation template and uses it to generate an absoluteUrl.

@susnux susnux requested review from skjnldsv, come-nc, a team and icewind1991 and removed request for a team October 29, 2023 23:52
@susnux susnux requested a review from kesselb October 29, 2023 23:53
@szaimen szaimen added this to the Nextcloud 28 milestone Oct 30, 2023
core/templates/installation.php Outdated Show resolved Hide resolved
Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

Code make sense and worked for me

@skjnldsv skjnldsv mentioned this pull request Nov 1, 2023
This was referenced Nov 6, 2023
This was referenced Nov 14, 2023
@blizzz blizzz modified the milestones: Nextcloud 28, Nextcloud 29 Nov 23, 2023
@icewind1991 icewind1991 requested a review from kesselb March 8, 2024 11:02
@Altahrim
Copy link
Collaborator

@HolonProduction CI failures seems unrelated.
Can you reabse and reword your commit to use conventional commit please?

@HolonProduction HolonProduction force-pushed the webroot-setup branch 2 times, most recently from 23d1b45 to 13352ca Compare March 12, 2024 19:05
@HolonProduction
Copy link
Contributor Author

Done

@skjnldsv skjnldsv merged commit c6af46e into nextcloud:master Mar 14, 2024
152 of 159 checks passed
Copy link

welcome bot commented Mar 14, 2024

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

@Altahrim Altahrim removed the 3. to review Waiting for reviews label Mar 14, 2024
@Altahrim Altahrim mentioned this pull request Mar 14, 2024
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.

[Bug]: Setup form does not respect custom webroot
7 participants