Skip to content

[CP Staging] Revert "[No QA] Change offline status when Pusher PINGPONG fails"#57680

Merged
MarioExpensify merged 1 commit intomainfrom
revert-57255-tgolen-pingpong-triggeroffline
Mar 3, 2025
Merged

[CP Staging] Revert "[No QA] Change offline status when Pusher PINGPONG fails"#57680
MarioExpensify merged 1 commit intomainfrom
revert-57255-tgolen-pingpong-triggeroffline

Conversation

@tgolen
Copy link
Contributor

@tgolen tgolen commented Mar 3, 2025

Reverts #57255

This is a straight revert. It was still taking clients offline a little too much so we need to discuss how to roll this out further.

@tgolen tgolen self-assigned this Mar 3, 2025
@tgolen tgolen requested a review from a team as a code owner March 3, 2025 15:47
@tgolen tgolen changed the title Revert "[No QA] Change offline status when Pusher PINGPONG fails" [CP Staging] Revert "[No QA] Change offline status when Pusher PINGPONG fails" Mar 3, 2025
@melvin-bot melvin-bot bot requested review from mollfpr and removed request for a team March 3, 2025 15:47
@melvin-bot
Copy link

melvin-bot bot commented Mar 3, 2025

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

@MarioExpensify MarioExpensify merged commit 0b5045d into main Mar 3, 2025
14 of 17 checks passed
@MarioExpensify MarioExpensify deleted the revert-57255-tgolen-pingpong-triggeroffline branch March 3, 2025 15:49
@melvin-bot melvin-bot bot added the Emergency label Mar 3, 2025
@melvin-bot
Copy link

melvin-bot bot commented Mar 3, 2025

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

@MarioExpensify
Copy link
Contributor

Not an emergency, it is a straight revert.

OSBotify pushed a commit that referenced this pull request Mar 3, 2025
…-triggeroffline

[CP Staging] Revert "[No QA] Change offline status when Pusher PINGPONG fails"

(cherry picked from commit 0b5045d)

(CP triggered by puneetlath)
@github-actions github-actions bot added the CP Staging marks PRs that have been CP'd to staging label Mar 3, 2025
@OSBotify
Copy link
Contributor

OSBotify commented Mar 3, 2025

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

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2025

🚀 Cherry-picked to staging by https://github.com/puneetlath in version: 9.1.7-2 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅
🤖🔄 android HybridApp 🤖🔄 cancelled 🔪
🍎🔄 iOS HybridApp 🍎🔄 success ✅

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2025

🚀 Deployed to production by https://github.com/puneetlath in version: 9.1.7-2 🚀

platform result
🤖 android 🤖 true ❌
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅
🤖🔄 android HybridApp 🤖🔄 failure ❌
🍎🔄 iOS HybridApp 🍎🔄 failure ❌

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2025

🚀 Cherry-picked to staging by https://github.com/puneetlath in version: 9.1.7-2 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅
🤖🔄 android HybridApp 🤖🔄 failure ❌
🍎🔄 iOS HybridApp 🍎🔄 success ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CP Staging marks PRs that have been CP'd to staging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants