Skip to content

[NO-QA]Add Workflow run URL to triggerWorkflowAndWait#7250

Merged
roryabraham merged 1 commit into
mainfrom
andrew-workflow-url
Jan 17, 2022
Merged

[NO-QA]Add Workflow run URL to triggerWorkflowAndWait#7250
roryabraham merged 1 commit into
mainfrom
andrew-workflow-url

Conversation

@AndrewGable

Copy link
Copy Markdown
Contributor

Details

Adds the workflow run URL to triggerWorkflowAndWait, which will be a small QOL improvement for us when viewing logs.

Fixed Issues

$ #7239

Tests

Since this code is deep in the weeds, it seems very hard to test this specific code without running some production code, therefore I would suggest:

  1. Merge this PR
  2. I will watch logs for the deploys and verify the URL appears correctly when we deploy, lock a checklist, or any other workflow that uses triggerWorkflowAndWait

@AndrewGable AndrewGable requested a review from a team as a code owner January 14, 2022 23:41
@AndrewGable AndrewGable self-assigned this Jan 14, 2022
@AndrewGable AndrewGable changed the title Add Workflow run URL to triggerWorkflowAndWait [NO-QA]Add Workflow run URL to triggerWorkflowAndWait Jan 14, 2022
@MelvinBot MelvinBot requested review from joelbettner and removed request for a team January 14, 2022 23:41

@joelbettner joelbettner left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍 LGTM

@roryabraham roryabraham merged commit ec7ab04 into main Jan 17, 2022
@roryabraham roryabraham deleted the andrew-workflow-url branch January 17, 2022 20:17
@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.

@AndrewGable

Copy link
Copy Markdown
Contributor Author

Confirmed this is working as intended, tests pass ✅

Screen Shot 2022-01-18 at 3 08 08 PM

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @roryabraham in version: 1.1.30-4 🚀

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

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @chiragsalian in version: 1.1.31-1 🚀

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

4 participants