Skip to content

[NoQA] Revert "[Sentry] Enhance OpenReport metrics span with new attributes"#75781

Merged
mountiny merged 1 commit intomainfrom
revert-75172-sentry/open-report-2
Nov 21, 2025
Merged

[NoQA] Revert "[Sentry] Enhance OpenReport metrics span with new attributes"#75781
mountiny merged 1 commit intomainfrom
revert-75172-sentry/open-report-2

Conversation

@rlinoz
Copy link
Copy Markdown
Contributor

@rlinoz rlinoz commented Nov 21, 2025

Reverts #75172

@rlinoz rlinoz self-assigned this Nov 21, 2025
@rlinoz rlinoz requested review from a team as code owners November 21, 2025 18:08
@melvin-bot melvin-bot bot requested review from JmillsExpensify and inimaga and removed request for a team November 21, 2025 18:08
@melvin-bot
Copy link
Copy Markdown

melvin-bot bot commented Nov 21, 2025

@inimaga Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@melvin-bot melvin-bot bot removed the request for review from a team November 21, 2025 18:09
@mountiny
Copy link
Copy Markdown
Contributor

Straight revert just to make it easier to get this to staging #75713 we will re-introduce this in a bit

@mountiny mountiny merged commit 78666bb into main Nov 21, 2025
31 of 33 checks passed
@mountiny mountiny deleted the revert-75172-sentry/open-report-2 branch November 21, 2025 18:09
@melvin-bot
Copy link
Copy Markdown

melvin-bot bot commented Nov 21, 2025

@mountiny looks like this was merged without a test passing. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.

@mountiny
Copy link
Copy Markdown
Contributor

Straight revert

@mountiny mountiny changed the title Revert "[Sentry] Enhance OpenReport metrics span with new attributes" [NoQA] Revert "[Sentry] Enhance OpenReport metrics span with new attributes" Nov 21, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 21, 2025

Codecov Report

❌ Looks like you've decreased code coverage for some files. Please write tests to increase, or at least maintain, the existing level of code coverage. See our documentation here for how to interpret this table.

Files with missing lines Coverage Δ
src/CONST/index.ts 85.84% <ø> (ø)
src/components/LHNOptionsList/OptionRowLHN.tsx 95.52% <ø> (ø)
.../MoneyRequestReportView/MoneyRequestReportView.tsx 0.00% <ø> (ø)
...ReportPreview/MoneyRequestReportPreviewContent.tsx 61.59% <ø> (-0.14%) ⬇️
...portActionItem/MoneyRequestReportPreview/index.tsx 93.10% <ø> (ø)
src/components/Search/index.tsx 34.35% <ø> (+0.08%) ⬆️
src/libs/telemetry/activeSpans.ts 100.00% <100.00%> (ø)
src/pages/home/ReportScreen.tsx 69.25% <ø> (-0.08%) ⬇️
src/libs/actions/IOU.ts 64.52% <0.00%> (ø)
src/libs/Navigation/Navigation.ts 56.09% <71.42%> (+0.61%) ⬆️
... and 7 files with indirect coverage changes

@OSBotify
Copy link
Copy Markdown
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify
Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by https://github.com/mountiny in version: 9.2.63-0 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

@OSBotify
Copy link
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/marcaaron in version: 9.2.63-8 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

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