Skip to content

Conversation

@matus-tomlein
Copy link
Contributor

Bug fixes

  • Fix SecurityError when accessing localStorage in restricted browser environments
  • Add TypeScript definitions to "package.json" for "react-native-tracker", thanks to @nelyousfi for the contribution

matus-tomlein and others added 2 commits October 7, 2025 16:35
…r" (#1453)

### Issue:

When importing `@snowplow/react-native-tracker`, we got the following error:

```
This result could not be resolved under your current 'moduleResolution' setting. Consider updating to 'node16', 'nodenext', or 'bundler'.
```

Changing the "moduleResolution" to "node16" fixes this issue, but breaks part of our build pipeline.

### Similar issue:

snowplow/snowplow-react-native-tracker#217

---------

Co-authored-by: Naoufal ElYoussoufi <[email protected]>
…nvironments (#1452)

* Initial plan

* Wrap localStorage access in try-catch blocks to handle SecurityError

* Add console.error logging for localStorage access failures

Co-authored-by: matus-tomlein <[email protected]>

* Fix test to use proper EventStorePayload format

Co-authored-by: matus-tomlein <[email protected]>

* Run rush change

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: matus-tomlein <[email protected]>
Co-authored-by: Matus Tomlein <[email protected]>
@matus-tomlein matus-tomlein merged commit b3f618b into master Oct 7, 2025
1 of 2 checks passed
@matus-tomlein matus-tomlein deleted the release/4.6.7 branch October 7, 2025 14:51
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