#metabrainz

/

      • HenryG has quit
      • 2020-08-05 21843, 2020

      • HenryG joined the channel
      • 2020-08-05 21828, 2020

      • supersandro2000 has quit
      • 2020-08-05 21837, 2020

      • supersandro2000 joined the channel
      • 2020-08-05 21831, 2020

      • supersandro2000 has quit
      • 2020-08-05 21852, 2020

      • supersandro2000 joined the channel
      • 2020-08-05 21820, 2020

      • Nyanko-sensei has quit
      • 2020-08-05 21801, 2020

      • Nyanko-sensei joined the channel
      • 2020-08-05 21814, 2020

      • Lotheric has quit
      • 2020-08-05 21859, 2020

      • djwhitey joined the channel
      • 2020-08-05 21802, 2020

      • MajorLurker has quit
      • 2020-08-05 21816, 2020

      • BestSteve has quit
      • 2020-08-05 21852, 2020

      • BestSteve joined the channel
      • 2020-08-05 21849, 2020

      • Gore has quit
      • 2020-08-05 21803, 2020

      • Gore joined the channel
      • 2020-08-05 21825, 2020

      • slriv has quit
      • 2020-08-05 21830, 2020

      • slriv joined the channel
      • 2020-08-05 21825, 2020

      • ishaanshah
        Morning!
      • 2020-08-05 21822, 2020

      • MajorLurker joined the channel
      • 2020-08-05 21805, 2020

      • white_shadow joined the channel
      • 2020-08-05 21809, 2020

      • sumedh joined the channel
      • 2020-08-05 21834, 2020

      • white_shadow has quit
      • 2020-08-05 21813, 2020

      • Nyanko-sensei has quit
      • 2020-08-05 21850, 2020

      • jmp_music_
        Morning!
      • 2020-08-05 21825, 2020

      • white_shadow joined the channel
      • 2020-08-05 21808, 2020

      • reosarevok
        ruaok, iliekcomputers: Someone wants to interview MeB for a Python podcast - obviously I'm not the right person for it given I have written like two lines of Python for MeB, but if you are interested see email ;)
      • 2020-08-05 21848, 2020

      • white_snack joined the channel
      • 2020-08-05 21837, 2020

      • white_shadow has quit
      • 2020-08-05 21845, 2020

      • diru1100
        Morning y'all!!
      • 2020-08-05 21829, 2020

      • diru1100
        I think there's some glitch in apple music app for Android, I am using it for free after my membership is over :)
      • 2020-08-05 21840, 2020

      • white_snack
        you used credit card for membership?
      • 2020-08-05 21853, 2020

      • sumedh has quit
      • 2020-08-05 21816, 2020

      • BrainzGit
        [bookbrainz-site] prabalsingh24 opened pull request #483 (UserCollection…test-add/remove-items): tests for `collection/add` and `collection/remove` https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2020-08-05 21822, 2020

      • diru1100
        white_snack: yup, but my membership expired at July 31st :)
      • 2020-08-05 21827, 2020

      • diru1100
        It's not working in MacOS as well, only in android
      • 2020-08-05 21801, 2020

      • Nyanko-sensei joined the channel
      • 2020-08-05 21850, 2020

      • shivam-kapila remembers my spotify premium is gonna expire
      • 2020-08-05 21858, 2020

      • alastairp
        ruaok: ah, right. if this code is completely standalone then I think it does make sense to have a separate image
      • 2020-08-05 21852, 2020

      • sumedh joined the channel
      • 2020-08-05 21834, 2020

      • ruaok
        moooin!
      • 2020-08-05 21853, 2020

      • ruaok
        alastairp: not only standalone, but also running on a different machine.
      • 2020-08-05 21804, 2020

      • alastairp
        and doesn't need any code from lb or msb?
      • 2020-08-05 21818, 2020

      • ruaok
        alastairp: iliekcomputers : do either of you want to take this podcast interview?
      • 2020-08-05 21827, 2020

      • ruaok
        alastairp: none, at all.
      • 2020-08-05 21836, 2020

      • alastairp
        perfect, keep it separate then
      • 2020-08-05 21843, 2020

      • alastairp
        forward me the email?
      • 2020-08-05 21851, 2020

      • Nyanko-sensei has quit
      • 2020-08-05 21809, 2020

      • prabal
        Mr_Monkey: hi
      • 2020-08-05 21829, 2020

      • prabal
        You mentioned about checking the error message in tests
      • 2020-08-05 21841, 2020

      • prabal
        for tests that are supposed to throw error
      • 2020-08-05 21853, 2020

      • prabal
        Right now i am only checking status code
      • 2020-08-05 21802, 2020

      • ruaok
        done.
      • 2020-08-05 21823, 2020

      • prabal
        but the thing is, whenever we catch an error, we do `next(err)` and it does not return a json response
      • 2020-08-05 21834, 2020

      • prabal
        rather it returns and html page with status code 500
      • 2020-08-05 21855, 2020

      • prabal
      • 2020-08-05 21829, 2020

      • prabal
        I think we should replace all the `next(err)` with this https://github.com/bookbrainz/bookbrainz-site/blo…
      • 2020-08-05 21841, 2020

      • prabal
        `error.sendErrorAsJSON`
      • 2020-08-05 21822, 2020

      • prabal
        then `res.body.error`will have the error we're looking for
      • 2020-08-05 21826, 2020

      • iliekcomputers
        ruaok: I'm happy to take it.
      • 2020-08-05 21825, 2020

      • ruaok
        alastairp how do you feel about it?
      • 2020-08-05 21828, 2020

      • Nyanko-sensei joined the channel
      • 2020-08-05 21829, 2020

      • Darkloke joined the channel
      • 2020-08-05 21811, 2020

      • sumedh has quit
      • 2020-08-05 21824, 2020

      • sumedh joined the channel
      • 2020-08-05 21841, 2020

      • ruaok
        iliekcomputers: did you see my questions from last night above? about the same/separate image?
      • 2020-08-05 21830, 2020

      • iliekcomputers
        I did not. Let me read the backlog
      • 2020-08-05 21842, 2020

      • iliekcomputers
        This is the messybrainz-labs code?
      • 2020-08-05 21856, 2020

      • ruaok
        yes
      • 2020-08-05 21839, 2020

      • iliekcomputers
        Do you think it'll eventually get merged into LB, given that messybrainz is in LB already?
      • 2020-08-05 21840, 2020

      • ruaok
        I've already got it working as a separate image and it would be easy, I think, to make a cron image out of it.
      • 2020-08-05 21854, 2020

      • ruaok
        yes, that is the whole point -- how do we do this?
      • 2020-08-05 21842, 2020

      • iliekcomputers
        I don't have strong opinions on whether it should be a different image or not. If piggybacking gives us some benefit, we should do it, otherwise standalone sounds good to me too.
      • 2020-08-05 21806, 2020

      • iliekcomputers
        However, we should merge the code into LB eventually
      • 2020-08-05 21812, 2020

      • ruaok
        ok, great. I'm hoping to have a PR today or tomorrow. its been lingering for too long.
      • 2020-08-05 21825, 2020

      • iliekcomputers
        Sounds good.
      • 2020-08-05 21827, 2020

      • legoktm has quit
      • 2020-08-05 21823, 2020

      • white_snack has quit
      • 2020-08-05 21801, 2020

      • Gazooo has quit
      • 2020-08-05 21826, 2020

      • legoktm joined the channel
      • 2020-08-05 21851, 2020

      • Gazooo joined the channel
      • 2020-08-05 21852, 2020

      • white_shadow joined the channel
      • 2020-08-05 21856, 2020

      • ruaok
      • 2020-08-05 21841, 2020

      • ruaok
        artist msid lookup is now live on labs.api -- please note that in this case the [artist_credit_mbids] was also renamed to [artist_credit_mbid] for consistency sake.
      • 2020-08-05 21804, 2020

      • ruaok
        please use this endpoint from now on.
      • 2020-08-05 21824, 2020

      • ishaanshah
        Cool, I will update it
      • 2020-08-05 21844, 2020

      • ishaanshah
        Thanks :)
      • 2020-08-05 21846, 2020

      • ruaok
        now I have a thicket of PRs to prepare.
      • 2020-08-05 21848, 2020

      • ruaok
        np!
      • 2020-08-05 21825, 2020

      • white_shadow has quit
      • 2020-08-05 21852, 2020

      • v6lur joined the channel
      • 2020-08-05 21854, 2020

      • white_shadow joined the channel
      • 2020-08-05 21853, 2020

      • iliekcomputers
        ruaok: Freso: can we reach out to this user to politely ask them to stop spamming the same artist every 2 minutes 24/7? https://listenbrainz.org/user/hds
      • 2020-08-05 21809, 2020

      • iliekcomputers
        1196 listens / day seems a bit much
      • 2020-08-05 21826, 2020

      • ruaok
        should we just log into the DB and change their submit key?
      • 2020-08-05 21825, 2020

      • iliekcomputers
        eh, i'd like to ping them once as a warning before disabling submissions etc.
      • 2020-08-05 21806, 2020

      • iliekcomputers
        also interested in how MusicBrainz handles cases like this, just outright ban or warning first?
      • 2020-08-05 21845, 2020

      • ruaok
        when its clearly spamming its delete account. if not clearly spamming, warn.
      • 2020-08-05 21849, 2020

      • ruaok
        Freso: over to you then.
      • 2020-08-05 21817, 2020

      • BrainzGit
        [bookbrainz-site] prabalsingh24 opened pull request #484 (UserCollection…test-deleteCollection): Test: Delete Collection https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2020-08-05 21802, 2020

      • ishaanshah
        ruaok: Internal server error for the POST endpoint
      • 2020-08-05 21826, 2020

      • ruaok
        😭
      • 2020-08-05 21832, 2020

      • shivam-kapila
        uh oh
      • 2020-08-05 21808, 2020

      • ruaok
        ahhh, the tests I am writing *just* caught this. will have it fixed in a few.
      • 2020-08-05 21821, 2020

      • ishaanshah
        thanks :)
      • 2020-08-05 21840, 2020

      • slriv has quit
      • 2020-08-05 21802, 2020

      • slriv joined the channel
      • 2020-08-05 21828, 2020

      • slriv has quit
      • 2020-08-05 21846, 2020

      • slriv joined the channel
      • 2020-08-05 21848, 2020

      • Mr_Monkey
        Hi prabal! I don't think we need to return `sendErrorAsJSON`, as I'm sure I've dealt with errors with a regular `next(error)` in the recent past. Let me look for it.
      • 2020-08-05 21840, 2020

      • Mr_Monkey
        prabal: Here it is:
      • 2020-08-05 21840, 2020

      • Mr_Monkey
        Where the error is thrown (using `next(err…)`): https://github.com/bookbrainz/bookbrainz-site/blo…
      • 2020-08-05 21840, 2020

      • Mr_Monkey
        And and example of testing for the error message: https://github.com/bookbrainz/bookbrainz-site/blo…
      • 2020-08-05 21803, 2020

      • Mr_Monkey
        See if that method works fo you. I'll have a quick check of your testing code to see if there's any obvious deal breaker
      • 2020-08-05 21841, 2020

      • ruaok
        ishaanshah: try again!
      • 2020-08-05 21854, 2020

      • sumedh has quit
      • 2020-08-05 21809, 2020

      • ishaanshah
        ruaok: Working!
      • 2020-08-05 21817, 2020

      • ruaok
        fabu. :)
      • 2020-08-05 21822, 2020

      • prabal
        Mr_Monkey: Yeah`resonpse.res.statusMessage` has the error message.
      • 2020-08-05 21835, 2020

      • prabal
        I was checking `response.body`earlier
      • 2020-08-05 21839, 2020

      • prabal
        it has html page
      • 2020-08-05 21815, 2020

      • Mr_Monkey
        Yeah, that's where I would have looked at it too.
      • 2020-08-05 21849, 2020

      • prabal
        yeah if it were returning json response then it would have been in response.body
      • 2020-08-05 21854, 2020

      • prabal
        I'll make that change
      • 2020-08-05 21819, 2020

      • BrainzGit
        [listenbrainz-server] mayhem opened pull request #1013 (master…labs-api): Add dataset hoster based labs.api.listenbrainz.org https://github.com/metabrainz/listenbrainz-server…
      • 2020-08-05 21819, 2020

      • Mr_Monkey
        OK. I'll start having a look at your newly opened PRs
      • 2020-08-05 21857, 2020

      • Darkloke has quit
      • 2020-08-05 21824, 2020

      • Darkloke joined the channel
      • 2020-08-05 21835, 2020

      • iliekcomputers
        ruaok: the PR looks good to me. Just a few nits, imo the files should be in `listenbrainz/labs/api` instead of `listenbrainz/api/labs` as i could see people thinking `listenbrainz/api` contains the actual API as well. Also, we should pin the dataset-hoster requirement to a git tag.
      • 2020-08-05 21849, 2020

      • kieto joined the channel
      • 2020-08-05 21857, 2020

      • ruaok
        I'm cool with the tag...
      • 2020-08-05 21840, 2020

      • ruaok
        And I see your point on the confusion.... But labs is under the api concept, not the other way around.
      • 2020-08-05 21853, 2020

      • iliekcomputers
        hmm
      • 2020-08-05 21808, 2020

      • ruaok
        Rename the top ish level dir to labs-api ?
      • 2020-08-05 21814, 2020

      • iliekcomputers
        maybe `listenbrainz/webserver/views/labs`
      • 2020-08-05 21836, 2020

      • ruaok
        Too hidden, I think.
      • 2020-08-05 21856, 2020

      • iliekcomputers
        that works too, but `labs_api` instead, so that it's a valid python name
      • 2020-08-05 21806, 2020

      • ruaok
        Yeah, ok.
      • 2020-08-05 21813, 2020

      • ruaok
        Will fix after lunch.
      • 2020-08-05 21815, 2020

      • iliekcomputers
        thanks!
      • 2020-08-05 21835, 2020

      • ruaok
        I'll add a read me too to point out that this is not the main api.
      • 2020-08-05 21827, 2020

      • Mr_Monkey
        prabal: I just noticed something: in all three open PRs with tests, you use `beforeEach` to set up the loggedInUser and request agent, but it should be `before` instead, since it only needs to be done once.
      • 2020-08-05 21824, 2020

      • prabal
        Oh yeah I'll change that
      • 2020-08-05 21817, 2020

      • Mr_Monkey
        Same for `afterEach`-> `after`
      • 2020-08-05 21856, 2020

      • kieto has quit
      • 2020-08-05 21845, 2020

      • white_shadow has quit
      • 2020-08-05 21858, 2020

      • slriv has quit
      • 2020-08-05 21841, 2020

      • slriv joined the channel
      • 2020-08-05 21814, 2020

      • Lotheric joined the channel
      • 2020-08-05 21819, 2020

      • iconoclasthero joined the channel
      • 2020-08-05 21824, 2020

      • Darkloke has quit