kadirahq / meteor-dochead

Isomorphic way to manipulate document.head for Meteor apps
MIT License
131 stars 17 forks source link

Does dochead work with Blaze in 1.2 #33

Open txs opened 8 years ago

txs commented 8 years ago

The dochead doesn't work for me in 1.2 with blaze.

In the head code, I set the default to run in flowrouter trigger enter and in router action. The head show the code 2 times. But in facebook debug, I just doesn't make it work!

maxenceC commented 8 years ago

Exact same thing here, blaze@2.1.7 :

krishaamer commented 8 years ago

Has this been resolved?

txs commented 8 years ago

Mine even with Google is not there. I think I have a issue with phantom in mup On May 21, 2016 6:22 AM, "Kris Haamer" notifications@github.com wrote:

Has this been resolved?

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/kadirahq/meteor-dochead/issues/33#issuecomment-220732460

Pagebakers commented 8 years ago

This will make it work with Blaze

FlowRouter.route('/post/:postId', {
  name: 'post',
  action(params) {
    if (Meteor.isClient) {
      BlazeLayout.render('layout')
    } else {
      Meteor.subscribe('posts', params.postId)
      if ((post = Posts.findOne(params.postId))) {
        DocHead.addMeta({
          property: 'og:image',
          content: post.image
        })
      }
    }
  }
})
txs commented 8 years ago

no!!! I rewrite it all with react router! Thanks BTW I'll try it On Jun 6, 2016 6:50 PM, "Eelco Wiersma" notifications@github.com wrote:

This will make it work with Blaze

FlowRouter.route('/post/:postId', { name: 'post', action(params) { if (Meteor.isClient) { BlazeLayout.render('layout') } else { Meteor.subscribe('posts', params.postId) if ((post = Uploads.findOne(params.postId))) { DocHead.addMeta({ name: 'property="og:image"', content: post.image }) } } } })

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/kadirahq/meteor-dochead/issues/33#issuecomment-223926433, or mute the thread https://github.com/notifications/unsubscribe/AAh1WXZQTWj2nqPrnDrwjCCKaN7iQpYVks5qI_togaJpZM4IWDbo .