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

fix: production url #175

Merged
merged 1 commit into from
Jun 26, 2024
Merged

Conversation

jaipaljadeja
Copy link
Contributor

Resolves #174

Also did a lighthouse report and everything seems good to me except a few things firstly something we all forget and care less, which is adding more "Accessibility" obviously and The best practices score is less because it looks like the text area of monaco editor gets flagged as some field which don't allow user to paste anything.

image

I would also suggest to check Google Search Console, you might find some interesting insights for the website such as indexing of pages and much more and we can add keywords meta tag too.

@mazurroman mazurroman requested a review from barabanovro June 14, 2024 08:22
Copy link
Contributor

@barabanovro barabanovro left a comment

Choose a reason for hiding this comment

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

LGTM

@barabanovro barabanovro merged commit 8eddfa9 into walnuthq:main Jun 26, 2024
1 check passed
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.

Remove localhost from the source code
2 participants