feat: updates to download resources page #17
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Playwright e2e testsTo view traces locally, unzip the report and run: npx playwright show-report ~/Downloads/playwright-report |
| sections, | ||
| totalSections, | ||
| totalSectionsComplete, | ||
| } = useDownloadView(chat); |
There was a problem hiding this comment.
This diff is mostly changing the file name. But the introduction of this hook is the substantive change in this file
| {`${totalSectionsComplete} of ${totalSections} sections complete`} | ||
| </span> | ||
| </div> | ||
| {sections.map((section) => { |
There was a problem hiding this comment.
Also using this sections array rather than what was in the previous version
|
|
General changes and functionality of this fix are looking good. One thing I noted was that there was a request in Miro for a couple of icon changes but these don't seem to have been reflected in this PR: |
|
🎉 This PR is included in version 1.3.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Description
lessonDetails)Issue(s)
Fixes https://www.notion.so/oaknationalacademy/Update-descriptions-of-each-section-on-the-download-resources-page-164770c7a7fa419c94a8367cf64f3d8f?pvs=4
How to test
Screenshots
How it used to look (delete if n/a):
How it should now look: