-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Suggest description #1620
Suggest description #1620
Conversation
✅ Deploy Preview for expressjscom-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
We used to have |
Although the page doesn't display it directly, search engines do use it and show it to users when they search for Express. Also, now when a link is shared on any social media, the description will appear below the title. It may have lesser importance, but it can be improved to make the page a This would be a change that adds just one line, and the page infrastructure already supports it, so it wouldn’t hurt to add it. Making this change across all pages would be a long-term task (and of low relevance), but it could be useful for those who want to get started in the project by making these changes. |
LGTM. |
Since #1572 was merged into the main branch, the page now supports editing the description metadata, which is important for the page's SEO.
edit: I'm going to open an issue in the future to report that the description needs to be added to the pages