Skip to content

required-registration-fields 2.0.2: tested up to Shopclass 7.1 - #30

Merged
navjottomer merged 1 commit into
mainfrom
rrf-tested-7.1
Nov 6, 2026
Merged

navjottomer merged 1 commit into
mainfrom
rrf-tested-7.1

Conversation

@navjottomer

Copy link
Copy Markdown
Member

Installed from Browse on Shopclass 7.1, saved its settings, and checked sign-up: an empty required field is refused on the server, and a full sign-up saves every field.

@github-actions

github-actions Bot commented Nov 5, 2026

Copy link
Copy Markdown

Package validation

Automated checks against docs/PACKAGE-SPEC.md. Gates marked ❌ block merging;
⚠️ and ⏭️ never do — see CONTRIBUTING.md §3 for what each gate means.

required-registration-fields — ✅ Passing

Gate Result
Gate Status
--- ---
Structure, manifest & security ✅
Deprecated API ⚠️
Smoke install ✅
2. Manifest (shopclass.json schema) ✅ 0 error(s), 0 warning(s)
5. Parse (php -l, 8.0–8.5) ✅ 0 failure(s) across 6 PHP version(s)
6. PHP 8.0 floor (PHPCompatibility) ✅ 0 error(s), 0 warning(s)
10. Style (PSR-12) — warning only ⚠️ 0 warning(s) — never blocking

Package CI — required-registration-fields

Errors

None.

Warnings — not blocking

These do not gate anything. A pull request may be merged with warnings outstanding — they exist to inform, not to reject.

  • index.php:158 — Preference::newInstance is deprecated since 7.0.0, with no scheduled removal version yet. Use getInstance() instead.
  • index.php:248 — User::newInstance is deprecated since 7.0.0, with no scheduled removal version yet. Use getInstance() instead.

Updated for commit 21e287b.

@navjottomer
navjottomer merged commit b7196d0 into main Nov 6, 2026
5 checks passed
@navjottomer
navjottomer deleted the rrf-tested-7.1 branch November 6, 2026 03:23
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.

1 participant