#metabrainz

/

      • BrainzGit
        [listenbrainz-server] 14amCap1712 merged pull request #2017 (03master…listen-count-fe-bug): Fix extraneous 0 in listen count card https://github.com/metabrainz/listenbrainz-server…
      • 2022-05-26 14607, 2022

      • alastairp
        if the place has no concerts, it might switch to the festival tab. but then if someone adds a concert then refreshing the same page will show the concerts tab
      • 2022-05-26 14620, 2022

      • Etua joined the channel
      • 2022-05-26 14632, 2022

      • alastairp
        so I wanted to add something that would be explicit to say "we're actually showing a different tab to the default"
      • 2022-05-26 14600, 2022

      • alastairp
        in this case, it means that no parameter always means the concerts tab (or a redirect)
      • 2022-05-26 14629, 2022

      • alastairp
        yes, it's a bit inefficient, because we gather all of the information for the page and then redirect (so another request, another set of queries, etc), but for the size of the site this is OK
      • 2022-05-26 14630, 2022

      • ansh
        Understood
      • 2022-05-26 14613, 2022

      • alastairp
        there are definitely other ways of doing it. If we had more javascript in the frontend we could automatically add the query parameter to the URL on the client side
      • 2022-05-26 14642, 2022

      • alastairp
        a slightly more efficient way would be to check related events at the beginning of the view and redirect before we do too many DB queries
      • 2022-05-26 14654, 2022

      • reosarevok
      • 2022-05-26 14622, 2022

      • bitmap
        huh. looks like some kind of internal error with chrome remote debugging
      • 2022-05-26 14639, 2022

      • bitmap
        I'll try running them again
      • 2022-05-26 14657, 2022

      • bitmap
        reosarevok: would it be possible to rebase https://github.com/metabrainz/musicbrainz-server/… ? (I see it's based on two prs, maybe merging the react-annotations one first would make it easier)
      • 2022-05-26 14601, 2022

      • reosarevok
        Yes, that was the plan, to rebase once those are merged
      • 2022-05-26 14644, 2022

      • skelly37 joined the channel
      • 2022-05-26 14646, 2022

      • hackerman joined the channel
      • 2022-05-26 14646, 2022

      • BrainzGit
        [sir] 14amCap1712 opened pull request #134 (03master…demock): Fix mocking of entity.build_entity_query https://github.com/metabrainz/sir/pull/134
      • 2022-05-26 14629, 2022

      • BrainzGit
        [critiquebrainz] 14alastair opened pull request #430 (03master…place-part-of-parts): Show both forward and backward place-place rels (Parts and Part of) https://github.com/metabrainz/critiquebrainz/pull…
      • 2022-05-26 14622, 2022

      • BrainzGit
        [critiquebrainz] 14alastair opened pull request #431 (03master…api-default-sort): CB-429: API: Default sort by published_on desc if no sort params provided https://github.com/metabrainz/critiquebrainz/pull…
      • 2022-05-26 14644, 2022

      • BrainzGit
        [musicbrainz-server] 14mwiencek merged pull request #2544 (03master…react-18): Update react, react-dom to v18 https://github.com/metabrainz/musicbrainz-server/…
      • 2022-05-26 14619, 2022

      • lucifer
        yvanzo: test is now passing in CI as well. i'll cleanup the PR and add more tests.
      • 2022-05-26 14656, 2022

      • Lotheric_ is now known as Lotheric
      • 2022-05-26 14650, 2022

      • skelly37 has quit
      • 2022-05-26 14622, 2022

      • KevlarNoir has quit
      • 2022-05-26 14637, 2022

      • KevlarNoir joined the channel
      • 2022-05-26 14620, 2022

      • SothoTalKer joined the channel
      • 2022-05-26 14602, 2022

      • KevlarNoir has quit