Skip to content

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

$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 <[email protected]>
@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.

5 participants