Skip to content

ShopCurrency: Add configurable decimals and NumericField scaffolding#859

Merged
wilr merged 3 commits intomainfrom
copilot/add-numeric-fields-for-price
May 2, 2026
Merged

ShopCurrency: Add configurable decimals and NumericField scaffolding#859
wilr merged 3 commits intomainfrom
copilot/add-numeric-fields-for-price

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 1, 2026

  • Add decimals config to ShopCurrency (default: 2)
  • Update Nice() to use the decimals config instead of hardcoded 2
  • Add scaffoldFormField() method returning NumericField with setScale() from config
  • Update tests to verify decimals config usage in Nice() and scaffoldFormField()
  • Fix phpcs lint issues in existing files (missing newlines at end of file, spacing after IF keyword)

Copilot AI assigned Copilot and wilr May 1, 2026
Copilot AI linked an issue May 1, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Add scaffold numeric fields for price in ShopCurrency ShopCurrency: Add configurable decimals and NumericField scaffolding May 1, 2026
Copilot AI requested a review from wilr May 1, 2026 22:47
Copilot finished work on behalf of wilr May 1, 2026 22:47
@wilr wilr marked this pull request as ready for review May 1, 2026 22:51
@wilr
Copy link
Copy Markdown
Contributor

wilr commented May 2, 2026

@copilot Fix the lint issues and rebase

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 2, 2026

@copilot Fix the lint issues and rebase

Fixed in commit 3abf529b. Used phpcbf to auto-fix 9 pre-existing phpcs violations (missing newlines at end of files in src/Tasks/*.php and a missing space after if in src/Forms/CheckoutComponentValidator.php). The branch is already on top of origin/main (no rebase was needed as it was already current).

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.

ShopCurrency: Scaffold Numeric Fields for Price etc...

2 participants