Skip to content

Add support for DKK, SEK, and NOK currencies#201

Merged
a1denvalu3 merged 4 commits intomainfrom
feat/add-nordic-currencies
Mar 3, 2026
Merged

Add support for DKK, SEK, and NOK currencies#201
a1denvalu3 merged 4 commits intomainfrom
feat/add-nordic-currencies

Conversation

@a1denvalu3
Copy link
Collaborator

Summary

This PR adds support for Danish Krone (DKK), Swedish Krona (SEK), and Norwegian Krone (NOK) to the Numo app.

Changes

  • Updated Amount.Currency enum to include DKK, SEK, and NOK.
  • Updated Amount.parse to handle multi-character symbols and ambiguous symbols (e.g., "kr") by prioritizing the user's selected currency.
  • Updated CurrencyManager to provide correct symbols ("kr.", "kr") and Coinbase API URLs.
  • Updated BitcoinPriceWorker to fetch and cache prices for the new currencies.
  • Updated CurrencySettingsActivity and UI to include radio buttons for selecting these currencies.
  • Added translation strings for the new options.
  • Added unit tests for Amount parsing and formatting with the new currencies.

Testing

  • Verified Amount.parse correctly handles "kr." (DKK) and "kr" (SEK/NOK).
  • Verified CurrencyManager returns correct symbols and API URLs.
  • Verified UI updates correctly when selecting new currencies.

Adds Danish Krone, Swedish Krona, and Norwegian Krone to supported currencies. Updates Amount parsing to handle multi-character symbols and ambiguous currency codes (kr). Updates UI to allow selecting new currencies.
@github-project-automation github-project-automation bot moved this to Backlog in Numo Mar 3, 2026
@a1denvalu3 a1denvalu3 force-pushed the feat/add-nordic-currencies branch from 82de26d to 48eb5c4 Compare March 3, 2026 09:59
@a1denvalu3 a1denvalu3 merged commit a4433c8 into main Mar 3, 2026
3 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in Numo Mar 3, 2026
@a1denvalu3 a1denvalu3 deleted the feat/add-nordic-currencies branch March 3, 2026 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant