Skip to content

Add reference to react-native-maps upgrade#10

Open
eduardinni wants to merge 2 commits intoexpo:mainfrom
eduardinni:ref-react-native-maps
Open

Add reference to react-native-maps upgrade#10
eduardinni wants to merge 2 commits intoexpo:mainfrom
eduardinni:ref-react-native-maps

Conversation

@eduardinni
Copy link

Context

This housekeeping reference will take care of it, once react-native-maps Expo plug-in is fixed.

- ./references/new-architecture.md -- SDK +53: New Architecture migration guide
- ./references/react-19.md -- SDK +54: React 19 changes (useContext → use, Context.Provider → Context, forwardRef removal)
- ./references/react-compiler.md -- SDK +54: React Compiler setup and migration guide
- ./references/react-native-maps.md -- SDK +54: react-native-maps Expo plug-in configuration
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- ./references/react-native-maps.md -- SDK +54: react-native-maps Expo plug-in configuration
- ./references/react-native-maps.md -- SDK +54: react-native-maps Config Plugin setup

- Remove implicit packages from `package.json`: `@babel/core`, `babel-preset-expo`, `expo-constants`.
- If the babel.config.js only contains 'babel-preset-expo', delete the file
- If the metro.config.js only contains expo defaults, delete the file
- Configure Expo plug-in for `react-native-maps` if the package is used by the app
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Configure Expo plug-in for `react-native-maps` if the package is used by the app

Doesn't apply to every app


Expo SDK 55 uses `react-native-maps` version 1.26.20+

An Expo plug-in for `react-native-maps` was introduced in version 1.22 and it requires Expo SDK 53+
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
An Expo plug-in for `react-native-maps` was introduced in version 1.22 and it requires Expo SDK 53+
`react-native-maps@1.22.0` now has an Expo Config Plugin (SDK 53+)

Comment on lines 66 to 76
### Migration Steps

1. Remove legacy configuration
2. Add `react-native-maps` Expo plug-in configuration

## Cleanup Checklist

When upgrading to SDK 55:

- [ ] Use `rect-native-maps` Expo plug-in
- [ ] Add Google Maps API keys for each platform in use
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
### Migration Steps
1. Remove legacy configuration
2. Add `react-native-maps` Expo plug-in configuration
## Cleanup Checklist
When upgrading to SDK 55:
- [ ] Use `rect-native-maps` Expo plug-in
- [ ] Add Google Maps API keys for each platform in use

redundant

Copy link
Author

Choose a reason for hiding this comment

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

thanks for the review - pushed a commit that includes your suggestions and other minor changes

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.

2 participants