#bookbrainz

/

      • vedik joined the channel
      • 2023-03-16 07514, 2023

      • vedik has quit
      • 2023-03-16 07514, 2023

      • kellnerd joined the channel
      • 2023-03-16 07509, 2023

      • RohanSasne joined the channel
      • 2023-03-16 07535, 2023

      • RohanSasne
        Hey monkey, should I add API endpoints examples in my proposal for the project "Import other open databases"?
      • 2023-03-16 07557, 2023

      • monkey
        Hi RohanSasne, yes, generally the more details the better !
      • 2023-03-16 07534, 2023

      • RohanSasne
        Cool, thanks for the quick reply:)  would come up with the proposal really soon !
      • 2023-03-16 07545, 2023

      • monkey
        Nice, eager to see it :)
      • 2023-03-16 07504, 2023

      • monkey
        kellnerd: Not a lot more to say about the ORM discussion, other than I want to avoid moving ORMs twice in a span of a year or two :D
      • 2023-03-16 07505, 2023

      • monkey
        But there is also the (increasingly) pressing matter of moving away from the old version of Knex
      • 2023-03-16 07558, 2023

      • monkey
        I think it would be an improvement to be on the sunset (or perhaps continuing, there is hope…) version of Objection over the current Bookshelf version.
      • 2023-03-16 07525, 2023

      • monkey
        So maybe it's worth the move anyway? It depends on how much time you think the move might take, roughly.
      • 2023-03-16 07539, 2023

      • monkey
        Either that, or deciding now to go for another ORM
      • 2023-03-16 07522, 2023

      • kellnerd
        I fully agree, maybe we could just use a fork of Bookshelf and bump the knex version ourselves as a short-term solution to give us more time to decide on a long-term solution?
      • 2023-03-16 07525, 2023

      • kellnerd
        I accidentally did that for Objection by the way, it only expects knex 0.95 but I had already installed 2.4 in my bookbrainz-data-js fork and it worked flawlessly :)
      • 2023-03-16 07557, 2023

      • monkey
        Ah, that's interesting! I was about to say that I would expect lots of compatibility issues between versions, but…
      • 2023-03-16 07553, 2023

      • monkey
      • 2023-03-16 07504, 2023

      • kellnerd
        There already is https://github.com/bookshelf/bookshelf/pull/2120 maybe we have luck with that
      • 2023-03-16 07537, 2023

      • RohanSasne has quit
      • 2023-03-16 07551, 2023

      • monkey
        Oh, well that looks easy
      • 2023-03-16 07537, 2023

      • monkey
        That would certainly be a good temporary improvement, fixing the vulnerability while we figure out what we want to do (which might involve waiting a bit to see what happens of Objection)
      • 2023-03-16 07545, 2023

      • kellnerd
        Would it be appropriate to fork this under the metabrainz GH org or does this not matter for a temporary solution?
      • 2023-03-16 07534, 2023

      • monkey
        How about I create it using the BookBrainz user? This used to host the main repos before being moved to the MeB org
      • 2023-03-16 07535, 2023

      • monkey
      • 2023-03-16 07546, 2023

      • kellnerd
        Hehe, keeping it away from the official repos ;) Sounds good!
      • 2023-03-16 07513, 2023

      • monkey
        For a temporary solution that makes sense to me :) We can always transfer it if we do end up keeping it longer
      • 2023-03-16 07532, 2023

      • monkey
        master branch only?
      • 2023-03-16 07513, 2023

      • monkey
        Yeah. All stale branches otherwise
      • 2023-03-16 07515, 2023

      • kellnerd
        Yeah, the others are 8+ years old
      • 2023-03-16 07507, 2023

      • kellnerd
        https://github.com/bookbrainz/CaliBBre <3 (probably outdated, but still...)
      • 2023-03-16 07520, 2023

      • monkey
        You should receive an invite
      • 2023-03-16 07528, 2023

      • kellnerd
        Thanks, accepted
      • 2023-03-16 07534, 2023

      • monkey
        👍
      • 2023-03-16 07527, 2023

      • kellnerd
        Are you already working with the fork? If not, I could prepare the necessary changes in all affected repos.
      • 2023-03-16 07528, 2023

      • kellnerd
        ^ monkey
      • 2023-03-16 07551, 2023

      • monkey
        Haven't done anything with the fork
      • 2023-03-16 07527, 2023

      • kellnerd
        (y)
      • 2023-03-16 07533, 2023

      • monkey
        And would very much appreciate you preparing for the replacement :)
      • 2023-03-16 07541, 2023

      • monkey
        Thank you
      • 2023-03-16 07558, 2023

      • monkey is wrestling with upgrades of LB test runner
      • 2023-03-16 07538, 2023

      • riya73 joined the channel
      • 2023-03-16 07558, 2023

      • riya73 has quit
      • 2023-03-16 07517, 2023

      • RohanSasne joined the channel
      • 2023-03-16 07525, 2023

      • RohanSasne has quit
      • 2023-03-16 07556, 2023

      • stoichy joined the channel
      • 2023-03-16 07525, 2023

      • aerozol joined the channel
      • 2023-03-16 07520, 2023

      • stoichy has left the channel
      • 2023-03-16 07517, 2023

      • kellnerd has quit