Skip to content

Conversation

@pwltr
Copy link
Collaborator

@pwltr pwltr commented Feb 28, 2025

Description

For new installs we can get the user's local currencies with react-native-localize and check if there is one supported by the exchange rate API. If we find one put it in the redux store.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring (improving code without creating new functionality)

Tests

  • Detox test
  • Unit test
  • No test

Screenshots

Simulator.Screen.Recording.-.iPhone.16.-.2025-02-28.at.21.37.56.mp4

@socket-security
Copy link

socket-security bot commented Feb 28, 2025

👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report↗︎

@pwltr pwltr force-pushed the feat/locale-currency branch 2 times, most recently from 37f82ca to dcb5e82 Compare February 28, 2025 19:53
@pwltr pwltr force-pushed the feat/locale-currency branch from dcb5e82 to 1d8bc7a Compare February 28, 2025 20:22
@socket-security
Copy link

Updated dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/[email protected]3.4.1 None 0 127 kB zoontek

View full report↗︎

@pwltr pwltr marked this pull request as ready for review February 28, 2025 22:54
@pwltr pwltr requested a review from coreyphillips March 3, 2025 14:09
@pwltr pwltr merged commit ee8d8be into master Mar 3, 2025
7 checks passed
@pwltr pwltr deleted the feat/locale-currency branch March 3, 2025 15:59
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.

3 participants