#metabrainz

[listenbrainz-android] 14dependabot[bot] opened pull request #396 (03dev…dependabot/gradle/dev/compose_version-1.6.6): Bump compose_version from 1.6.5 to 1.6.6 https://github.com/metabrainz/listenbrainz-andr...

[listenbrainz-android] 14dependabot[bot] opened pull request #397 (03dev…dependabot/gradle/dev/io.sentry.android.gradle-4.5.0): Bump io.sentry.android.gradle from 4.4.1 to 4.5.0 https://github.com/metabrainz/listenbrainz-andr...

[listenbrainz-android] 14dependabot[bot] opened pull request #398 (03dev…dependabot/gradle/dev/androidx.datastore-datastore-preferences-1.1.0): Bump androidx.datastore:datastore-preferences from 1.0.0 to 1.1.0 https://github.com/metabrainz/listenbrainz-andr...

[listenbrainz-android] 14dependabot[bot] opened pull request #399 (03dev…dependabot/gradle/dev/androidx.compose-compose-bom-2024.04.01): Bump androidx.compose:compose-bom from 2024.04.00 to 2024.04.01 https://github.com/metabrainz/listenbrainz-andr...

[listenbrainz-android] 14dependabot[bot] opened pull request #400 (03dev…dependabot/gradle/dev/org.jetbrains.kotlin-kotlin-gradle-plugin-1.9.23): Bump org.jetbrains.kotlin:kotlin-gradle-plugin from 1.9.22 to 1.9.23 https://github.com/metabrainz/listenbrainz-andr...

[listenbrainz-android] 14dependabot[bot] opened pull request #401 (03dev…dependabot/gradle/dev/com.squareup.okhttp3-mockwebserver-5.0.0-alpha.14): Bump com.squareup.okhttp3:mockwebserver from 5.0.0-alpha.12 to 5.0.0-alpha.14 https://github.com/metabrainz/listenbrainz-andr...

[listenbrainz-android] 14dependabot[bot] opened pull request #402 (03dev…dependabot/gradle/dev/com.squareup.okhttp3-logging-interceptor-5.0.0-alpha.14): Bump com.squareup.okhttp3:logging-interceptor from 5.0.0-alpha.12 to 5.0.0-alpha.14 https://github.com/metabrainz/listenbrainz-andr...

[listenbrainz-android] 14dependabot[bot] opened pull request #403 (03dev…dependabot/gradle/dev/com.squareup.okhttp3-okhttp-5.0.0-alpha.14): Bump com.squareup.okhttp3:okhttp from 5.0.0-alpha.12 to 5.0.0-alpha.14 https://github.com/metabrainz/listenbrainz-andr...

[listenbrainz-android] 14dependabot[bot] opened pull request #404 (03dev…dependabot/gradle/dev/com.google.devtools.ksp-1.9.23-1.0.20): Bump com.google.devtools.ksp from 1.9.22-1.0.17 to 1.9.23-1.0.20 https://github.com/metabrainz/listenbrainz-andr...

Hi, is this the right place to ask for help with removing an auto-accepted edit I made on musicbrainz?

I wanted a promo release to an album so that both versions would show up but I think I did it wrong so I want to undo it all

jetpackjakson: probably more #musicbrainz, but sure. Link to the release group/album?

I probably wouldn’t remove it, just fix it, but let’s see :)

aerozol: just making sure I don't need to forward late reviews to you separately

(I mean community not admin)

reosarevok: oh, gotcha. Do me a favor though and give me a nudge if you see one for me, though! Because I just skim them

Well, there's one rimskii review from yesterday :)

Hmm, maybe I don’t get those emails, I can’t find that one

mayhem: can you add aerozol to community-manager@?

I'll forward this one

hmph that haker news link donna work for me

hacker*

monkey: hi! connecting spotify/other external services is broken at the moment because of a react router error. https://usercontent.irccloud-cdn.com/file/gYRLC...

Oh noes

Let me get on it

great, thanks!

i'll reply to the support email

reosarevok: moin. I'm trying to add aerozol, but I can't even find the community manager setup. google apps has too many options....

did you mean "musicbrainz-account-admin@" ?

but aerozol is already part of that.

No

I mean community-manager@, where the meeting reviews come

ah, I finally found it. its an alias for your account, not a group/.

which means I cannot add another person to it.

we can make a community-manager@ group and add you and aerozol to it.

Oh!

That would make sense

k

Because I assume this was an alias for freso before me

So it seems nicer to have a group rather than moving aliases around

this should be public, yes?

we could make it team only, if want to avoid spam.

I think it already is

Yeah

I haven't gotten any (non-automatically-filtered) spam to it, so probably safe to leave it as-is

did you just get an email from me?

non-meb account

mayhem: I did

then I think we're done.

just checking, right now the group is set to anyone on the team can read the messages. that cool?

lucifer: the user similarity still seems to have some problems: https://community.metabrainz.org/t/compatibilit...

[listenbrainz-server] 14MonkeyDo opened pull request #2847 (03master…missing-profile-routes): Fix missing profile routes https://github.com/metabrainz/listenbrainz-serv...

ApeKattQuest: I guess hacker news really did block me! I can’t see my post in incognito mode/not logged in either! Not the end of the world tbh

Got the test message, thanks mayhem, reosarevok

great. why did HN block you?

