Closed nickkeesG closed 4 months ago
Name | Link |
---|---|
Latest commit | b9c77cad2ad0b46357bd7b04f62a93a5c10e08fe |
Latest deploy log | https://app.netlify.com/sites/gleeful-biscuit-343233/deploys/66474dbc2783dd0008a7bc67 |
Deploy Preview | https://deploy-preview-65--gleeful-biscuit-343233.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
Lighthouse |
1 paths audited Performance: 65 (🟢 up 1 from production) Accessibility: 95 (no change from production) Best Practices: 100 (no change from production) SEO: 92 (no change from production) PWA: 80 (no change from production) View the detailed breakdown and full score reports |
To edit notification comments on pull requests, go to your Netlify site configuration.
Awesome work on breaking the PR down into smaller bits :)
Thoughts/notes based on testing on Netlify:
The hint should be updated:
IDK if this is relevant to this PR but the first and second comment I got were worse than usual:
Will look at the code now
I've updated the hint.
NO CHANGES TO REDUX STATE
Changes:
Note that I've changed the column names. I've made sure the names are backwards compatible, but would imagine us deprecating that compatibility in the future.
Also I looked it up and it seems that despite looking really ugly, this is pretty standard: myArray[Math.floor(Math.random() * myArray.length)]