Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Tracker.afterRun and fixed onRendered callbacks #5392

Closed
wants to merge 2 commits into from

Conversation

mitar
Copy link
Contributor

@mitar mitar commented Oct 8, 2015

Fixes #4410 by implementing #5354.

@dgreensp
Copy link
Contributor

I just left a comment in #4410. On the PR, ideally we would solve this without a new Tracker API method.

@mitar
Copy link
Contributor Author

mitar commented Oct 13, 2015

That Tracker API method is then useful for me also for server-side autorun.

That is why I think it is necessary/useful to have it. And can solve both problems.

@zol zol removed the backlog label May 3, 2016
@benjamn
Copy link
Contributor

benjamn commented May 3, 2016

Thanks for this PR, and sorry we never got around to merging it. We are currently cleaning up the repository and removing old PRs; if you think this PR still has value and are interested in resurrecting it, we encourage you to re-submit it against current devel.

@benjamn benjamn closed this May 3, 2016
@mitar
Copy link
Contributor Author

mitar commented May 3, 2016

This should probably go against https://github.com/meteor/blaze now, no?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants