Skip to content

Upgrade RN#2

Open
oleksandrkaDev wants to merge 9 commits intomainfrom
upgrade-rn
Open

Upgrade RN#2
oleksandrkaDev wants to merge 9 commits intomainfrom
upgrade-rn

Conversation

@oleksandrkaDev
Copy link
Copy Markdown

Problem

The old version of the component not work with the new version of React Native

Solution

Upgrade libs

Copy link
Copy Markdown

@adebroux adebroux left a comment

Choose a reason for hiding this comment

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

LGTM! Don't forget to update the version number

@danicunhac
Copy link
Copy Markdown

This one has conflicts

oleksandrkaDev and others added 7 commits August 21, 2025 14:50
* Update packages

* Address PR review feedback for custom SVG icons

- Restrict URL loading to HTTPS only (drop http://)
- Fix isSVG check to parse URL pathname instead of naive includes
- Add in-memory SVG fetch cache to avoid repeated network requests
- Add propTypes and fix Prettier/ESLint formatting in TintedSvgIcon
- Remove console.error usage
- Move react-native-svg to peerDependencies
- Revert peerDependency version ranges to unconstrained

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Fix URL parsing for Android native

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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