Skip to content

Conversation

ShubhamOulkar
Copy link
Member

@ShubhamOulkar ShubhamOulkar commented Aug 18, 2025

See #1933 (comment)
closes #1804

image

@ShubhamOulkar ShubhamOulkar requested a review from a team as a code owner August 18, 2025 06:42
Copy link

netlify bot commented Aug 18, 2025

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit 2814d64
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/68b43f655784b00008c20550
😎 Deploy Preview https://deploy-preview-2040--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

github-actions bot commented Aug 18, 2025

🚦 Lighthouse Results (Mobile & Desktop)

URL Device Perf A11y Best Practices
/ mobile 🔴 62 🟢 100 🟢 96
/en/blog/posts.html mobile 🟠 76 🟢 96 🟢 96
/en/5x/api.html mobile 🔴 72 🟢 95 🟢 96
/ desktop 🟢 100 🟢 100 🟢 96
/en/blog/posts.html desktop 🟢 100 🟢 96 🟢 93
/en/5x/api.html desktop 🟢 97 🟢 95 🟢 96

@bjohansebas bjohansebas marked this pull request as draft August 19, 2025 03:31
@ShubhamOulkar ShubhamOulkar marked this pull request as ready for review August 26, 2025 15:05
@ShubhamOulkar ShubhamOulkar changed the title (WIP) feat : lang-picker feat : language picker Aug 26, 2025
@ShubhamOulkar ShubhamOulkar changed the title feat : language picker feat : add new language picker component Aug 26, 2025
DELAY=10

echo "Checking Netlify preview: $PREVIEW_URL"
for i in $(seq 1 $MAX_RETRIES); do
if curl -s --head --max-time 5 "$PREVIEW_URL" | grep "200 OK" > /dev/null; then
if curl -s -I "$PREVIEW_URL" | grep "HTTP/.* 200" > /dev/null; then
Copy link
Member Author

@ShubhamOulkar ShubhamOulkar Aug 31, 2025

Choose a reason for hiding this comment

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

Correcting header status string. Now lighthouse audit is not running because of wrong status checking.

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.

Language picker: Add identifying feature and remove/fix duplicate nav ids
1 participant