Skip to content

Conversation

@TaprootFreak
Copy link
Collaborator

@TaprootFreak TaprootFreak commented Dec 31, 2025

Summary

  • Add Liechtenstein bank holidays configuration for 2026
  • Prevent fiat outputs with LI IBANs and LiqManagement type from getting isReadyDate on Liechtenstein bank holidays
  • Block transactions on day before bank holiday after 16:00 (LI IBANs with LiqManagement type only)

Changes

  • New file: liechtenstein-bank-holiday.config.ts with all 2026 Liechtenstein bank holidays
  • Updated fiat-output-job.service.ts to check Liechtenstein bank holidays for LI IBANs with type LiqManagement
  • Only FiatOutputs with LI IBAN AND type=LiqManagement are affected by holiday checks
  • Other IBAN countries and other FiatOutput types are not affected

Test plan

  • Verify LI IBANs with type LiqManagement don't get isReadyDate on Liechtenstein bank holidays
  • Verify LI IBANs with type LiqManagement don't get isReadyDate after 16:00 on day before Liechtenstein bank holiday
  • Verify LI IBANs with other types (BuyFiat, etc.) are not affected
  • Verify CH and other IBANs are not affected by Liechtenstein bank holidays

- Add liechtenstein-bank-holiday.config.ts with 2026 holidays
- Update fiat-output-job.service to check for LI/CH bank holidays
- Block isReadyDate on bank holidays and day before after 16:00
- Bank holiday blocking now only applies to Liechtenstein IBANs
- CH and other IBANs are not affected by holiday checks
- Remove unused isBankHoliday import
- Bank holiday blocking now only applies to FiatOutputs with type LiqManagement
- Other types (BuyFiat, BuyCryptoFail, etc.) are not affected
- Add verbose logging when FiatOutput is blocked due to bank holiday
- Use early return (continue) for better readability
- Only calculate isAfter16 when actually needed (performance)
- Clearer variable scoping and logic flow
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