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

Add SetField Msg #493

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MattiaVerticchio
Copy link

PR Summary for issue #492

  • Added the previously commented SetField message
  • Implemented a new Cypress test
  • In cases where the form or field is not found, I insert the new value with status = NotVisited and submitAttempted = False.

Notes: I attempted to run the Cypress tests from form-submissions.cy.js on a fresh clone of the repository. However, the test that "logs in in with errors, then re-submits the form successfully" is not working due to an issue with the /login route. The error message I encountered was:

BackendTask.Env.expect was expecting a variable `SESSION_SECRET` but couldn't find a variable with that name.

I'm unsure why this issue is occurring, but the new test I implemented passes as expected.

Let me know if this works or if you'd like further adjustments! 😁

Copy link

netlify bot commented Oct 8, 2024

👷 Deploy request for elm-pages-todos pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 5271a83

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.

1 participant