Skip to content

fix(setup): hide config section if autoconfig is valid#56231

Open
skjnldsv wants to merge 2 commits intomasterfrom
skjnldsv-patch-1
Open

fix(setup): hide config section if autoconfig is valid#56231
skjnldsv wants to merge 2 commits intomasterfrom
skjnldsv-patch-1

Conversation

@skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Nov 6, 2025

Fix #56160

@skjnldsv skjnldsv added this to the Nextcloud 33 milestone Nov 6, 2025
@skjnldsv skjnldsv self-assigned this Nov 6, 2025
@skjnldsv skjnldsv requested a review from a team as a code owner November 6, 2025 07:29
@skjnldsv skjnldsv added bug 3. to review Waiting for reviews labels Nov 6, 2025
@skjnldsv skjnldsv removed the request for review from a team November 6, 2025 07:29
@skjnldsv
Copy link
Member Author

skjnldsv commented Nov 6, 2025

/backport d73eec1 to stable32

@skjnldsv
Copy link
Member Author

skjnldsv commented Nov 6, 2025

/compile

@skjnldsv skjnldsv mentioned this pull request Nov 6, 2025
Open
szaimen
szaimen approved these changes Nov 6, 2025
View reviewed changes
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

🐘

@scubamuc
Copy link

scubamuc commented Nov 6, 2025

@skjnldsv confirmed

@kra-mo
Copy link
Member

kra-mo commented Nov 7, 2025

No strong opinions on whether it should be done that way in the first place, but it seems like an improvement over the status quo.

But I'd wait for any disagreements to be voiced first.

@skjnldsv
Copy link
Member Author

skjnldsv commented Nov 7, 2025

/compile rebase

@skjnldsv
Copy link
Member Author

skjnldsv commented Nov 18, 2025

/compile rebase

skjnldsv and others added 2 commits November 18, 2025 05:06
@skjnldsv @nextcloud-command
fix(setup): hide config section if autoconfig is valid
6a0858b 
Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
@nextcloud-command
chore(assets): Recompile assets
0ba362e 
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@Pilzinsel64 Pilzinsel64 mentioned this pull request Nov 22, 2025
Closed
5 tasks
This was referenced Jan 7, 2026
Merged
Merged
@nextcloud-bot nextcloud-bot mentioned this pull request Jan 14, 2026
Merged
This was referenced Jan 20, 2026
Merged
Merged
This was referenced Jan 29, 2026
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jancborchardt jancborchardt jancborchardt approved these changes

@szaimen szaimen szaimen approved these changes

@kra-mo kra-mo kra-mo approved these changes

@nfebe nfebe Awaiting requested review from nfebe nfebe is a code owner automatically assigned from nextcloud/server-frontend

@sorbaugh sorbaugh Awaiting requested review from sorbaugh sorbaugh is a code owner automatically assigned from nextcloud/server-frontend

Assignees

@skjnldsv skjnldsv

Labels

3. to review Waiting for reviews backport-request bug feature: install and update

Projects

None yet

Milestone

Nextcloud 33

Development

Successfully merging this pull request may close these issues.

installer: database section now visible if auto-setup is used

6 participants

@skjnldsv @scubamuc @kra-mo @jancborchardt @szaimen @nextcloud-command