Open franceslonsdale opened 2 weeks ago
Name | Link |
---|---|
Latest commit | 5cd1668bf768f3284add2c99b5bf84b463c599b3 |
Latest deploy log | https://app.netlify.com/sites/taskorial/deploys/66327cfd58c4a90008537980 |
Deploy Preview | https://deploy-preview-261--taskorial.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
Lighthouse |
1 paths audited Performance: 76 Accessibility: 98 Best Practices: 92 SEO: 93 PWA: 70 View the detailed breakdown and full score reports |
To edit notification comments on pull requests, go to your Netlify site configuration.
Thanks for opening a PR! I'll review it soon
Issue #217
What's being changed (it is reccomended to show visual proof of the changes if applicable)
I've sorted out the validation depth for 3 of the 8 occurrences (LandingPage.tsx, Privacy.tsx, Terms.tsx) and so the issue isn't fully resolved but I just wanted to push my current changes as I was struggling with reducing the depth of the JSX tree for some of the others with nesting depth 7 to 4 without changing the appearance of the page. Also I have a deepsource.toml file which is just so I could track if my changes were working so you don't need to merge this but I just didn't know how to not add this in with the pull request!
Check off the following: