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

Port admin delegation selects #36650

Merged
merged 1 commit into from
Mar 1, 2023
Merged

Conversation

Pytal
Copy link
Member

@Pytal Pytal commented Feb 10, 2023

Summary

Screenshot
Before image
After image

Checklist

@Pytal
Copy link
Member Author

Pytal commented Feb 10, 2023

/backport to stable25

@@ -82,9 +82,11 @@
$settings = array_merge(
$settings,
array_map(function (IDelegatedSettings $setting) use ($section) {
$sectionName = $section->getName() . ($setting->getName() !== null ? ' - ' . $setting->getName() : '');

Check notice

Code scanning / Psalm

PossiblyNullOperand

Cannot concatenate with a possibly null null|string
@skjnldsv skjnldsv mentioned this pull request Feb 23, 2023
@Pytal Pytal force-pushed the enh/a11y-admin-delegation-select branch from efc20d1 to 4690d15 Compare February 24, 2023 01:16
@Pytal
Copy link
Member Author

Pytal commented Feb 24, 2023

Rebased

@Pytal
Copy link
Member Author

Pytal commented Feb 28, 2023

💡 would be nice if the double approve counted as two separate ones

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.

🐘

Signed-off-by: Christopher Ng <chrng8@gmail.com>
@Pytal Pytal force-pushed the enh/a11y-admin-delegation-select branch from 4690d15 to db5d5c1 Compare March 1, 2023 00:09
@Pytal Pytal added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 1, 2023
@Pytal Pytal merged commit b92e28e into master Mar 1, 2023
@Pytal Pytal deleted the enh/a11y-admin-delegation-select branch March 1, 2023 01:12
@blizzz blizzz mentioned this pull request Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish accessibility feature: settings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants