check if manually invalidated before removing computation

stubailo / meteor-reactive-method

Call methods synchronously inside Tracker.autorun [deprecated]

https://atmospherejs.com/simple/reactive-method

MIT License

98 stars 11 forks source link

check if manually invalidated before removing computation #11

Closed ecwyne closed 7 years ago

stubailo commented 9 years ago

Won't this fix cause us to ignore the Tracker flush cycle and rerun immediately rather than doing it asynchronously?

ecwyne commented 9 years ago

@stubailo I believe this will now work within the normal Tracker flush cycle and without deleting the computation if it's invalidated because of a ReactiveMethod.invalidateApply

ecwyne commented 9 years ago

any thoughts @stubailo ?

ecwyne commented 9 years ago

@stubailo this continues to be an issue for me in production. Any more thoughts?

ecwyne commented 9 years ago

@stubailo ?