Skip to content

Conversation

@DanDaManTran
Copy link
Contributor

https://bitovi.atlassian.net/browse/FE-184

Checked and copied academy to demo. Most are already similar.

@DanDaManTran DanDaManTran requested a review from CMLabay April 8, 2025 21:18
Copy link

@CMLabay CMLabay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FormSwitch has different props in academy repo.
FormText has different props in academy repo.
RestaurantHeader has several differences in academy repo.
Box has several differences in academy repo.

A lot of the index files in academy have more in them than this repo

Maybe I am misunderstanding, but it seems like there are a lot of changes in Academy/exercises/react-native/20-building/01-solution that are not captured in this repo. I stopped at Design/Box, but to me it looks like we need to go back through everything and make sure each file are exactly the same.

Copy link

@CMLabay CMLabay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for looking back over this!

@DanDaManTran DanDaManTran merged commit 70abdf8 into main Apr 14, 2025
1 check passed
@DanDaManTran DanDaManTran deleted the FE-184-native-app-migrate-improvements-from-academy-into-demo-app branch April 14, 2025 20:29
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