Bug Description
When a PDF report is generating and the user re-opens the download side sheet, the panel does not show the "Your report is being generated" warning notice. The Download report button is correctly disabled while a generation is in flight, but the accompanying notice, which explains why the button is disabled, was missed when the real generation flow was wired up. We already use this style of notice in other side sheets, so this is a case of rendering it while a report is generating.
Steps to reproduce
- Enable the
pdfGeneration feature flag with Search Console, Analytics 4, and PageSpeed Insights connected and holding data.
- Open the Site Kit dashboard and open the PDF export panel from the header.
- Keep all sections selected and click Download report. The panel closes and the report starts generating.
- While the report is still generating, re-open the PDF export panel from the header.
- See that the Download report button is disabled but no warning notice explains why.
Screenshots
Additional Context
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
- Re-opening the PDF export panel while a report is generating shows a warning notice reading "Your report is being generated. To create another report, please wait for the current download to complete."
- The Download report button stays disabled for as long as the notice is shown.
- The notice does not appear in the moment after clicking Download report while the side sheet is closing.
- If the report finishes downloading in the background while the panel is open, the notice clears and the Download report button becomes active again.
- The notice uses the standard warning notice style, consistent with the same notice in other side sheets.
- Behaviour is gated behind the
pdfGeneration feature flag.
Implementation Brief
Files to modify
Test Coverage
assets/js/components/pdf-export/PDFSectionsSelectionPanel/PanelContent.test.tsx - asserts the warning notice renders when the panel is open and isExporting() is true, does not render when the panel is closed while a generation is in flight, and disappears once exporting finishes.
assets/js/components/pdf-export/PDFSectionsSelectionPanel/index.stories.tsx - add a generating-state story with isExporting seeded true so the warning notice is captured by VRT.
QA Brief
- Enable the
pdfGeneration feature flag, and connect Search Console, Analytics 4, and PageSpeed Insights on a site that holds data.
- Open the Site Kit dashboard and click the Download PDF report icon in the header.
- Keep every topic selected and click the Download report button. Confirm that no notice appears while the panel closes.
- Click the Download PDF report icon again while the report still generates. Confirm that the panel holds a warning notice with the title Your report is being generated and the description To create another report, please wait for the current download to complete., and that the Download report button stays off while the notice shows.
- Leave the panel open until the report finishes downloading. Confirm that the notice clears on its own and the Download report button works again, with no reload.
Changelog entry
- Fix bug that could cause the PDF Report generation notice to disappear when the download panel is closed and opened again.
Bug Description
When a PDF report is generating and the user re-opens the download side sheet, the panel does not show the "Your report is being generated" warning notice. The Download report button is correctly disabled while a generation is in flight, but the accompanying notice, which explains why the button is disabled, was missed when the real generation flow was wired up. We already use this style of notice in other side sheets, so this is a case of rendering it while a report is generating.
Steps to reproduce
pdfGenerationfeature flag with Search Console, Analytics 4, and PageSpeed Insights connected and holding data.Screenshots
Additional Context
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
pdfGenerationfeature flag.Implementation Brief
Files to modify
assets/js/components/pdf-export/PDFSectionsSelectionPanel/index.tsx- pass the existingisOpenvalue down toPanelContentas a prop so the in-progress notice can gate on the panel being open, not on the close animation still running.assets/js/components/pdf-export/PDFSectionsSelectionPanel/PanelContent.tsx- deriveinFlightfromselect( CORE_PDF ).isExporting()andgetStatus() === 'progress'(mirroringFooter.tsx), and render aSelectionPanelNotice type={ NOTICE_TYPES.WARNING }with title "Your report is being generated" and description "To create another report, please wait for the current download to complete." whenisOpen && inFlight. Gating onisOpenprevents the flash:handleDownloadClicksets the open keyfalsein the same tick it callsstartExporting(), so the notice never renders during the slide-out. When the export finishes,finishExporting()clearsisExportingand movesstatusoffprogress, soinFlightgoes false and both the notice and the Footer's disabled button reset without extra wiring.Test Coverage
assets/js/components/pdf-export/PDFSectionsSelectionPanel/PanelContent.test.tsx- asserts the warning notice renders when the panel is open andisExporting()is true, does not render when the panel is closed while a generation is in flight, and disappears once exporting finishes.assets/js/components/pdf-export/PDFSectionsSelectionPanel/index.stories.tsx- add a generating-state story withisExportingseeded true so the warning notice is captured by VRT.QA Brief
pdfGenerationfeature flag, and connect Search Console, Analytics 4, and PageSpeed Insights on a site that holds data.Changelog entry