Consistent page metadata

[jerone]

Add consistent page metadata, containing title, description and keywords.

[jerone]

A few testcases:

• http://localhost:8080/users/jerone
  • Title: jerone | Users | OpenUserJS
  • Description: Download userscripts to enhance your browser.
  • Keywords: userscript, userscripts, javascript, Greasemonkey, Scriptish, Tampermonkey, extension, browser
• http://localhost:8080/?library=true&flagged=true
  • Title: Flagged Libraries | Moderation | OpenUserJS
  • Description: Download userscripts to enhance your browser.
  • Keywords: userscript, userscripts, javascript, Greasemonkey, Scriptish, Tampermonkey, extension, browser
• http://localhost:8080/scripts/DoctorSnowman/Rainymood_Cleanifier
  • Title: About | Rainymood Cleanifier | Scripts | OpenUserJS
  • Description: A script written by kurabiyecanavari: Clears out everything but the video on
  • Keywords: userscript, userscripts, javascript, Greasemonkey, Scriptish, Tampermonkey, extension, browser, GM config, Brainfuck, Facegroup

---

ps. can anyone make me an admin on the dev database...

[Zren]

ps. can anyone make me an admin on the dev database...

done.

[jerone]

done

TY :)

[Martii]

Needs at least one undefined, null, etc check(s). Example: Exception thrown using this url e.g. 404 pages and #37

Missing at the following urls too:

• http://localhost:8080/users/Marti/scripts
• http://localhost:8080/users/Marti/comments

I would also recommend a different identifier name... something like pageMeta instead of metaData which makes more sense in the context of what it is doing... so it doesn't confuse future/contributors. You did type "page meta" into your commit message too. ;)

Other than these blocking issues this is going to be an eventual need for SEO management. :)

[jerone]

would you be willing to change that to "Site API Keys"...

Done.

Exception thrown using this url e.g. 404 pages

Fixed.

I would also recommend a different identifier name

Agreed. We also have scripts metadata and that might be confusing.

[Martii]

Should our page "breadcrumbs" nearly (not exactly in some areas I think) match? ... Just noticed "Auth Strategy API Keys" and "Site API Keys" seem a bit different. Kind of the same with "http://localhost:8080/users/Marti/scripts" showing "Marti | Users | OpenUserJS". Not blocking from me but probably important for SEO management down the line.

Thanks for the 400 fix.. I was worried I'd lose oujs - Meta View but you fixered it up well. :)

+1 up to this point... unless you want to address the question present in a new issue/pr. ;)

[jerone]

I think more title 'breadcrumbs' could be improved, but I haven't looked at all of them in detail. This PR contains the logic to make it easier and consistent to adjust as needed.

[Martii]

make it easier and consistent to adjust as needed.

Alrighty... already edited with my vote above... kind of figured I might be reaching a bit. Might want to slap a PR READY label on if you think it's ready. No conflicts detected yet with other PR READYs in a test merge here. :)