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

RN-973 Upgrade Meditrak #5120

Merged
merged 59 commits into from
Nov 24, 2023
Merged

RN-973 Upgrade Meditrak #5120

merged 59 commits into from
Nov 24, 2023

Conversation

IgorNadj
Copy link
Contributor

@IgorNadj IgorNadj commented Oct 29, 2023

Issue #: RN-973

Package updates:

Start from a brand new react-native. Add up to date packages and get it building. Both Android and iOS.

DEPRECATED PACKAGES LIST

  • react-native-camera

    • react-native-vision-camera
    • must upgrade, old version causing all sorts of issues
    • ✅ Upgraded
  • react-native-qrcode-scanner

    • The recommendation of most developers that were previously using this library is to consider using react-native-vision-camera with its plugin vision-camera-code-scanner
    • ✅ Upgraded
  • react-native-side-menu

    • Recommendation is react-native-side-menu-updated
    • 🟡 Not upgraded, works fine as is, will leave for a future PR
  • react-navigation

    • Recommendation is @react-navigation/native
    • 🟡 Not upgraded, works fine as is, will leave for a future PR
    • Take react-navigation-redux-helpers with it?
  • sha256

    • 🟡 Not upgraded, works fine as is, will leave for a future PR

AppCenter changes:

Fixed the builds for Android and iOS.
iOS has been re-setup to build and release through AppCenter following AppCenter documentation.

Manual release tasks

  • Update Slab for Meditrak app releasing

@IgorNadj IgorNadj changed the title RN-973 Upgrade Meditrak RN-973 Upgrade Meditrak (Builds locally) Oct 30, 2023
@IgorNadj IgorNadj marked this pull request as draft October 30, 2023 00:06
@IgorNadj IgorNadj changed the base branch from dev to upgrade-meditrak-epic October 30, 2023 00:08
@IgorNadj IgorNadj force-pushed the rn-973-upgrade-meditrak20 branch from ca47f0d to bfb78bd Compare October 30, 2023 01:00
@IgorNadj IgorNadj marked this pull request as ready for review October 30, 2023 01:11
@IgorNadj IgorNadj force-pushed the rn-973-upgrade-meditrak20 branch 5 times, most recently from 4e6db0c to b477624 Compare November 3, 2023 05:33
@IgorNadj IgorNadj changed the title RN-973 Upgrade Meditrak (Builds locally) RN-973 Upgrade Meditrak Nov 6, 2023
@IgorNadj IgorNadj changed the base branch from upgrade-meditrak-epic to dev November 6, 2023 05:10
@IgorNadj IgorNadj force-pushed the rn-973-upgrade-meditrak20 branch from 74d6a56 to 60a9af1 Compare November 6, 2023 05:12
@IgorNadj IgorNadj force-pushed the rn-973-upgrade-meditrak20 branch 2 times, most recently from c3940e4 to 5d9ecee Compare November 9, 2023 05:47
@IgorNadj IgorNadj force-pushed the rn-973-upgrade-meditrak20 branch from 5d9ecee to a1025d2 Compare November 9, 2023 06:05
@IgorNadj IgorNadj force-pushed the rn-973-upgrade-meditrak20 branch from 560ba48 to 3d677e6 Compare November 9, 2023 23:16
Copy link
Collaborator

@rohan-bes rohan-bes 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 this @IgorNadj!

Ran through a local install and had a minor issue with realm but aside from that all working well. Wanna go through this together on Friday and then hopefully we can merge 👍


Android and iOS app for surveying medical facilities
- nvm use
Copy link
Collaborator

Choose a reason for hiding this comment

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

I had to add an .nvmrc file with 16.20.2 in it to the root of the meditrak-app package

"redux-persist": "^5.9.1",
"react-navigation-stack": "^2.10.4",
"react-redux": "^8.1.2",
"realm": "^12.1.0",
Copy link
Collaborator

Choose a reason for hiding this comment

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

When testing this locally I was having issues with a missing realm-constants.json file in node_modules/ream. There seems be something that some other users have encountered: realm/realm-js#6270

Apparently there's meant to be a post-install script that creates that file but something seemed to be going wrong for me.

Bumping to 12.3.0 fixed the issue though, currently yarn.lock has resolved to 12.2.1

Copy link

Choose a reason for hiding this comment

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

Try to upgrade to v12.3.1 as we have a fix for it 😄

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks @kneth 🙏

Copy link
Collaborator

@rohan-bes rohan-bes left a comment

Choose a reason for hiding this comment

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

Awesome stuff, thanks for all the work!

@IgorNadj
Copy link
Contributor Author

Merging before test pass because this only affects MediTrak App, which is on a change freeze pending this upgrade.

I have done a smoke test on the build and run of web-frontend and it passed, any other issues will be caught by release regression testing.

@IgorNadj IgorNadj merged commit c949df4 into dev Nov 24, 2023
42 checks passed
@IgorNadj IgorNadj deleted the rn-973-upgrade-meditrak20 branch November 24, 2023 03:33
@IgorNadj IgorNadj restored the rn-973-upgrade-meditrak20 branch November 30, 2023 05:18
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