Adding form submission video page to the tutorial #219
Conversation
| @@ -17,4 +17,5 @@ We have a lot of ground to cover, so let's get started! | |||
| | -- | -- | | |||
| | 1. {% url "Project setup" project-setup %} | {% fa fa-video-camera %} 3:52 | | |||
| | 2. {% url "Testing inputs" testing-inputs %} | {% fa fa-video-camera %} 7:17 | | |||
| | More tutorials to come! | | | |||
| | 3. {% url "Form submission and XHR stubbing" form-submission %} | {% fa fa-video-camera %} 10:21 | | |||
| | More tutorials to come! | | | |||
There was a problem hiding this comment.
Without this the table cell collapsed and didn't show the gray background like the left cell did.
source/_data/sidebar.yml
Outdated
| @@ -152,6 +152,7 @@ tutorials: | |||
| introduction-to-tutorials: introduction-to-tutorials.html | |||
| project-setup: project-setup.html | |||
| testing-inputs: testing-inputs.html | |||
| form-submission: form-submission.html | |||
There was a problem hiding this comment.
you need to ensure that these filenames are not generic enough that they won't conflict with ANY doc that's ever added in the future (hence why I renamed 'introduction').
The idea is that you can always link directly from cypress.on (https://on.cypress.io/form-submission). Just something to keep in mind while you're added pages.
Also, you should probably become familiar with the cypress.on repo. Mostly, this manifest.yml file needs to stay up to date with this sidebar.yml file (merge PR to develop, PR dev to prod, it auto deploys). This should be automated, but it isn't.
There was a problem hiding this comment.
Gotcha, good to know. I'll poke around that repo a bit.
I'll push another commit with this file renamed too.
|
|
||
| ## What's covered | ||
|
|
||
| We will implement form submission for our todo app, leveraging {% url `cy.server()` server %} and {% url `cy.route()` route %} to stub calls to our API. We will iterate on our test and implementation, focusing on the application's "happy path" first. Once our form is working, we'll use another stubbed XHR call to setup a failure scenario and implement the code to properly display an error message. |
There was a problem hiding this comment.
Yay, url tag helpers! We use these instead of regular markdown links because during build, the docs literally check every single url tag to ensure we haven't written any incorrectly.
|
Is this ready to be merged in? |
|
It should be fine. Since I pushed a commit that dismissed the original approval, I wanted to make sure we were good before merging it. I'll update the video permissions so it displays properly when this makes it to production. |
|
Okay I'm merging in now and will be pushing to prod soon. |
Closes #209