Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ping ApplauseLeads when PRs get CPed #4414

Merged
merged 12 commits into from
Aug 20, 2021
Merged

Ping ApplauseLeads when PRs get CPed #4414

merged 12 commits into from
Aug 20, 2021

Conversation

thienlnam
Copy link
Contributor

@thienlnam thienlnam commented Aug 5, 2021

Details

Will ping Applause leads when a PR gets cherry picked

Fixed Issues

$ #4212

QA Steps

CP a PR to staging.

Make sure you see the message
The PR title did not include [No QA], so this likely is a CP that requires QA Expensify/applauseleads

@thienlnam thienlnam self-assigned this Aug 5, 2021
@thienlnam thienlnam changed the title ping applause Ping ApplauseLeads when PRs get CPed Aug 5, 2021
@thienlnam thienlnam marked this pull request as ready for review August 5, 2021 00:48
@thienlnam thienlnam requested a review from a team as a code owner August 5, 2021 00:48
@MelvinBot MelvinBot requested review from madmax330 and removed request for a team August 5, 2021 00:48
Copy link
Contributor

@roryabraham roryabraham left a comment

Choose a reason for hiding this comment

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

@thienlnam is it cool if we put this on HOLD until #4274 is merged and tested? It introduces some substantial changes to how this action works, and we're waiting for the next deploy checklist to test it.

@thienlnam
Copy link
Contributor Author

@roryabraham Sure sure cool with me 👍

@thienlnam thienlnam changed the title Ping ApplauseLeads when PRs get CPed [HOLD 4274] Ping ApplauseLeads when PRs get CPed Aug 5, 2021
@roryabraham roryabraham changed the title [HOLD 4274] Ping ApplauseLeads when PRs get CPed Ping ApplauseLeads when PRs get CPed Aug 11, 2021
@roryabraham
Copy link
Contributor

@thienlnam Taking this off hold now

@roryabraham
Copy link
Contributor

Files changed: 0

@thienlnam
Copy link
Contributor Author

Ready for review! cc @madmax330 @roryabraham

@roryabraham roryabraham merged commit 3e5d031 into main Aug 20, 2021
@roryabraham roryabraham deleted the jack-pingApplause branch August 20, 2021 01:41
@OSBotify
Copy link
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
Contributor

🚀 Deployed to production by @roryabraham in version: 1.0.88-2 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 failure ❌
🕸 web 🕸 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