Skip to content

[stable28] fix: added a label element to the NcSelect in update settings - #43158

Merged
susnux merged 2 commits into
stable28from
backport/43094/stable28
Feb 2, 2024
Merged

[stable28] fix: added a label element to the NcSelect in update settings#43158
susnux merged 2 commits into
stable28from
backport/43094/stable28

Conversation

@backportbot

@backportbotbackportbotBot commented Jan 27, 2024

Copy link
Copy Markdown

Backport of #43094

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove [skip ci] tag

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbotbackportbotBot added this to the Nextcloud 28.0.2 milestone Jan 27, 2024
@blizzzblizzz mentioned this pull request Jan 29, 2024
1 task
@emoral435

Copy link
Copy Markdown
Contributor

/compile amend/

@emoral435
emoral435 marked this pull request as ready for review January 29, 2024 13:38
Signed-off-by: Eduardo Morales <emoral435@gmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
[skip ci]
Signed-off-by: Eduardo Morales <emoral435@gmail.com>
@emoral435
emoral435force-pushed the backport/43094/stable28 branch from cef7ae9 to f59ef7bCompareJanuary 29, 2024 14:33
@Pytal

Copy link
Copy Markdown
Member

Bot 🤖 left some checkboxes to check off @emoral435 :)

@emoral435

emoral435 commented Jan 30, 2024

Copy link
Copy Markdown
Contributor

@Pytal Thank you for the heads up! I should have left a comment explaining the compile commits I added. Everything is compiled correctly with this PR, the number of files being different just comes from different compiling of unsupported-browser js files, but this one just has the changed file, and the associated 2 compiled JS files

EDIT: I did now change the description for the PR now to reflect that any issues have been resolved 👍

@Pytal

Copy link
Copy Markdown
Member

Thanks @emoral435!

@PytalPytal added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 30, 2024
@susnux
susnux merged commit 7dc0234 into stable28Feb 2, 2024
@susnux
susnux deleted the backport/43094/stable28 branch February 2, 2024 11:07
@skjnldsvskjnldsv mentioned this pull request Feb 14, 2024
8 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishaccessibilitybug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@emoral435@Pytal@susnux@blizzz