zas: hi! can you please add test.critiquebrainz service to gateways?
2024-09-09 25301, 2024
lucifer[m]1
also, there is a new database critiquebrainz_test_db that needs to be added pg_hba.conf on production databases but I guess we'll need bitmap for that?
lucifer: I didn't get around to working on the leaderboard page API call definitions yet (too much BS work atm). have you done any work on this front?
2024-09-09 25340, 2024
lucifer[m]1
[@mayhem:chatbrainz.org](https://matrix.to/#/@mayhem:chatbrainz.org) i didn't start yet because I figured we might want to do [@aerozol:matrix.org](https://matrix.to/#/@aerozol:matrix.org) 's suggestions first but can do before meeting if not
2024-09-09 25335, 2024
mayhem[m]
that suggestion is tricky on a number of fronts and will require more discussion at the summit. for now, we should proceed with the plans as laid out in the doc. let me finish off more stupid stuff and then take a stab at the API defs.
lucifer: ready to start thinking about API endpoints
2024-09-09 25326, 2024
mayhem[m]
should this page be paginated? if so, add count/offset args to the API endpoint?
2024-09-09 25315, 2024
lucifer[m]1
mayhem: hi! so we will have an endpoint to retrieve the most recent donors and the biggest donors? both paginated?
2024-09-09 25341, 2024
lucifer[m]1
do we also want to do historical leaderboards? top donors of last year etc?
2024-09-09 25344, 2024
mayhem[m]
paginated yes.
2024-09-09 25312, 2024
mayhem[m]
I am unsure about the top donors page -- since we agreed that we didn't want to show amounts, showing a top donor page without amounts seems... strange. no?
2024-09-09 25347, 2024
lucifer[m]1
oh i thought we wanted to show amounts.
2024-09-09 25358, 2024
lucifer[m]1
so its not a leaderboard? just a list of patrons?
2024-09-09 25357, 2024
mayhem[m] doesn't trust his bring
2024-09-09 25306, 2024
mayhem[m]
brain: brain. not bring.
2024-09-09 25323, 2024
mayhem[m]
so I double checked with monkey and we do want to show amounts.
2024-09-09 25326, 2024
mayhem[m]
so, then yes.
2024-09-09 25346, 2024
lucifer[m]1
right, iiuc. we settled on a recent donor lists and a leaderboard with amounts.
2024-09-09 25307, 2024
monkey[m] joined the channel
2024-09-09 25308, 2024
monkey[m]
We could still have a leaderboard without amounts, but it's not as clear nor as enticing
2024-09-09 25324, 2024
mayhem[m]
I was just confused as usual. :)
2024-09-09 25344, 2024
lucifer[m]1
since we are doing leaderboards, do we want historical ones as well?
2024-09-09 25305, 2024
mayhem[m]
historical in which sense?
2024-09-09 25319, 2024
lucifer[m]1
last year's, last month's etc.
2024-09-09 25339, 2024
mayhem[m]
for now, lets keep it simple and just recent donations and top donations, paginated.
2024-09-09 25349, 2024
lucifer[m]1
sure will do.
2024-09-09 25333, 2024
lucifer[m]1
about storing flair in database, i think a JSONB column in the "user" table would make sense.
2024-09-09 25343, 2024
mayhem[m]
yep.
2024-09-09 25302, 2024
lucifer[m]1
okay so endpoints to store and update the flair.
2024-09-09 25329, 2024
mayhem[m]
where does caching of that info happen?
2024-09-09 25337, 2024
lucifer[m]1
redis.
2024-09-09 25304, 2024
lucifer[m]1
actually, no need to cache it.
2024-09-09 25316, 2024
lucifer[m]1
we always read the "user" table for username.
2024-09-09 25328, 2024
lucifer[m]1
so reading flair is another column in the same query.
2024-09-09 25333, 2024
mayhem[m]
ah, ok.
2024-09-09 25342, 2024
lucifer[m]1
the donation endpoints will be cached in redis.
2024-09-09 25346, 2024
lucifer[m]1
updated hourly i guess.
2024-09-09 25338, 2024
mayhem[m]
ok, leaderboard API endpoints defined in the doc.
2024-09-09 25338, 2024
lucifer[m]1
cool will implement.
2024-09-09 25355, 2024
mayhem[m]
all 4 of those?
2024-09-09 25359, 2024
mayhem[m]
or just 2?
2024-09-09 25300, 2024
lucifer[m]1
sure
2024-09-09 25316, 2024
lucifer[m]1
all 4. can divide if you prefer.
2024-09-09 25336, 2024
mayhem[m]
it would be great if you can do all 4 -- this week is looking like a business heavy week for me.
2024-09-09 25350, 2024
lucifer[m]1
no worries, can do all.
2024-09-09 25301, 2024
mayhem[m]
I just realized that I really need to do Q3 finances at least a bit to arrive at a financial picture I can present at the summit.
2024-09-09 25304, 2024
mayhem[m]
great, thanks.
2024-09-09 25348, 2024
pite joined the channel
2024-09-09 25305, 2024
lucifer[m]1
mayhem: there is an issue with donations in MeB, only username is stored and not user id. so issues with donors who changed musicbrainz username.
2024-09-09 25357, 2024
mayhem[m]
feck.
2024-09-09 25335, 2024
mayhem[m]
well, if we change that and store UID as well, then LB leaderboard page would be fine going forward, yes?
2024-09-09 25358, 2024
lucifer[m]1
yes.
2024-09-09 25325, 2024
mayhem[m]
we should probably do that first then, I guess.
2024-09-09 25322, 2024
lucifer[m]1
we can handle usernames too for historical donors too. plus least create a copy script to copy over user ids for previous ones.
2024-09-09 25330, 2024
lucifer[m]1
mayhem[m]: sure
2024-09-09 25335, 2024
mayhem[m]
great -- I think don't worry about resolving historical donors who have renamed their accounts. lets deal with those by hand as they arives -- there may not be any. or at least very few.
ah, no. its german. I confuse swahili and german all the time.
2024-09-09 25350, 2024
reosarevok[m]
heh
2024-09-09 25304, 2024
reosarevok[m]
Ok, so each one of us gets the language we have on our user defined languages, as intended
2024-09-09 25305, 2024
reosarevok[m]
That's good
2024-09-09 25316, 2024
reosarevok[m]
Now, what happens with the LB request I dunno :)
2024-09-09 25327, 2024
reosarevok[m]
When logged out I get DE
2024-09-09 25347, 2024
mayhem[m]
🙈🙉🙊
2024-09-09 25348, 2024
reosarevok[m]
So I guess DE is just the default unless you have FR selected
2024-09-09 25314, 2024
monkey[m]
Ah, fair enough, thanks. In incognito mode I get the German version. So since we do the LB front-end call without a cookie attached, I suppose, it does not fetch my user info and the FR hint
2024-09-09 25359, 2024
reosarevok[m]
Yeah
2024-09-09 25325, 2024
reosarevok[m]
We could maybe modify the endpoint to only send English (or whatever the requested language) and return nothing otherwise, I guess?
2024-09-09 25338, 2024
reosarevok[m]
As in, having a param for language
2024-09-09 25352, 2024
reosarevok[m]
Since LB is more... eh, maybe dumbed down is not the right word 😅
2024-09-09 25358, 2024
reosarevok[m]
Slick-intended? :D
2024-09-09 25308, 2024
reosarevok[m]
Maybe the clash of languages is more jarring than for MB users
2024-09-09 25333, 2024
reosarevok[m]
"non-weirdo-friendly"? :p
2024-09-09 25345, 2024
monkey[m]
Not sure if no extract is better than an extract in another language, but it might be nice to let users know this is 'normal' as there is no article in $language