Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Icon performance testing #3367

Closed
wants to merge 127 commits into from
Closed

Icon performance testing #3367

wants to merge 127 commits into from

Conversation

isaaclombardssw
Copy link
Member

  • Affected routes:

  • Fixed #{{ ISSUE NUMBER }}

  • If adding a new page, I have followed the 📃 New Webpage issue template

  • Include done video or screenshots

isaaclombardssw and others added 30 commits November 20, 2024 17:51
Copy link
Contributor

github-actions bot commented Dec 3, 2024

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

Copy link
Contributor

github-actions bot commented Dec 3, 2024

Coverage report

St.
Category Percentage Covered / Total
🔴 Statements 0.2% 69/33943
🔴 Branches 6.52% 24/368
🔴 Functions 0.86% 3/347
🔴 Lines 0.2% 69/33943

Test suite run success

13 tests passing in 1 suite.

Report generated by 🧪jest coverage report action from fd1a9cb

Copy link
Contributor

github-actions bot commented Dec 3, 2024

Deployed changes to https://app-sswwebsite-9eb3-pr-3367.azurewebsites.net

Copy link
Contributor

github-actions bot commented Dec 3, 2024

PageSpeed Insights

Category Score
Accessibility 95
Best practices 96
Performance 64
SEO 69

View the full report https://pagespeed.web.dev/report?url=https://app-sswwebsite-9eb3-pr-3367.azurewebsites.net

@isaaclombardssw
Copy link
Member Author

This was a flop :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants