search

hamlet-io / executor-bash

Executor for hamlet based on bash
GNU General Public License v3.0
0 stars 4 forks source link

fix(sentry): specify a sentry url prefix for expo-managed apps #266

Closed kshychko closed 3 years ago

kshychko commented 3 years ago

Intent of Change

Description

Stores an additional context parameter to pass a sentry prefix URL for expo managed apps.

Motivation and Context

React-native apps are using standard filenames for source files and corresponding source maps. For expo managed apps the prefix needs to be specified in order to load source maps properly during event processing on Sentry side. The prefix URL for expo-managed apps matches a prefix for a public OTA URL

How Has This Been Tested?

Tested locally.

Related Changes

Prerequisite PRs:

Dependent PRs:

Consumer Actions:

kshychko commented 3 years ago

Closing this PR in favour of changes that can be applied on the expo Sentry integration side: https://docs.expo.io/guides/using-sentry/#self-hosting-ota