Closed ebma closed 3 months ago
Name | Link |
---|---|
Latest commit | ca6e58f6f8e07b3eb3c8385763b95a0ef94e58a9 |
Latest deploy log | https://app.netlify.com/sites/pendulum-pay/deploys/66accc7e199cd20008a6bb6e |
Deploy Preview | https://deploy-preview-72--pendulum-pay.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
@ebma is this blocked by any ticket? asking because no development in last 2 weeks
Yes, the change itself is very simple but it doesn't make sense to merge this now before the continuable flow is merged because then we need to change it again.
yarn.lock
changesStatus | Count |
---|---|
63 | |
21 | |
5 | |
156 |
Exactly, the tests are used to show that the squidrouter endpoint returns data for both currencies. If it didn't return a route for one of them, the token wouldn't be supported.
We replace jest with vitest, because jest was repeatedly causing issues due to the expected browser environment and the effort to configure it probably is too high. Vitest works out of the box and requires no extra configuration at all since we are already using vite in our project. However, our existing test for the NumericInput is somehow incompatible with the new environment. It's possible to configure vitest to run tests in a browser environment but somehow this still causes issues. Since the test for the NumericInput is not important, and we already have the same test in the portal, we delete it from this repository.
Closes #67.