No idea. But because the only thing I’ve ever posted is MeB stuff, and I don’t do anything else there, maybe that was too ‘corporate shill’ for someone

and we're such corporate fuckwits too. sigh

[listenbrainz-server] 14MonkeyDo merged pull request #2847 (03master…missing-profile-routes): Fix missing profile routes https://github.com/metabrainz/listenbrainz-serv...

There is currently an annoying issue with M$ apt mirrors, breaking github actions builds for Picard (and much more), it may happen on others MeB repos using Github actions as well

See also https://github.com/microsoft/linux-package-repo...

[listenbrainz-server] release 03v-2024-04-24.0 has been published by 14MonkeyDo: https://github.com/metabrainz/listenbrainz-serv...

Deploying new LB release

OK, OAuth applications are back to working order lucifer (IRC)

Sent a support query to SoundCloud to change the redirect_uri, but in the meantime I re-established the missing routes and singing in works on porod

prod* Interestingly it only works on prod, but I think that was a problem before too because there is only a single redirect URI and none for beta.LB and test.LB so I don't think it ever worked there.

lachieh (IRC): logging into SoundCloud should be working again, please let me know if you have other issues

And thanks for reporting the issue :)

monkey: thank you for secondary nav change! I feel like I’m back in my cozy LB home 🥰

Me too !

Glad that's fixed

Was procrastinating from another job today, and wanted to do something fun. This was an idea that outsidecontext had at the last summit. What do you think mayhem? Image sizing needs work but the graphic is fun!

A crate for each project would be cool

(I fell into this while looking at fun mockups, a la the Picard box, to accompany the many many LB updates we’re going to be posting… but that’s proven harder to get right)

michael & marlon https://usercontent.irccloud-cdn.com/file/KLXVp...

monkey[m]: Hi! Since the react-query branch has been merged, i believe we're ready to merge the SPA feature branch. What do you think?

Hello!

We need to do another round of testing, but apart from that I'm not opposed, we can finish off the other PRs afterwrads and merge them in

i.e. the BrainzPlayer PR and the few refactors we need to do

Speaking of refactors... how do you feel about the idea of using wrappers for the class components fro the BP PR?

Hmm, it should work on most of the cases

Might be a good way to get BP merged with "finalized" features, and do the refactors on the heels of that

unless we're fetching the listens inside the component itself, and updating them based on user action

aerozol: super cute, I love it!

<3

Right, I guess in that case we will need to refactor

I’ll try go back to coming up with a LB image tomorrow, to make noise about Apple Music support!

Eyyy

ansh (IRC): I'll deploy the single-page-app branch to test.LB then

And we can do a round of testing everything

monkey[m]: Can you deploy the `spa-improvement` branch instead. I fixed a small bug in the login

Sure thing

The fault has been fixed. michael & marlon are back.

ansh (IRC): spa-improvement deployed to test.LB .

You take the explore section and I take the user dashboard section (or vice versa if you prefer?) and we test every button and action on the page (I guess except deleting listens, can try that on my alt account)?

Sure! I'll test the explore section

K

mayhem: i looked into it, i think i have found the issue. will fix.

monkey[m]: thanks!

!m lucifer

You're doing good work, lucifer!

ansh (IRC): Found an issue with the user dashboard, navigating using the date picker doesn't work (or more precisely, it works but the listens get overwritten straight away by the most recent ones)

I know where the issue comes from, will fix and push to spla-improvement

It's something I ran into while refactoring the user feed: storing the data from react-query in the state results in stale state in a bunch of cases. If we want to delete/modify items (for example deleting a listen) we need to either refetch or modify the query cache directly.

monkey[m]: I went through every button, just one small issue. Also LB-Radio is not working. I'm investigating that https://usercontent.irccloud-cdn.com/file/sd1dO...

ansh (IRC): the cover art stuff I think is because the css style for `#react-container` has `height:100vh` . `min-height: 100vh;` fixes the issue, but needs a check to see if that breaks any page

monkey[m]: Even on prod, these two examples of LB Radio give an error. So there is just one css change, which is also fixed now https://usercontent.irccloud-cdn.com/file/lR7dJ...

```LB Radio generation failed: Cannot fetch recordings for tags. {"code":400,"error":"pop_begin param is missing"}```

Ahh, I see, indeed that's not from the routing

mayhem (IRC): ^ error being thrown by LBRadio when using the examples on the page i.e. https://listenbrainz.org/explore/lb-radio/?prom...

artist and stats work fine, tags have issues

I also discovered two small issues which are there even on prod https://usercontent.irccloud-cdn.com/file/9EhPf...

nvm

the overview image on YIM 23 needs a bit fix. On copying and downloading the image, some text is overlapping the image

monkey[m]: yeah, tags are borked, understood, will fix early next week.

👍️

bitmap, yvanzo: MBS-13555 might be related with the issue we had with track name indices I guess?

MBS-13555: index row size exceeds btree version 4 maximum https://tickets.metabrainz.org/browse/MBS-13555

I understand the ctid their import gave won't work for us, so not sure what's the way if any to find that recording

The longest recording name I could find is 37960058

yeah looks like it's from the same "artist" as before

it's not the exact same issue though, since we do have this index in production. I'm guessing our btree versions differ and have slightly different size limits

Hmm. So we should tell them to change their btree size? Is that configurable?