Skip to content
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

Fabric migration - @react-native-community/datetimepicker -> react-native-date-picker #3104

Merged
merged 5 commits into from
Mar 6, 2024

Conversation

haileyok
Copy link
Contributor

@haileyok haileyok commented Mar 5, 2024

Testing requires building a client after checking out #3096

This PR replaces @react-native-community/datetimepicker with react-native-date-picker. The former is not compatible with Fabric.

Note: An unrelated issue to this PR that we need to fix is that the timezones are not maintained between the picker and the displayed date. This is obvious during user onboarding, whenever a user picks their date of birth that should put them at 18, but actually results in them being 17 and 364 days old 🫠 - Update this is fixed!

Test Plan

  • Settings -> Birthday -> Press Show
    • iOS
    • Android
  • Storyboard -> Date picker
    • iOS
    • Android
  • Create New Account
    • iOS
    • Android

@haileyok haileyok changed the base branch from main to hailey/fabric-migration-pt1 March 5, 2024 05:31
@haileyok haileyok marked this pull request as ready for review March 5, 2024 06:09
@haileyok haileyok merged commit f813ddc into hailey/fabric-migration-pt1 Mar 6, 2024
4 checks passed
@haileyok haileyok deleted the hailey/fabric-migration-pt6 branch June 8, 2024 07:52
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.

1 participant