Skip to content

Release/4.6.7#1454

Merged
Matus Tomlein (matus-tomlein) merged 2 commits intomasterfrom
release/4.6.7
Oct 7, 2025
Merged

Release/4.6.7#1454
Matus Tomlein (matus-tomlein) merged 2 commits intomasterfrom
release/4.6.7

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 Naoufal ELYOUSSOUFI (@nelyousfi) for the contribution

…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 <26110116+nelyousfi@users.noreply.github.com>
…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 <42166+matus-tomlein@users.noreply.github.com>

* Fix test to use proper EventStorePayload format

Co-authored-by: matus-tomlein <42166+matus-tomlein@users.noreply.github.com>

* Run rush change

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: matus-tomlein <42166+matus-tomlein@users.noreply.github.com>
Co-authored-by: Matus Tomlein <matus.tomlein@gmail.com>
@matus-tomlein Matus Tomlein (matus-tomlein) merged commit b3f618b into master Oct 7, 2025
1 of 2 checks passed
@matus-tomlein 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

Development

Successfully merging this pull request may close these issues.

3 participants