Remove deprecated APIs in the next major

Offen
#6,969 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
45/100
Issue-Typ
Refactoring
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
react-native, typescript

Rechercherichtung

Start with the listed entry points: options.ts, client.ts, touchevents.tsx, index.ts, tracing/timetodisplay.tsx, and tracing/integrations/appStart.ts. Remove the deprecated APIs and options identified in the issue, then update the referenced TODOs in test/client.test.ts and test/sdk.test.ts. Check the v8 to v9 migration guide work tracked in #6735; done means the untracked removals and follow-up notes are addressed.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

React-Native Task

Description

Several APIs and options are still exported after one or more majors, some with JSDoc that says "will be removed in the next major version". Remove them in the next major.

Items that already have their own issue are listed for reference only.

To remove (no dedicated issue)
  • initialScope option (options.ts). Use Sentry.getGlobalScope().setTag(...) / setUser(...) instead.
  • ReactNativeClient.captureUserFeedback (client.ts). Use Sentry.captureFeedback instead. Also update the remaining test TODO in test/client.test.ts.
  • ignoredDisplayNames prop on TouchEventBoundary (touchevents.tsx). Use ignoreNames instead.
  • useThreadsForMessageStack option (options.ts). Use beforeSend instead.
  • bufferSize on ReactNativeTransportOptions (options.ts). Use the root maxQueueSize option instead.
  • instrumentLangGraph re-export (index.ts). Use instrumentStateGraph instead.
  • startTimeToInitialDisplaySpan / startTimeToFullDisplaySpan (tracing/timetodisplay.tsx). Use the <TimeToInitialDisplay> / <TimeToFullDisplay> components instead.
  • Internal captureAppStart (tracing/integrations/appStart.ts). Use Sentry.appLoaded() instead.
Tracked separately
  • #5882: _experiments.replaysSessionSampleRate / replaysOnErrorSampleRate
  • #6681: _experiments.androidProfilingOptions
  • #6680: enableExperimentalViewRenderer
  • #5932: FeedbackWidget* aliases
  • #5896: FeedbackButton, showFeedbackButton, hideFeedbackButton
  • #6143: TTID/TTFD record prop
  • #6001: sendDefaultPii
Follow-up
  • Remove the related // TODO: No longer an experiment on major version. notes in test/sdk.test.ts.
  • Document each removal in the v8 → v9 migration guide (#6735).
  • Consider a check that flags @deprecated APIs older than one major, so this list doesn't build up again.
Vorherrschende Sprache
TypeScript
Sterne
1.8k
Forks
373
Ø Merge
1 T. 7 Std.
Gemergte PRs (30 T.)
106

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus getsentry/sentry-react-native

Alle Issues in getsentry/sentry-react-native

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.