search

qri-io / qri

you're invited to a data party!
https://qri.io
GNU General Public License v3.0
1.11k stars 66 forks source link

fix(run): add ctx to run.Store #1893

Closed Arqu closed 3 years ago

Arqu commented 3 years ago

Also added an owner profile.ID to ListByStatus as cloud will have to scope it down and from a multi tenancy standpoint makes sense to control for it. And I want to avoid hacks we used to do where we magically pull out the relevant user from context.

Arqu commented 3 years ago

Updated the PR with a bit more stuff, the step.Duration can exceed a 32bit int and was giving me some trouble, so aligned that and also removed ctx from the AddEvent handler as that needs to run on a separate context from the event caller.