#metabrainz

/

      • akshaaatt
        Also monkey, mayhem, another contributor had added the spotify player as well and we can control spotify listens from the app’s now playing bar
      • I plan to move it to BP soon but for now it works fine
      • BrainzGit
        [listenbrainz-android] 14akshaaatt opened pull request #171 (03main…fixes): Fixes App Start issue https://github.com/metabrainz/listenbrainz-andr...
      • monkey
        akshaaatt: I think I'm getting bad interactions between music players. I'm playing through Bandcamp app, but was using Spotify earlier today. The app shows the "Now playing" from Spotify rather than the actual track that is playing from bandcamp.
      • I do see a correct playing_now listen came in though. But no full listens this time
      • mayhem
        akshaaatt: ah, interesting.
      • jasje
        monkey: the app rn is only submitting playing nows
      • not getting them
      • monkey
        Right, but I've got Spotify disabled in the listening apps list, but it shows my locally playing track from spotify that isn't actually playing
      • mayhem
        jasje: I'm not getting now listens from the app. there was a menu from the circle slash button that allwed me to control which apps are enabled. is that gone?
      • listen app list. that.
      • akshaaatt
        jasje is right about that monkey. We need to establish a socket connection to show the playing nows from LB server
      • jasje
        it is still there
      • monkey
        mayhem: Be aware that we are having RMQ issues, so you might need to refresh your page
      • jasje
        monkey: smol steps
      • mayhem
        monkey: thanks!
      • maybe testing is better done after the consul migration.
      • akshaaatt
        XD
      • monkey
        I don't get it. It's currently already showing "Now playing" on the app, but from an app that 1. isn't currently playing (paused) and 2. isn't enabled in the listening apps list
      • mayhem
        jasje: you said the listening apps list is still there? where?
      • monkey
        mayhem: Profile tab( last one)
      • jasje
        mayhem: the same stop sign
      • on profile
      • FAB
      • monkey
        Oops, app crashed
      • jasje
        monkey: aah i know that one
      • mayhem
      • monkey
        Exactly at the end of the song
      • mayhem
        listens apps should be on this screen, yes?
      • jasje
        mayhem: no access token filled
      • akshaaatt
        XD
      • That’s why I moved that access token to profile page. Thinking it could ease the process
      • mayhem
        it says it was fetched from the site.
      • akshaaatt
        But ugh. We’ll have to redo it
      • No it wasn’t mayhem. You’ll have to add it manually
      • jasje
        akshaaatt a dialog for handling this stuff would be better
      • akshaaatt
        But when to show that dialog?
      • mayhem
        oh, I fully misunderstood that. the startup wizard made it seem like it loaded it.
      • jasje
        akshaaatt: we'll just show a small prompt of profile if listens service hasn't been enabled
      • prompt on*
      • when filled, just remove that prompt
      • now playing and listens only then
      • akshaaatt
        I mean, we can’t pester them when they are in
      • The app doesn’t only have listens submissions right
      • mayhem
        ok, listen apps menu is now there, since I manually added the token.
      • akshaaatt
        Yay!
      • jasje
        akshaaatt: currently we are pestering 100 times more with the big card in front
      • akshaaatt
        XD
      • monkey
        OK, listens are coming in now, looks to be working pretty nicely.
      • It did drop my first track however.
      • jasje
        also, i think there might be libraries which have made functions for settings screen
      • akshaaatt
        I’m looking forward to the new settings design from aerozol
      • jasje
        monkey: big mystery
      • akshaaatt
        Have added a ticket for him
      • monkey
        Good progress though, this is nice !
      • jasje
        akshaaatt: i don't think we need to trouble him for this small thing
      • it should be straight forward imo
      • akshaaatt
        How is it small?
      • jasje
        3 settings literally
      • akshaaatt
        I’m in favour of a settings screen like pano scrobbler
      • The bottom sheet
      • monkey
        Considering the back-and-forth on the settings design, I think it's worth asking.
      • akshaaatt
        But I don’t think that design fits well with meb
      • Yes monkey
      • Let’s see what aerozol brings up
      • monkey
        In the app, what is the strategy for when to push the listen? After how long?
      • jasje
        monkey: half listen complete
      • monkey
        OK, thanks
      • jasje
        akshaaatt: weird design choice buy them imo
      • if i see a bottom nav component i must see a screen
      • it is compact but that is because not much to it
      • because there is*
      • all the stuff is still hidden is a configure section still
      • in a*
      • the app is noice tho
      • some performance issues on my device
      • but ui wise its very aligned
      • BrainzGit
        [musicbrainz-server] 14reosarevok opened pull request #2963 (03master…MBS-12026): MBS-12026: Still change deleted editor votes if they lack privs https://github.com/metabrainz/musicbrainz-serve...
      • reosarevok
        bitmap ^ I *think* it was needed to change the array to an array ref, but let me know if I'm being dumb
      • monkey
        akshaaatt, jasje: Overall on a small listening session the listen submission worked pretty well. But it did drop y first and last listens (playin_now listens for those tracks did come through though)
      • jasje
        alright I’ll have a look
      • petitminion has quit
      • Pratha-Fish
        hey I wonder if there's an option to locally host test.musicbrainz.org? My internet is going out every 5 mins for some damn reason.. I've already moved away from wolf and I'm trying to develop everything locally 💀
      • BrainzGit
        [listenbrainz-android] 14akshaaatt merged pull request #171 (03main…fixes): Fixes App Start issue https://github.com/metabrainz/listenbrainz-andr...
      • petitminion joined the channel
      • akshaaatt
        The design is great tbh jasje. We are giving too much importance to settings. I’ve never even gone to settings pages of apps in years. I don’t even remember the last time I saw settings pages for whatsapp, instagram, or even spotify
      • Pratha-Fish
        is it just me or is everyone seeing this on test.musicbrainz.org? https://usercontent.irccloud-cdn.com/file/85AO8...
      • akshaaatt
        That’s why I don’t see the need for settings page tbh. Just onboard the user and take permissions from them at the get go
      • As for theme, just use the device theme
      • That would be the ideal case for me
      • jasje
        sounds good
      • akshaaatt
        But since people do want settings, let’s keep it something sleek and not use an entire activity/screen for settings
      • Pratha-Fish
        reosarevok: hey can you please confirm if you're getting the same error on test.metabrainz.org?
      • Lotheric_ has quit
      • Lotheric joined the channel
      • rozlav joined the channel
      • FichteFoll has quit
      • petitminion has quit
      • petitminion joined the channel
      • atj
        Pratha-Fish: yes, I see the same error
      • Pratha-Fish
        atj: any idea when it would be resolved?
      • FichteFoll joined the channel
      • atj
        Pratha-Fish: i would imagine it will be resolved during/after maintenance today at 15:00 UTC
      • outsidecontext
        akshaaatt, jasje: +1 for needing this settings primarily one time for onboarding. But it would be good if it can be changed afterwards, including disabling the listen submission again
      • BrainzGit
        [bookbrainz-site] 14MonkeyDo merged pull request #989 (03master…BB-740-work-types): BB-740: Extend and improve the list of work types https://github.com/metabrainz/bookbrainz-site/p...
      • Pratha-Fish
        atj thenks
      • mayhem wishes he had a DSH-yacc
      • mayhem
        you're a machine, lucifer.
      • !m lucifer
      • BrainzBot
        You're doing good work, lucifer!
      • BrainzGit
        [bookbrainz-site] 14MonkeyDo merged pull request #993 (03administration-system…adminSystemSchema): SQL: Schema Changes for Administration System https://github.com/metabrainz/bookbrainz-site/p...
      • petitminion_ joined the channel
      • mayhem
        lucifer: these popularity datasets is from MLHD+ and our listens? or just MLHD+?
      • petitminion has quit
      • monkey
        reosarevok: Just FYI (because it's been triple-reviewed already I figured I would leave you alone), first small PR for the BB admin project is merged: https://github.com/metabrainz/bookbrainz-site/p...
      • Early 2000's playlist right there mayhem
      • mayhem
        yep, very clearly MLHD+ data.
      • Pratha-Fish: ^^
      • reosarevok
        lol
      • Not just early 2000s, but early 2000s white indie guy
      • monkey: neat! :)
      • monkey
        Nailed it on the head reosarevok
      • 2000s indie/rock-but-not-too-hard white guy
      • reosarevok
        Also
      • atj
        i think there may be a problem with the caa indexer? possibly related to the consul change yesterday?
      • reosarevok
        Why the fuck is A Rush of Blood to the Head by Coldplay under 静寂の世界 ?
      • mayhem: is that an issue with canonicalisation? :D
      • atj
      • mayhem
        maybe a query error
      • Pratha-Fish
        mayhem: MLHD is showing its age 🗿
      • BTW I wonder what use we might be able to get out of MLHD+ at this moment?
      • mayhem
        It's just another data bias, really.
      • That data may not be terribly useful. The others will be.
      • Pratha-Fish
        mayhem: like what I wonder? For music recommendations on listenbrainz maybe?
      • Oh which also reminds me, I'd be happy to try out my hand at listenbrainz recommendations after this GSOC project ends :D are there any specific skills I should pick up in advance for it?
      • mayhem
        Very useful for recs, that is why I asked lucifer to create them
      • No real new skills need to hack on troo. I'm happy to give you an intro when you are ready.
      • monkey
        troi*
      • yvanzo
        atj, cc bitmap: It's likely related, I stopped caa-indexer container again.
      • petitminion_ has quit
      • CatQuest
        [12:43] <monkey> Good band name
      • ^^Hard agree
      • :D
      • Strict Mayhem needs to legit be a thing lmao
      • mayhem
        it is. these two companies are now having to deal with it.
      • CatQuest
        akshaaatt: pls remember that *your* user experience is not universal.
      • ex *my* very first thing to do in any new app or prog is to look at settings! :|
      • monkey
        lucifer: Should I try to fix the commit history that I messed up the other day? https://github.com/metabrainz/metabrainz.org/pu...