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

Fixing homepage broken image #2363

Merged
merged 89 commits into from Apr 2, 2024
Merged

Fixing homepage broken image #2363

merged 89 commits into from Apr 2, 2024

Conversation

amankumarrr
Copy link
Member

@amankumarrr amankumarrr commented Mar 25, 2024

Affected Route: /
Fixed : #2362

  • Adding 384 width device support
  • Adding tests for the mobile viewport
  • Adding pipeline to run on daily basis to test images on Production

Adding device sizes array for SIZE props.
https://nextjs.org/docs/pages/api-reference/components/image#devicesizes

Affected Route: `/`
Fixed : #2362

Adding device sizes array for SIZE props.
https://nextjs.org/docs/pages/api-reference/components/image#devicesizes
Copy link

github-actions bot commented Mar 25, 2024

Coverage report

St.
Category Percentage Covered / Total
🔴 Statements 0.25% 69/27649
🔴 Branches 8.86% 24/271
🔴 Functions 1.2% 3/250
🔴 Lines 0.25% 69/27649

Test suite run success

13 tests passing in 1 suite.

Report generated by 🧪jest coverage report action from f0365f0

Copy link

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

Copy link

PageSpeed Insights

Category Score
Accessibility 90
Best practices 96
Performance 49
SEO 93

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

Copy link

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

Copy link

PageSpeed Insights

Category Score
Accessibility 90
Best practices 96
Performance 38
SEO 93

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

Copy link

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

Copy link

PageSpeed Insights

Category Score
Accessibility 90
Best practices 96
Performance 44
SEO 93

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

Copy link

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

Copy link

PageSpeed Insights

Category Score
Accessibility 90
Best practices 96
Performance 30
SEO 86

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

@amankumarrr amankumarrr marked this pull request as draft March 25, 2024 23:31
Copy link

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

Copy link

PageSpeed Insights

Category Score
Accessibility 90
Best practices 96
Performance 39
SEO 93

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

@amankumarrr amankumarrr marked this pull request as ready for review March 26, 2024 01:08
@amankumarrr amankumarrr requested review from Harry-Ross and removed request for wicksipedia March 26, 2024 01:09
Copy link

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

Copy link

PageSpeed Insights

Category Score
Accessibility 90
Best practices 96
Performance 37
SEO 93

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

@amankumarrr amankumarrr marked this pull request as draft March 26, 2024 01:42
Copy link

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

Copy link

PageSpeed Insights

Category Score
Accessibility 96
Best practices 96
Performance 29
SEO 86

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

Copy link

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

Copy link

PageSpeed Insights

Category Score
Accessibility 90
Best practices 96
Performance 39
SEO 93

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

Copy link

github-actions bot commented Apr 2, 2024

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

Copy link

github-actions bot commented Apr 2, 2024

PageSpeed Insights

Category Score
Accessibility 90
Best practices 96
Performance 34
SEO 93

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

Copy link
Member

@wicksipedia wicksipedia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let’s merge this
Can you add a pbi to review and improve

@amankumarrr
Copy link
Member Author

Let’s merge this Can you add a pbi to review and improve

Done - here is the issue #2398

@amankumarrr amankumarrr merged commit b78bc53 into main Apr 2, 2024
12 checks passed
@amankumarrr amankumarrr deleted the 2362-broken-images-homepage branch April 2, 2024 01:31
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.

🐛 Homepage - Broken images
2 participants