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

feat: update the protocol to be able to flex logic based on project config #30594

Conversation

ryanthemanuel
Copy link
Collaborator

@ryanthemanuel ryanthemanuel commented Nov 9, 2024

Fixes #30560

Additional details

In order to flex behavior in the protocol based on certain config values, we updated the app/protocol interface to accept values from the project config.

Steps to test

How has the user experience changed?

PR Tasks

@ryanthemanuel ryanthemanuel self-assigned this Nov 9, 2024
cli/CHANGELOG.md Outdated Show resolved Hide resolved
Copy link

cypress bot commented Nov 13, 2024

cypress    Run #58402

Run Properties:  status check passed Passed #58402  •  git commit d117fe4e5e: fix ts
Project cypress
Branch Review ryanm/feat/add-various-config-values-to-protocol-interface
Run status status check passed Passed #58402
Run duration 23m 32s
Commit git commit d117fe4e5e: fix ts
Committer Ryan Manuel
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 7
Tests that did not run due to a developer annotating a test with .skip  Pending 1326
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 29400
View all changes introduced in this branch ↗︎
UI Coverage  46.11%
  Untested elements 189  
  Tested elements 166  
Accessibility  92.55%
  Failed rules  3 critical   8 serious   2 moderate   2 minor
  Failed elements 903  

@ryanthemanuel ryanthemanuel merged commit 0975b2f into develop Nov 13, 2024
83 of 85 checks passed
@ryanthemanuel ryanthemanuel deleted the ryanm/feat/add-various-config-values-to-protocol-interface branch November 13, 2024 21:42
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.

[Test Generation] - Send a subset of Cypress config data to the protocol
2 participants