Closed karenroldan closed 1 year ago
@karenroldan -- can you try running the tests now? I got confetti mocked, and canvas
removed, so I think it should pass for you locally! 🤞
(Also, a couple tests later in the specs after the failures you were seeing needed some tweaking, so I just made those real quick 👍 ).
Oh, yay! The tests passed locally now! 🎉 Thank you so much for updating the tests! I'll go ahead and merge the changes.
What Does This PR Add/Change?
This PR updates the order of the steps. Previously, the
Type and Title
step is the second step, but now it will be the first step.There are no changes to the functionality itself, the flow should still work as expected!
Context: pciE2j-1VQ-p2#comment-1667
Testing Instructions
Canvas
node, I had to remove it from the packages. There are 2 failing issues related to the confetti, if you could pull the branch and runyarn test
I would appreciate it!Issues
Related to # https://github.com/Automattic/bugomattic/pull/50 Closes #