Skip to content

Added Additional Currency Formats#116

Merged
mitchelsellers merged 1 commit intodevelopfrom
feature/new-templates
Feb 3, 2026
Merged

Added Additional Currency Formats#116
mitchelsellers merged 1 commit intodevelopfrom
feature/new-templates

Conversation

@mitchelsellers
Copy link
Contributor

@mitchelsellers mitchelsellers commented Feb 3, 2026

Added New Currency Formats & addressed a casing issue impacting formatting.

Summary by CodeRabbit

Release Notes

  • New Features
    • Added support for currency formatting with 3 and 4 decimal places in spreadsheet columns, providing greater flexibility for financial data representation.
    • Enhanced conditional formatting to apply formatting only when explicitly specified.

Added New Currency Formats & addressed a casing issue impacting formatting.
@mitchelsellers mitchelsellers added the enhancement New feature or request label Feb 3, 2026
@mitchelsellers mitchelsellers merged commit 9c9a6a1 into develop Feb 3, 2026
1 of 3 checks passed
@mitchelsellers mitchelsellers deleted the feature/new-templates branch February 3, 2026 14:52
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 3, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

The pull request extends spreadsheet formatting capabilities by adding support for Currency3 and Currency4 formats. Changes include new currency format constants in SpreadsheetColumnAttribute and enhanced conditional formatting logic in OpenXmlSpreadsheetGenerator to apply these formats alongside existing options.

Changes

Cohort / File(s) Summary
Currency Format Enhancement
src/NetCore.Utilities.Spreadsheet/SpreadsheetColumnAttribute.cs
Added two new public constants Currency3 = "c3" and Currency4 = "c4" to the ColumnFormats class for currency formatting with 3 and 4 decimal places.
Spreadsheet Generator Formatting
src/NetCore.Utilities.Spreadsheet/OpenXmlSpreadsheetGenerator.cs
Extended formatting logic to conditionally apply formats only when non-null/non-empty. Added Currency3 and Currency4 support to format mapping switch statement. Extended FontStyleIndex enum and stylesheet numbering formats (IDs 401, 402) to support new currency formats.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A nibble of change, so precise and neat,
Currency formats now taste complete,
Three decimals, four, with numbers so bright,
The spreadsheet dances in formatting light! ✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/new-templates

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Failure to add the new IP will result in interrupted reviews.


Comment @coderabbitai help to get the list of available commands and usage tips.

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant