#metabrainz

/

      • yvanzo
      • 2020-09-12 25620, 2020

      • _lucifer
        yeah it showed up after all release groups
      • 2020-09-12 25628, 2020

      • _lucifer
        also yvanzo, EPs are exported in a dump as well right?
      • 2020-09-12 25610, 2020

      • _lucifer
        i am trying to track down a bug in CB where all release groups are shown but EPs are missing
      • 2020-09-12 25646, 2020

      • yokel joined the channel
      • 2020-09-12 25649, 2020

      • yvanzo
        _lucifer: yup, it's included in sample dump
      • 2020-09-12 25640, 2020

      • _lucifer
        👍
      • 2020-09-12 25616, 2020

      • yokel has quit
      • 2020-09-12 25621, 2020

      • shivam-kapila
        pristine___: excited for the recs
      • 2020-09-12 25651, 2020

      • yokel joined the channel
      • 2020-09-12 25647, 2020

      • snazz
        Do we actually import listens from Spotify (as in past listening history) or do we only use Spotify's "scrobbling" functionality?
      • 2020-09-12 25608, 2020

      • snazz
        From what I've seen its only the later, but I may be incorrect...
      • 2020-09-12 25634, 2020

      • ishaanshah
        snazz: Yep spotify only allows the past 50 listens for an user
      • 2020-09-12 25647, 2020

      • ishaanshah
        So we cant import the whole history
      • 2020-09-12 25653, 2020

      • shivam-kapila
        Actually theres a workaround
      • 2020-09-12 25600, 2020

      • shivam-kapila
        But havent got to that
      • 2020-09-12 25616, 2020

      • snazz
        That's actually a good thing, it makes it much easier to word on the website what we are doing with Spotify
      • 2020-09-12 25623, 2020

      • pristine___
        shivam-kapila: react tests remaining. Scary 😱
      • 2020-09-12 25627, 2020

      • shivam-kapila
        The way we optimised spotify imports that can be used to import whole history
      • 2020-09-12 25646, 2020

      • shivam-kapila
        pristine___: jest is easier than react
      • 2020-09-12 25600, 2020

      • shivam-kapila
        pristine___: jest is easier than react
      • 2020-09-12 25620, 2020

      • pristine___
        Phew!
      • 2020-09-12 25655, 2020

      • yokel has quit
      • 2020-09-12 25642, 2020

      • yokel joined the channel
      • 2020-09-12 25645, 2020

      • snazz has quit
      • 2020-09-12 25633, 2020

      • yokel has quit
      • 2020-09-12 25633, 2020

      • MightyJay has quit
      • 2020-09-12 25643, 2020

      • MightyJay joined the channel
      • 2020-09-12 25626, 2020

      • yokel joined the channel
      • 2020-09-12 25607, 2020

      • ishaanshah
        iliekcomputers: #1082 is ready for review
      • 2020-09-12 25641, 2020

      • ishaanshah
        I'll open a separate PR for frontend so that we can deploy it a week after this PR has been deployed
      • 2020-09-12 25639, 2020

      • mckean has quit
      • 2020-09-12 25631, 2020

      • snazz joined the channel
      • 2020-09-12 25656, 2020

      • yokel has quit
      • 2020-09-12 25616, 2020

      • snazz
        lmao
      • 2020-09-12 25632, 2020

      • snazz
        I forgot that pytest made my laptop feel like it was taking off
      • 2020-09-12 25602, 2020

      • shivam-kapila
        any rust dev here
      • 2020-09-12 25614, 2020

      • snazz
        poor thinkpad is about to burn a hole in my lap
      • 2020-09-12 25625, 2020

      • BrainzGit
        [listenbrainz-server] tomnieuwland opened pull request #1086 (master…LB-686-user-information-cleanup): LB-686 Cleaned up a variety of user information pages https://github.com/metabrainz/listenbrainz-server…
      • 2020-09-12 25626, 2020

      • BrainzBot
        LB-686: Separate, obvious page for ways to scrobble and import listens https://tickets.metabrainz.org/browse/LB-686
      • 2020-09-12 25614, 2020

      • snazz
        iliekcomputers: By any chance do you remember the other newbie ticket that was floating around? Something about local time?
      • 2020-09-12 25616, 2020

      • shivam-kapila
        LB-642
      • 2020-09-12 25617, 2020

      • BrainzBot
        LB-642: Specify a field for "local time" for listens https://tickets.metabrainz.org/browse/LB-642
      • 2020-09-12 25626, 2020

      • shivam-kapila
        snazz: ^^
      • 2020-09-12 25638, 2020

      • snazz
        shivam-kapila: cheers! ^_^
      • 2020-09-12 25652, 2020

      • iliekcomputers
        snazz: thanks for the PR!
      • 2020-09-12 25655, 2020

      • snazz
        No worries. I went a little outside the scope of the original ticket, because I noticed a lot of information was either seemingly in the wrong spot on the website, or was being repeated elsewhere...
      • 2020-09-12 25609, 2020

      • snazz
        I'm sure it will make sense with the PR... I hope!
      • 2020-09-12 25644, 2020

      • shivam-kapila
        iliekcomputers: doesnt add to whitelist work for brainzbot now
      • 2020-09-12 25619, 2020

      • shivam-kapila
      • 2020-09-12 25620, 2020

      • BrainzBot
        LB-723: Labs recording MBID lookup API doesn't return anything for merged recordings
      • 2020-09-12 25628, 2020

      • yokel joined the channel
      • 2020-09-12 25613, 2020

      • snazz has quit
      • 2020-09-12 25601, 2020

      • sumedh has quit
      • 2020-09-12 25630, 2020

      • sumedh joined the channel
      • 2020-09-12 25623, 2020

      • yokel has quit
      • 2020-09-12 25609, 2020

      • pristine___
        shivam-kapila: ping
      • 2020-09-12 25626, 2020

      • shivam-kapila
        pristine___: pong
      • 2020-09-12 25604, 2020

      • pristine___
      • 2020-09-12 25602, 2020

      • pristine___
        `Object is possibly 'undefined'` , error for offset line 4 and listens line 8
      • 2020-09-12 25611, 2020

      • pristine___
        it works fine though.
      • 2020-09-12 25631, 2020

      • pristine___
        I don't understanf the error since i have defined both of them in props and state
      • 2020-09-12 25640, 2020

      • pristine___
        shivam-kapila: ^
      • 2020-09-12 25602, 2020

      • yokel joined the channel
      • 2020-09-12 25623, 2020

      • shivam-kapila
        1.
      • 2020-09-12 25623, 2020

      • shivam-kapila
        ```
      • 2020-09-12 25623, 2020

      • shivam-kapila
        if (offset && offset >= this.expectedRecommendationsPerPage) {
      • 2020-09-12 25623, 2020

      • shivam-kapila
        ```
      • 2020-09-12 25652, 2020

      • pristine___
        why so?
      • 2020-09-12 25608, 2020

      • shivam-kapila
        thats typescript at work
      • 2020-09-12 25617, 2020

      • pristine___
        ooo
      • 2020-09-12 25642, 2020

      • shivam-kapila
        in case offset is not defined it cant effectively check that inequalty
      • 2020-09-12 25612, 2020

      • pristine___
        and for listens?
      • 2020-09-12 25621, 2020

      • pristine___
        `listens?.slice` ?
      • 2020-09-12 25641, 2020

      • shivam-kapila
        ```
      • 2020-09-12 25641, 2020

      • shivam-kapila
        listens: listens?.slice(0, this.expectedRecommendationsPerPage),
      • 2020-09-12 25641, 2020

      • shivam-kapila
        ```
      • 2020-09-12 25612, 2020

      • shivam-kapila
        you got that already
      • 2020-09-12 25615, 2020

      • shivam-kapila
        :)
      • 2020-09-12 25649, 2020

      • pristine___
        `offset < (recommendationLength - this.expectedRecommendationsPerPage)`
      • 2020-09-12 25607, 2020

      • pristine___
        shivam-kapila: it says that right side should be a number etc
      • 2020-09-12 25623, 2020

      • pristine___
        so that means i cannot write a var in here?
      • 2020-09-12 25640, 2020

      • shivam-kapila
        didnt get ya
      • 2020-09-12 25653, 2020

      • pristine___
        `recommendationLength - this.expectedRecommendationsPerPage`
      • 2020-09-12 25612, 2020

      • pristine___
        `The right-hand side of an arithmetic operation must be of type 'any', 'number', 'bigint' or an enum type.`
      • 2020-09-12 25616, 2020

      • pristine___
        the error ^
      • 2020-09-12 25659, 2020

      • shivam-kapila
        can you pastebin the fxn plz
      • 2020-09-12 25616, 2020

      • shivam-kapila
        not sure about the aim
      • 2020-09-12 25633, 2020

      • pristine___
      • 2020-09-12 25650, 2020

      • pristine___
        recommendationLength = length of listen array.
      • 2020-09-12 25652, 2020

      • shivam-kapila
        double &&
      • 2020-09-12 25659, 2020

      • shivam-kapila
        line 5
      • 2020-09-12 25603, 2020

      • yokel has quit
      • 2020-09-12 25612, 2020

      • pristine___
        gaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaah
      • 2020-09-12 25623, 2020

      • pristine___
        don't look at me :p
      • 2020-09-12 25626, 2020

      • shivam-kapila
        🙈
      • 2020-09-12 25614, 2020

      • pristine___
        shivam-kapila: line 11, i am assigning a value to here, so ideally there shouldn't be any prob but it say `Type 'Listen[] | undefined' is not assignable to type 'Listen[]'`
      • 2020-09-12 25656, 2020

      • pristine___
        i understand why there should be a check in if else, but why a check when we are assigning value to something.
      • 2020-09-12 25625, 2020

      • yokel joined the channel
      • 2020-09-12 25631, 2020

      • shivam-kapila
        ```listens: listens?.slice(0, this.expectedRecommendationsPerPage) || [],
      • 2020-09-12 25632, 2020

      • shivam-kapila
        ```
      • 2020-09-12 25616, 2020

      • pristine___
        why so? even if it is an empty listen it will work
      • 2020-09-12 25625, 2020

      • pristine___
        i think
      • 2020-09-12 25612, 2020

      • shivam-kapila
        listens can be undefined
      • 2020-09-12 25639, 2020

      • shivam-kapila
        in that case it wont slice
      • 2020-09-12 25658, 2020

      • shivam-kapila
        and return undefined
      • 2020-09-12 25624, 2020

      • pristine___
        ah! but isn't it wow. I mean this strictness, little room for bugs
      • 2020-09-12 25602, 2020

      • yokel has quit
      • 2020-09-12 25620, 2020

      • shivam-kapila
        this is typescript magic
      • 2020-09-12 25629, 2020

      • shivam-kapila
        Js is weakly typed
      • 2020-09-12 25658, 2020

      • shivam-kapila
        typescript makes sure that we exploit the weekness properly
      • 2020-09-12 25609, 2020

      • shivam-kapila
        weakness* properlt
      • 2020-09-12 25656, 2020

      • pristine___
        shivam-kapila: a && a < b && b
      • 2020-09-12 25603, 2020

      • pristine___
        is this valid?
      • 2020-09-12 25628, 2020

      • shivam-kapila
        interview scenes lol
      • 2020-09-12 25650, 2020

      • shivam-kapila
        a && b && a < b is better IG
      • 2020-09-12 25643, 2020

      • shivam-kapila
        L to R associativity
      • 2020-09-12 25645, 2020

      • pristine___
        haha. interviews.
      • 2020-09-12 25605, 2020

      • pristine___
        they are scary
      • 2020-09-12 25622, 2020

      • pristine___
        for me atleast :p
      • 2020-09-12 25600, 2020

      • shivam-kapila
        me too
      • 2020-09-12 25636, 2020

      • yokel joined the channel
      • 2020-09-12 25659, 2020

      • sumedh has quit
      • 2020-09-12 25601, 2020

      • yokel has quit
      • 2020-09-12 25645, 2020

      • yokel joined the channel
      • 2020-09-12 25655, 2020

      • yokel has quit
      • 2020-09-12 25635, 2020

      • yokel joined the channel
      • 2020-09-12 25648, 2020

      • yokel has quit
      • 2020-09-12 25653, 2020

      • yokel joined the channel
      • 2020-09-12 25605, 2020

      • yokel has quit
      • 2020-09-12 25636, 2020

      • yokel joined the channel
      • 2020-09-12 25620, 2020

      • yokel has quit
      • 2020-09-12 25640, 2020

      • yokel joined the channel
      • 2020-09-12 25602, 2020

      • yokel has quit
      • 2020-09-12 25604, 2020

      • BrainzGit
        [listenbrainz-server] MonkeyDo opened pull request #1087 (master…follow-button-animation): Animate the follow button https://github.com/metabrainz/listenbrainz-server…
      • 2020-09-12 25643, 2020

      • yokel joined the channel
      • 2020-09-12 25650, 2020

      • piano_it_guy joined the channel
      • 2020-09-12 25637, 2020

      • yokel has quit
      • 2020-09-12 25600, 2020

      • v6lur has quit
      • 2020-09-12 25626, 2020

      • yokel joined the channel
      • 2020-09-12 25635, 2020

      • v6lur joined the channel
      • 2020-09-12 25616, 2020

      • yokel has quit
      • 2020-09-12 25653, 2020

      • yokel joined the channel
      • 2020-09-12 25605, 2020

      • yokel has quit
      • 2020-09-12 25625, 2020

      • yokel joined the channel
      • 2020-09-12 25605, 2020

      • yokel has quit
      • 2020-09-12 25655, 2020

      • yokel joined the channel
      • 2020-09-12 25655, 2020

      • yokel has quit
      • 2020-09-12 25656, 2020

      • yokel joined the channel
      • 2020-09-12 25605, 2020

      • yokel has quit