getsentry / getsentry/sentry-react-native
Migrate route param gating to `dataCollection.userInfo`
- 主要言語
- TypeScript
- スター
- 1.8k
- フォーク
- 366
- 平均マージ
- 1日 6時間
- マージ済み PR(30日)
- 94
説明
Replace `sendDefaultPii` check in React Navigation integration with `dataCollection.userInfo`.
## Callsite
* `reactnavigation.ts:488` — dynamic route params (e.g. `[id]`, `[...slug]`) are only attached to navigation spans when `sendDefaultPii` is true
Should read from resolved `dataCollection.userInfo` instead.
## References
* `packages/core/src/js/tracing/reactnavigation.ts`
* `packages/core/test/tracing/reactnavigation.test.ts` — existing tests for route param behavior
コントリビューションガイド
調査の方向性
packages/core/src/js/tracing/reactnavigation.ts の 488 行目付近にあるルートパラメーターの処理から始め、次に packages/core/test/tracing/reactnavigation.test.ts にある既存のルートパラメーターのテストを確認してください。解決済みの dataCollection.userInfo を使用するようにゲーティング動作を更新し、React Navigation の tracing テストを実行して、動的なルートパラメーターが期待どおりに動作することを確認してください。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- react-native, typescript
- 領域
- mobile
- issue の種類
- リファクタリング
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 静か
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 58/100