-
Notifications
You must be signed in to change notification settings - Fork 64
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
[WEBSITE-405] Use plugin-specific title and description in response metadata #66
Conversation
server.js
Outdated
@@ -35,6 +35,10 @@ app.use(jsPath, express.static('./dist/client')); | |||
app.engine('hbs', exphbs({extname: '.hbs'})); | |||
app.set('view engine', 'hbs'); | |||
|
|||
const defaultPluginTitle = 'Jenkins Plugins'; | |||
const defaultPluginDescription = 'Jenkins – an open source automation server which enables developers around the world to reliably build, test, and deploy their software'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This description does not help in the case of plugins. I believe that almost all plugins already set a kind of description, but we might want to have a somethinf more explicit. Not a blocker
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The description comes from the index.jelly so it's required AFAIK, this is used on pages that aren't plugins, i.e. at least the home page.
Happy to reword it, but this is what it was before
Co-Authored-By: Oleg Nenashev <o.v.nenashev@gmail.com>
Co-Authored-By: Oleg Nenashev <o.v.nenashev@gmail.com>
I will try to land it today |
@timja @oleg-nenashev does title change on navigation for you? |
no it doesn't struggling to fix it though, |
Fixed |
now the title is updated on navigation to a plugin, but not when you navigate back with the "fid plugins" link |
Seems to work for me? i.e. I did:
|
If I start by visiting the index page, it works |
Fixed thanks for the testing |
@timja looks good now 🎉 |
Downstream of jenkins-infra/jenkins.io#2674
Related to issue # WEBSITE-405
Summary of this pull request:
SEO fixes to use specific title, and description
cc @halkeye @zbynek @oleg-nenashev