I'm sure there's a very good reason why we use masking to store privileges rather than having an easy to query "is_autoeditor" boolean, but gah
2019-02-12 04321, 2019
reosarevok
Makes it a pain to update all our autoeditor emails with an SQL query
2019-02-12 04344, 2019
reosarevok
Ended up copying all the names from the privileged editors page and just doing where name in, which seems like it shouldn't be the best way to do this :p
2019-02-12 04330, 2019
Mr_Monkey
CatQuest: I love the silly names. I'm also amused that on page 3 there's a bunch of Ed Sheeran songs for no good reason :D
2019-02-12 04331, 2019
CatQuest
yea i voted them down (since they aren't "sillyname") but i had to enlist reo to hlpe me vote thme down becasue some screwhead had upvoted them? :P
I'm still not convinced that we don't have any transient errors.
2019-02-12 04351, 2019
ruaok
pristine_: still reading.
2019-02-12 04358, 2019
pristine_
Okay:)
2019-02-12 04334, 2019
iliekcomputers
ruaok: everyone won't be, because we added new permissions to get currently playing as well. :(
2019-02-12 04312, 2019
ruaok
well, it is a beta. so people should expect that things are bumpy.
2019-02-12 04334, 2019
ruaok
I would just truncate the table with the spotify connection in it and update the beta server.
2019-02-12 04348, 2019
pristine_
Well, I need to make some changes to the PR at lb - server that pushes data in db. Would do by tonight.
2019-02-12 04357, 2019
iliekcomputers
ruaok: sounds good :)
2019-02-12 04321, 2019
iliekcomputers
If we don't truncate it, everyone would get an email saying they should unlink and link again
2019-02-12 04334, 2019
iliekcomputers
Maybe that is okay too?
2019-02-12 04307, 2019
ruaok
that's even better. lets do that then.
2019-02-12 04326, 2019
ruaok
pristine_: :)
2019-02-12 04319, 2019
pristine_
Yay!
2019-02-12 04346, 2019
pristine_
iliekcomputers , ruaok there might be cases when a existing user was not active the previous month in that case we need to remove his stats from statistics.user. To recognize such user , shouldn't we send a list of users with the lb-export zip file so that we can push an empty list in the producer and check it at the lb - server side.
2019-02-12 04356, 2019
pristine_
an*
2019-02-12 04327, 2019
ruaok
why do we need to remove their stats? we're not going to calculate them in the first place, no?
2019-02-12 04304, 2019
pristine_
If the user was active in Jan , in Feb his stats would be calculated for previous month and would be pushed in db. now, if he is inactive in Feb and we calculate stats in March for previous month, his stats should be removed ?
2019-02-12 04325, 2019
pristine_
Because when there should be stats of feb in db for his id, we still will have stats for jan for his id .
2019-02-12 04357, 2019
ruaok
for now, let's not worry about that.
2019-02-12 04302, 2019
ruaok
but, a good thought.
2019-02-12 04313, 2019
pristine_
Okay. I will note it some where
2019-02-12 04319, 2019
pristine_
For future implementation
2019-02-12 04331, 2019
pristine_
Thank you:)
2019-02-12 04313, 2019
ruaok
also, and this is really a nitpick, but we just created a gender neutral policy for our projects.
2019-02-12 04329, 2019
ruaok
"his stats" -> "their stats"
2019-02-12 04355, 2019
pristine_
Yes yes.
2019-02-12 04301, 2019
ruaok
if we can just not use gender when we don't need to, we don't end up leaving on group out.
2019-02-12 04314, 2019
pristine_
I was using singular.
2019-02-12 04315, 2019
ruaok
or confusion our communications by trying to use him/her in balanced way.
2019-02-12 04320, 2019
pristine_
But would keep in mind.
2019-02-12 04342, 2019
ruaok
unless we're speaking about gender directly, we can always find a way to write without using genders.
2019-02-12 04310, 2019
ruaok
again, no big deal, I don't mean to pick on you, but we as a team all would like to be as welcoming to everyone as we can possibly be.
2019-02-12 04310, 2019
pristine_
Yes. This is something that should be followed. Thank you for pointing out.
ruaok: a tweet or blog post telling people to relink their spotify accounts on beta would be great.
2019-02-12 04349, 2019
ruaok
on it
2019-02-12 04351, 2019
ruaok
"We continue to tweak our Spotify integration and in order to release the next set of improvements, we had to unlink all the accounts that have been previously linked. Sorry for the inconvenience! Please re-link your accounts to keep recording your listens! -rk"
2019-02-12 04351, 2019
ruaok
??
2019-02-12 04320, 2019
ruaok
"We continue to tweak our Spotify integration and in order to release the next set of improvements, we had to unlink all the accounts that have been previously linked to Spotify. Please re-link your accounts to keep recording your listens! Sorry for the inconvenience! -rk"
