Skip to content

Revert "fix: Self DM appears twice in Recents section" - #80257

Merged
blimpich merged 1 commit into
mainfrom
revert-79640-fix/78606
Jan 22, 2026
Merged

blimpich merged 1 commit into
mainfrom
revert-79640-fix/78606

Conversation

@blimpich

@blimpich blimpich commented Jan 22, 2026

Copy link
Copy Markdown
Contributor

@blimpich
blimpich requested review from a team as code owners January 22, 2026 19:11
@melvin-bot
melvin-bot Bot requested review from JmillsExpensify and MariaHCD and removed request for a team January 22, 2026 19:11
@melvin-bot

melvin-bot Bot commented Jan 22, 2026

Copy link
Copy Markdown

@MariaHCD 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]

@blimpich
blimpich merged commit f4c5139 into main Jan 22, 2026
30 of 33 checks passed
@blimpich
blimpich deleted the revert-79640-fix/78606 branch January 22, 2026 19:16
@melvin-bot

melvin-bot Bot commented Jan 22, 2026

Copy link
Copy Markdown

@blimpich 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.

@blimpich

Copy link
Copy Markdown
Contributor Author

Not emergency, straight revert to fix broken type check

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

optimisticData.push(
{
onyxMethod: Onyx.METHOD.SET,
key: `${ONYXKEYS.COLLECTION.REPORT}${optimisticReportID}`,
value: {

P2 Badge Keep SELF_DM_REPORT_ID in sync when creating self DM

This block creates a new optimistic self‑DM report but no longer updates ONYXKEYS.SELF_DM_REPORT_ID; since findSelfDMReportID() returns the cached value without validating it (ReportUtils.ts around 1903‑1905), any stale cached ID (e.g., from persisted Onyx after the report was removed/filtered) will keep being used and the newly created report is ignored. That can lead to repeated creation of additional self‑DM reports or lookups against a non‑existent report ID (e.g., convertBulkTrackedExpensesToIOU relies on findSelfDMReportID and then fetches actions by that ID), which manifests as duplicate self‑DM entries in Recents or missing tracked‑expense actions. Consider restoring the merge of SELF_DM_REPORT_ID when a new self‑DM report is created.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@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/blimpich in version: 9.3.7-0 🚀

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

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/Beamanator in version: 9.3.7-3 🚀

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

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