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.

Sign up for GitHub

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

Jump to bottom

Set nodeId for redirect events for webentry #1674

Merged
merged 1 commit into from
Aug 27, 2024
Merged

Set nodeId for redirect events for webentry #1674

merged 1 commit into from
Aug 27, 2024

Conversation

nolanpro
Copy link
Contributor

@nolanpro nolanpro commented Aug 27, 2024 •
edited
Loading

@nolanpro nolanpro mentioned this pull request Aug 27, 2024
Merged
@nolanpro nolanpro requested a review from caleeli August 27, 2024 19:46
@cypress Cypress
Copy link

cypress bot commented Aug 27, 2024 •
edited
Loading

screen-builder    Run #1216

Run Properties:  status check passed Passed #1216  •  git commit e75ab0b538: Set nodeId for redirect events for webentry
Project screen-builder
Branch Review bugfix/FOUR-18851
Run status status check passed Passed #1216
Run duration 33m 12s
Commit git commit e75ab0b538: Set nodeId for redirect events for webentry
Committer Nolan Ehrstrom
View all properties for this run ↗︎
Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 19
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 296
View all changes introduced in this branch ↗︎

@ryancooley ryancooley merged commit 389a09b into next Aug 27, 2024
24 of 27 checks passed
@ryancooley ryancooley deleted the bugfix/FOUR-18851 branch August 27, 2024 20:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danloa danloa danloa approved these changes

@caleeli caleeli Awaiting requested review from caleeli

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nolanpro @danloa @ryancooley