Skip to content

Conversation

@kofimokome
Copy link
Contributor

@kofimokome kofimokome commented Oct 30, 2025

Fixes #1086

Alternative solution to #1093

Screenshot 2025-10-28 at 12 18 58 AM

@github-actions
Copy link

github-actions bot commented Oct 30, 2025

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: kofimokome <[email protected]>
Co-authored-by: davidperezgar <[email protected]>
Co-authored-by: ryanwelcher <[email protected]>
Co-authored-by: ernilambar <[email protected]>
Co-authored-by: productivemindspublishing <[email protected]>
Co-authored-by: jeffpaul <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@davidperezgar
Copy link
Member

Hello @kofimokome thanks for your contribution. Could you make columns for categories and type?, so It does not take more scroll?

@kofimokome
Copy link
Contributor Author

hi @davidperezgar
Is this okay?
Screenshot 2025-11-06 at 8 03 00 PM

@davidperezgar
Copy link
Member

A little more space and we are done.

@davidperezgar davidperezgar added this to the 1.8.0 milestone Nov 7, 2025
@kofimokome
Copy link
Contributor Author

Screenshot 2025-11-08 at 9 16 05 AM I have added more space

@kofimokome kofimokome force-pushed the feat/hide-errors-2 branch 5 times, most recently from a0389a9 to b47c3f8 Compare November 8, 2025 08:49
@davidperezgar
Copy link
Member

@kofimokome I've merged some PR. You could resolve the conflicts and we are done!

@kofimokome
Copy link
Contributor Author

kofimokome commented Nov 23, 2025

Hi @davidperezgar I discovered that the complexity of the Admin_Ajax file has increased to 53 after resolving conflicts. Do you have any suggestions on how to reduce it to below 50?
It looks like the methods in this class are now too many. Do we need to split this class into two?

@davidperezgar
Copy link
Member

Could you try to fix it? If you try and you cannot make it, we can see the option to disable this directive.

@kofimokome
Copy link
Contributor Author

Hello @davidperezgar I have reduced the complexity of the Admin_AJAX.php file.
All checks have pass.

@davidperezgar
Copy link
Member

Thanks!!!!

@davidperezgar davidperezgar merged commit f35be3a into WordPress:trunk Nov 28, 2025
28 checks passed
@jeffpaul
Copy link
Member

Seems the Props Bot wasn't used on this commit?

@davidperezgar
Copy link
Member

davidperezgar commented Dec 11, 2025

It's supposed to be automatic. We didn't change any set.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add ability to filter for Errors and Warnings

5 participants