simplyGits / simplyHomework

Education with a little more swag.
https://app.simplyHomework.nl/
Other
7 stars 1 forks source link

Fetch users asynchronously #305

Closed tomsmeding closed 7 years ago

tomsmeding commented 7 years ago

<Lieuwex> er staat daar Waitgroup.foreach users <Lieuwex> naja <Lieuwex> WaitGroup.foreach(users, functieblabla) <Lieuwex> moet dan <Lieuwex> users.forEach(functieblabla) zijn <Lieuwex> ¯\_(ツ)_/¯ <Lieuwex> alleen voor de users doen graag dan <Lieuwex> emails mag die asynchroon doen <Lieuwex> succes <Lieuwex> bij voorbaat dank :P <Lieuwex> <3

lieuwex commented 7 years ago

Closing this, and comitting it myself this afternoon: your forgot to do the same change to the messages cron job and the commit message should be 'synchronously' instead of 'asynchronously' 😉

But thanks for comitting it in the first place, since I just requested it out of the blue. 😊

tomsmeding commented 7 years ago

lieuwex commented 7 years ago

efe34218186591f890ea110e3b5bf97ae23d8534