Closed Bug 1007853 Opened 11 years ago Closed 11 years ago

webmaker.org Facebook linting fails spectacularly

Categories

(Webmaker Graveyard :: webmaker.org, defect)

x86
macOS
defect
Not set
normal

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: nobody → aki
Status: NEW → ASSIGNED
Whiteboard: [june13]
Depends on: 1019314
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+
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)
Ok, I get it -- what you did works fine ^_^
Flags: needinfo?(kate)
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
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.

Attachment

General

Created:
Updated:
Size: