Skip to content

bugfix/AB#32103-cannot-set-approved-amount-zero#2140

Merged
DavidBrightBcGov merged 1 commit into
devfrom
bugfix/AB#32103-cannot-set-approved-amount-zero
Mar 17, 2026
Merged

bugfix/AB#32103-cannot-set-approved-amount-zero#2140
DavidBrightBcGov merged 1 commit into
devfrom
bugfix/AB#32103-cannot-set-approved-amount-zero

Conversation

@DavidBrightBcGov
Copy link
Copy Markdown
Contributor

AB#32103
Added the setting from maskMoney's library of "allowZero" to the input tag.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to fix AB#32103 by allowing an approved amount of 0 to be entered/saved in the Assessment Results “Approved Amount” currency field.

Changes:

  • Adds a data-allow-zero="true" attribute to the approved amount input.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 561
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@DavidBrightBcGov DavidBrightBcGov merged commit c801c39 into dev Mar 17, 2026
26 checks passed
@JamesPasta JamesPasta deleted the bugfix/AB#32103-cannot-set-approved-amount-zero branch March 18, 2026 01:31
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.

2 participants