#bookbrainz

/

      • vedik joined the channel
      • vedik has quit
      • kellnerd joined the channel
      • RohanSasne joined the channel
      • RohanSasne
        Hey monkey, should I add API endpoints examples in my proposal for the project "Import other open databases"?
      • monkey
        Hi RohanSasne, yes, generally the more details the better !
      • RohanSasne
        Cool, thanks for the quick reply:)  would come up with the proposal really soon !
      • monkey
        Nice, eager to see it :)
      • 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
      • But there is also the (increasingly) pressing matter of moving away from the old version of Knex
      • 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.
      • So maybe it's worth the move anyway? It depends on how much time you think the move might take, roughly.
      • Either that, or deciding now to go for another ORM
      • 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?
      • 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 :)
      • monkey
        Ah, that's interesting! I was about to say that I would expect lots of compatibility issues between versions, but…
      • kellnerd
        There already is https://github.com/bookshelf/bookshelf/pull/2120 maybe we have luck with that
      • RohanSasne has quit
      • monkey
        Oh, well that looks easy
      • 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)
      • kellnerd
        Would it be appropriate to fork this under the metabrainz GH org or does this not matter for a temporary solution?
      • monkey
        How about I create it using the BookBrainz user? This used to host the main repos before being moved to the MeB org
      • kellnerd
        Hehe, keeping it away from the official repos ;) Sounds good!
      • monkey
        For a temporary solution that makes sense to me :) We can always transfer it if we do end up keeping it longer
      • master branch only?
      • Yeah. All stale branches otherwise
      • kellnerd
        Yeah, the others are 8+ years old
      • https://github.com/bookbrainz/CaliBBre <3 (probably outdated, but still...)
      • monkey
        You should receive an invite
      • kellnerd
        Thanks, accepted
      • monkey
        👍
      • kellnerd
        Are you already working with the fork? If not, I could prepare the necessary changes in all affected repos.
      • ^ monkey
      • monkey
        Haven't done anything with the fork
      • kellnerd
        (y)
      • monkey
        And would very much appreciate you preparing for the replacement :)
      • Thank you
      • monkey is wrestling with upgrades of LB test runner
      • riya73 joined the channel
      • riya73 has quit
      • RohanSasne joined the channel
      • RohanSasne has quit
      • stoichy joined the channel
      • aerozol joined the channel
      • stoichy has left the channel
      • kellnerd has quit