-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add missing currencies #7441
Add missing currencies #7441
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
This pull request significantly expands the currency support in the application, addressing issue #7038 by adding a comprehensive list of world currencies.
- Added 153 currency codes to the CurrencyCode enum in
packages/twenty-front/src/modules/object-record/record-field/types/CurrencyCode.ts
- Expanded
SETTINGS_FIELD_CURRENCY_CODES
inpackages/twenty-front/src/modules/settings/data-model/constants/SettingsFieldCurrencyCodes.ts
with new currency codes, labels, and icons - Introduced numerous new currency icons in
packages/twenty-ui/src/display/icon/components/TablerIcons.ts
- Corrected existing currency representations, such as updating the Thai Baht code from 'BHT' to 'THB'
3 file(s) reviewed, 3 comment(s)
Edit PR Review Bot Settings
packages/twenty-front/src/modules/object-record/record-field/types/CurrencyCode.ts
Show resolved
Hide resolved
packages/twenty-front/src/modules/settings/data-model/constants/SettingsFieldCurrencyCodes.ts
Outdated
Show resolved
Hide resolved
packages/twenty-front/src/modules/settings/data-model/constants/SettingsFieldCurrencyCodes.ts
Outdated
Show resolved
Hide resolved
packages/twenty-front/src/modules/settings/data-model/constants/SettingsFieldCurrencyCodes.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @BOHEUS
I'm back to code review after spending a lot of time on the tests :)
Left a few comments!
@charlesBochet I've answered your comments, please take a look in free time |
packages/twenty-front/src/modules/settings/data-model/constants/SettingsFieldCurrencyCodes.ts
Outdated
Show resolved
Hide resolved
packages/twenty-front/src/modules/settings/data-model/constants/SettingsFieldCurrencyCodes.ts
Outdated
Show resolved
Hide resolved
@BOHEUS TY, I have answered :) |
@charlesBochet done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @BOHEUS as always!
Thanks @BOHEUS for your contribution! |
/award 150 |
Awarding BOHEUS: 150 points 🕹️ Well done! Check out your new contribution on oss.gg/BOHEUS |
Related to twentyhq#7038 --------- Co-authored-by: Charles Bochet <charles@twenty.com>
Related to #7038