-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add e2e tests #694
Add e2e tests #694
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Just some final nits
Co-authored-by: Loren Yu <[email protected]>
We've accumulated quite a few changes, do we want to apply these changes back to |
@rylew1 yeah definitely. I kind of assumed you were testing the changes, so yeah if you haven't yet then let's do one last final test to make sure it all works |
Sorry accidentally closed the PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yay 👏 🎉
Ticket
Resolves navapbc/template-application-nextjs#358
Changes
Context
Original working PR on nextjs test repo => [Issue #358]: Add e2e tests against PR preview env platform-test-nextjs#85
Since we update the PR on this
template-infra
repo, I checked that it still runs successfully onplatform-test-nextjs
here: [Demo]: Updated e2e tests with changes from template-infra platform-test-nextjs#86The docs PR check error I don't think will resolve until this is merged