Fix #13 about El Salvador

thiagodp · Aug 11, 2023 · 6982ba0 · 6982ba0
1 parent b6f146e
commit 6982ba0
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/generate.js b/generate.js
@@ -71,6 +71,8 @@ const inclusionsOrFixes = [
   { countryCode: 'HR', currencyCode: 'EUR' },
   // According to ISO 4217, Netherlands Antilles ('AN') uses Netherlands Antillean Guilder, code 'ANG'. Thanks @jayPare
   { countryCode: 'AN', currencyCode: 'ANG' },
+  // According to Wikipedia, El Salvador (SV) change from 'SVC' to 'USD' since 2001. Thanks @chaitanya-d
+  { countryCode: 'SV', currencyCode: 'USD' },
 ];
 
 let inclusions = 0;

diff --git a/index.ts b/index.ts
@@ -209,7 +209,7 @@ export default {
   SR: 'SRD',
   SS: 'SSP',
   ST: 'STN',
-  SV: 'SVC',
+  SV: 'USD',
   SX: 'ANG',
   SY: 'SYP',
   SZ: 'SZL',

diff --git a/readme.md b/readme.md
@@ -67,6 +67,7 @@ console.log( countryToCurrency[ 'US' ] ); // USD
 - For `Chile` (`CLF`), the currency `CLP` is being assumed.
 - For `Uruguay` (`UY`), the currency `UYU` is being assumed.
 - For `Croatia` (`HR`), the currency `EUR` is being assumed since January 1st, 2023.
+- For `El Salvador` (`SV`), the currency `USD` is being assumed.
 
 > _Suggestions? Please [open an Issue](https://github.com/thiagodp/country-to-currency/issues/new)._