Paul Dowman pauldowman

Organizations

OK GROW!
Paul Dowman
  • Charlie DeTar 54641d6
    Merge pull request #19 from dalloro/file_not_found_until_restart_for_…
  • Charlie DeTar 1ec3c82
    Merge branch 'master' of github.com:AdmitHub/meteor-buildpack-horse
  • 8 more commits »
Paul Dowman

Thanks! This also fixes #14. Note to @rgould to update the major version number since it's a breaking change.

Paul Dowman

See also #14.

Paul Dowman

Thanks. We'll check this out. Sounds like we're not converting to EJSON properly.

Paul Dowman

I tend to think it should be as the docs say, which will actually be a breaking change, what do you think @rgould? Definitely should bump the major…

Paul Dowman

Thanks @DanielDornhardt and @nickw, and sorry for the slow reply. We'll fix it!

Paul Dowman
Needs Facebook OAuth login
pauldowman commented on issue okgrow/analytics#9
Paul Dowman

The configuration for those services are meant to be on the client side, it's not an issue. They're not sensitive. You can look at any web page tha…

Paul Dowman
Move analyticsSettings outside of public
pauldowman commented on issue okgrow/analytics#9
Paul Dowman

I don't think that's correct. Only Meteor.settings.public is available on the client. http://docs.meteor.com/#/full/meteor_settings

Paul Dowman
Invalidation Issue
Paul Dowman

Oh, weird, you're right, this totally breaks it for all route changes after hot code push. The docs say "Note that this hook won't run again if the…

Paul Dowman

s/another had/another hash/ (Mobile autocorrect).

Paul Dowman

I'm pretty sure I tested it with scrolling to another had on the same page, but if I broke that then you're right, that would be a problem. I'll do…

pauldowman commented on issue okgrow/analytics#8
Paul Dowman

Awesome @themeteorchef, thanks!

Paul Dowman

@jakobloekke Thanks, I'll investigate.

Paul Dowman
Invalidation Issue
Paul Dowman

8e876df and version 0.0.7 should fix this, try it out and let me know if you're having any problems.

Paul Dowman
  • Paul Dowman 8e876df
    Use onRun instead of onAfterAction to run only once
pauldowman commented on pull request okgrow/analytics#7
Paul Dowman

Awesome, thanks! Will merge ASAP.

Paul Dowman

@erasaur Thanks for pointing this out (Sorry for the slow reply!) I think the fix is that we should be using onRun instead of onAfterAction. I'll t…

pauldowman commented on pull request okgrow/meteor-image-upload#3
Paul Dowman

:ok:

pauldowman commented on issue okgrow/analytics#8
Paul Dowman

Yeah it would be great to support that. @themeteorchef if you could even just tell us the names of the variables to look for for each service (and …

pauldowman commented on pull request okgrow/meteor-image-upload#3
Paul Dowman

We should at least log it though I think.

pauldowman commented on issue okgrow/analytics#2
Paul Dowman

Just curious, was that the solution? It was an old version? I'm not sure how that would happen unless it was a dependency of another package (or un…

Paul Dowman
Add support for configuration with environment variables
pauldowman commented on issue okgrow/analytics#3
Paul Dowman

I don't think that would work, environment variables aren't available on the client, so I'm going to close this. But if there was any change to how…

Paul Dowman
Paul Dowman

Assuming you've added a cert to the Heroku app and SSL works (or else you're just using the Heroku app.com domain) then you shouldn't need to do an…