Problem/Motivation

The UI / workflow around checking fields is very tedious, especially for complex sites.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork scanner-3364450

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

DamienMcKenna created an issue. See original summary.

smustgrave’s picture

Version: 8.x-1.x-dev » 2.0.x-dev

smustgrave’s picture

Status: Active » Needs review
StatusFileSize
new182.63 KB
new154.97 KB

Took a crack at it with an opinionated approach for sure.

smustgrave’s picture

Disclosure used claude to fix the tests but it does look cleaner.

smustgrave’s picture

smustgrave’s picture

Rebased

damienmckenna’s picture

This is a good improvement, thank you.

A few minor items:

* I don't think the form classes need to include "Scanner" in their names, I think the names could be simplified.
* It might be worth naming the new path "entities" rather than "entity-scope", which is overly verbose IMHO.
* An empty hook_update_N() might be needed to force sites to rebuild the caches, so the new routes & menu items are picked up.

smustgrave’s picture

Status: Needs review » Needs work

All makes sense to me

smustgrave’s picture

Status: Needs work » Needs review

Did points 2 + 3 but for 1 I actually looked around and most modules have the name of the module in the form name so don't think we should drop scanner but open to other suggestions.

smustgrave’s picture

Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.