Skip to content
This repository was archived by the owner on Oct 29, 2024. It is now read-only.

Conversation

@Titozzz
Copy link
Contributor

@Titozzz Titozzz commented Dec 27, 2019

Android already supports it

Requirements

  • I have added test coverage for new or changed functionality
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

Provide links to any issues in this repository or elsewhere relating to this pull request.

Describe the solution you've provided

Provide a clear and concise description of what you expect to happen.

Describe alternatives you've considered

Provide a clear and concise description of any alternative solutions or features you've considered.

Additional context

Add any other context about the pull request here.

Android already supports it
@torchhound
Copy link
Contributor

Hi @Titozzz thanks for the PR, this change looks good! We'll do some testing and get this into the next release.

@torchhound torchhound requested review from torchhound and removed request for torchhound January 1, 2020 18:30
@Titozzz
Copy link
Contributor Author

Titozzz commented Jan 2, 2020

Btw web and iOS use String, but android uses enum, and crashes on unknown country

@torchhound torchhound merged commit 84dc998 into launchdarkly:master Jan 17, 2020
@torchhound
Copy link
Contributor

torchhound commented Jan 17, 2020

@Titozzz Thanks again for the PR, this change will go out in the next release very soon.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants