Closed
Bug 1007853
Opened 11 years ago
Closed 11 years ago
webmaker.org Facebook linting fails spectacularly
Categories
(Webmaker Graveyard :: webmaker.org, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: akirose, Assigned: akirose)
References
()
Details
(Whiteboard: [june13])
Attachments
(1 file)
Since our mission is to reach as many people as possible, I /suppose/ we should try passing the facebook linter for webmaker.org with some improved meta tags.
| Assignee | ||
Updated•11 years ago
|
Whiteboard: [june13]
| Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8433572 -
Flags: review?(kate)
Comment 2•11 years ago
|
||
Comment on attachment 8433572 [details] [review]
https://github.com/mozilla/webmaker.org/pull/771
Looks good, one suggestion on the current location path. Yay sharing!
Attachment #8433572 -
Flags: review?(kate) → review+
| Assignee | ||
Comment 3•11 years ago
|
||
Comment on attachment 8433572 [details] [review]
https://github.com/mozilla/webmaker.org/pull/771
I had one more question for you k88 about preventing the duplicate content hit in search engine findings. Left at https://github.com/mozilla/webmaker.org/pull/771#discussion_r13395451 , didn't think to needinfo? you here. Pasted below.
I wanted to make sure the canonical was always `https` without the `www`. Actually, what I *really* want to do to render the canonical is `https://webmaker.org/{{ lang }}/{{ path }}` but I couldn't figure out how to get my hands on a path variable that is guaranteed not to start with lang. This is a problem because if we have two canonicals, one with and one without lang, we could get hit for duplicate content. I explored all the `$route` objects I could find though, and couldn't find what I was looking for. Open to suggestions?
Flags: needinfo?(kate)
Comment 5•11 years ago
|
||
Commits pushed to master at https://github.com/mozilla/webmaker.org
https://github.com/mozilla/webmaker.org/commit/9d31efb74efa2695e305874212e5d29109e55be4
fixes Bug 1007853 - webmaker.org Facebook linting fails spectacularly
https://github.com/mozilla/webmaker.org/commit/479a6e7201ba049124f11e33d3f9cf112940fd3e
Merge pull request #771 from gesa/1007853/share-meta
Bug 1007853 - webmaker.org Facebook linting fails spectacularly
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•