#metabrainz

/

      • Gentlecat
        not sure why we would need that
      • 2017-05-29 14953, 2017

      • psolanki
        It's like that in MB
      • 2017-05-29 14921, 2017

      • Gentlecat
        ok, but why?
      • 2017-05-29 14955, 2017

      • CatQuest
        Freso, reosarevok I'm publishing the blog then, \o wish me luck in not fudging it up :D
      • 2017-05-29 14915, 2017

      • Freso
        CatQuest: Can I give it a final look over in a sec?
      • 2017-05-29 14921, 2017

      • CatQuest
        uh ok?
      • 2017-05-29 14922, 2017

      • CatQuest
        :)
      • 2017-05-29 14923, 2017

      • Freso writing up meeting notes just now
      • 2017-05-29 14910, 2017

      • CatQuest goes make dinner :)
      • 2017-05-29 14925, 2017

      • psolanki
        Gentlecat: Just thought it would be better
      • 2017-05-29 14901, 2017

      • ferbncode
        CatQuest: you're welcome :D
      • 2017-05-29 14957, 2017

      • bitmap
        what are you using for fks if you only have one table?
      • 2017-05-29 14949, 2017

      • psolanki
        entity_id in review table
      • 2017-05-29 14908, 2017

      • Freso
        ListMyCDs: What's your username on MB/the forums again? >_>
      • 2017-05-29 14921, 2017

      • CatQuest
        same?
      • 2017-05-29 14925, 2017

      • Freso
        Notes from tonight's meeting are up: https://community.metabrainz.org/t/notes-from-met…
      • 2017-05-29 14930, 2017

      • CatQuest
        ListMyCDs.com or .net..
      • 2017-05-29 14936, 2017

      • Freso
        CatQuest: Discourse doesn't register it.
      • 2017-05-29 14941, 2017

      • Freso
        Right. The dot something.
      • 2017-05-29 14949, 2017

      • ListMyCDs
        Freso: It's ListMyCDs.com
      • 2017-05-29 14930, 2017

      • Freso
        ListMyCDs: Yep. Fixed. :)
      • 2017-05-29 14934, 2017

      • bitmap
        psolanki: I see, so you store the mbid and its entity type in the review table. well, you'll have to write triggers if you want data integrity then
      • 2017-05-29 14907, 2017

      • Freso
        CatQuest: Can you close the post for a second so I can add a category without unintentionally overwriting any edits you've made?
      • 2017-05-29 14950, 2017

      • Freso
        I guess he went to do food stuff.
      • 2017-05-29 14906, 2017

      • Freso
        CatQuest: Feel free to publish, I'll add category later. :)
      • 2017-05-29 14912, 2017

      • Freso packs up and heads home...
      • 2017-05-29 14939, 2017

      • CatQuest
        oh ok
      • 2017-05-29 14949, 2017

      • CatQuest
        I'll piblish nad yo ucna add the category! :D
      • 2017-05-29 14953, 2017

      • CatQuest
        erh publish
      • 2017-05-29 14957, 2017

      • psolanki
        bitmap: during the time of migration?
      • 2017-05-29 14922, 2017

      • bitmap
        maybe I misunderstood the schema
      • 2017-05-29 14936, 2017

      • bitmap
        but I'm not sure what an average rating has to do with a review
      • 2017-05-29 14909, 2017

      • bitmap
        so why would there be an FK between those two tables?
      • 2017-05-29 14953, 2017

      • bitmap
        though you should ignore me since I haven't actually looked at the schema
      • 2017-05-29 14910, 2017

      • bitmap
        but the entity_id can't possibly be a primary key in review
      • 2017-05-29 14924, 2017

      • psolanki
        rating (score) column would be added to the revision table, so rating alone is also a review
      • 2017-05-29 14917, 2017

      • psolanki sent a long message: psolanki_2017-05-29_18:33:17.txt <https://matrix.org/_matrix/media/v1/download/matrix.org/olcdMLhtbJnsicjtgOdfJWZL>
      • 2017-05-29 14902, 2017

      • bitmap
        I'd have to see the actual SQL to say anything else
      • 2017-05-29 14937, 2017

      • bitmap
        sorry for interrupting your work with my uninformed questions
      • 2017-05-29 14939, 2017

      • CatQuest
        right I've done stuff today so now I'm going to relax with some game. \o see you tomorrow folks <3
      • 2017-05-29 14939, 2017

      • CatQuest
        freso: if you retrun, don't alter my term "initiate" as it's a (probably bad) pun on "initial"
      • 2017-05-29 14947, 2017

      • CatQuest
        like, since I was "initiated"
      • 2017-05-29 14919, 2017

      • CatQuest
        (I saw that reo chnaged it in one of his revisions :P)
      • 2017-05-29 14920, 2017

      • reosarevok
        ruaok: I assume we won't have finished the paperwork before invoice time and I should invoice in USD still this time?
      • 2017-05-29 14945, 2017

      • github joined the channel
      • 2017-05-29 14945, 2017

      • github
        [musicbrainz-server] mwiencek closed pull request #501: MBS-8640: Make adding work attribute auto-edit (master...mbs-8640-auto-attribute) https://git.io/vS8J1
      • 2017-05-29 14945, 2017

      • github has left the channel
      • 2017-05-29 14905, 2017

      • chirlu has quit
      • 2017-05-29 14953, 2017

      • chirlu joined the channel
      • 2017-05-29 14943, 2017

      • hibiscuskazeneko has quit
      • 2017-05-29 14921, 2017

      • alastairp
        Say it isn't so
      • 2017-05-29 14940, 2017

      • alastairp
        Freso posting meeting notes the *same day* as the meeting?
      • 2017-05-29 14951, 2017

      • CatQuest
        ikr?
      • 2017-05-29 14951, 2017

      • CatQuest
        early meeting time ftw
      • 2017-05-29 14905, 2017

      • arbenina_ has quit
      • 2017-05-29 14955, 2017

      • lazka joined the channel
      • 2017-05-29 14916, 2017

      • gcilou joined the channel
      • 2017-05-29 14939, 2017

      • yvanzo
        !m Freso
      • 2017-05-29 14939, 2017

      • BrainzBot
        You're doing good work, Freso!
      • 2017-05-29 14933, 2017

      • github joined the channel
      • 2017-05-29 14933, 2017

      • github
        [musicbrainz-server] mwiencek closed pull request #510: Node renderer fixes (master...node-fixes) https://git.io/v9Nya
      • 2017-05-29 14933, 2017

      • github has left the channel
      • 2017-05-29 14903, 2017

      • github joined the channel
      • 2017-05-29 14903, 2017

      • github
        [musicbrainz-server] mwiencek closed pull request #511: MBS-9347: Fix regression regarding page titles (master...mbs-9347) https://git.io/v9pVd
      • 2017-05-29 14903, 2017

      • github has left the channel
      • 2017-05-29 14908, 2017

      • github joined the channel
      • 2017-05-29 14908, 2017

      • github
        [musicbrainz-server] mwiencek closed pull request #512: MBS-8757: Fix WS browse-event-by-area requests. (master...mbs-8757) https://git.io/vHJhd
      • 2017-05-29 14908, 2017

      • github has left the channel
      • 2017-05-29 14939, 2017

      • yvanzo
        bitmap: there are 125 work language edits being stalled from before the schema change. Best way to solve this: redo it by hand?
      • 2017-05-29 14903, 2017

      • bitmap
        do you have a list of them?
      • 2017-05-29 14952, 2017

      • bitmap
        well, let me see what the error is
      • 2017-05-29 14905, 2017

      • bitmap
        it might be easy to fix
      • 2017-05-29 14909, 2017

      • yvanzo
      • 2017-05-29 14932, 2017

      • bitmap
        thanks
      • 2017-05-29 14951, 2017

      • yvanzo
      • 2017-05-29 14918, 2017

      • bitmap
        I think we just need to add a 'restore' method that converts the language_id property
      • 2017-05-29 14925, 2017

      • bitmap
        we'll skip today's release since I didn't really have time to merge anything over the past two weeks
      • 2017-05-29 14947, 2017

      • bitmap
        but we can get a fix for that on beta and I can run ModBot manually on beta
      • 2017-05-29 14924, 2017

      • yvanzo
        Ok, I can make that fix.
      • 2017-05-29 14930, 2017

      • bitmap
        awesome, thank you
      • 2017-05-29 14937, 2017

      • github joined the channel
      • 2017-05-29 14937, 2017

      • github
        [musicbrainz-server] mwiencek closed pull request #513: Update and improve support for CD Baby URLs (MBS-9311, MBS-9348) (master...mbs-9311-mbs-9348-cdbaby) https://git.io/vHIJ7
      • 2017-05-29 14937, 2017

      • github has left the channel
      • 2017-05-29 14957, 2017

      • alastairp
        bitmap:, there have been lots of people with the problem with the postgres driver
      • 2017-05-29 14902, 2017

      • github joined the channel
      • 2017-05-29 14902, 2017

      • github
        [musicbrainz-server] mwiencek closed pull request #514: Add support for Big Cartel URLs in mail order relationships (MBS-9350) (master...mbs-9350-bigcartel) https://git.io/vHIJF
      • 2017-05-29 14902, 2017

      • github has left the channel
      • 2017-05-29 14913, 2017

      • bitmap
        I know :(
      • 2017-05-29 14920, 2017

      • alastairp
        Anything we can do about a new schema release tag with the old version?
      • 2017-05-29 14958, 2017

      • alastairp
        I didn't have the same Raven problem that others had too, but perhaps that too?
      • 2017-05-29 14952, 2017

      • bitmap
        I was thinking of merging those those fixes to the schema-change-2017-q2 branch and creating a new tag based off of that
      • 2017-05-29 14922, 2017

      • bitmap
        we did something similar last year, had to make a -v2 tag
      • 2017-05-29 14949, 2017

      • alastairp
        Sounds like a good idea to me
      • 2017-05-29 14938, 2017

      • bitmap
        yvanzo: do you still get the renderer errors running tests? not sure why, but I don't anymore
      • 2017-05-29 14913, 2017

      • yvanzo
        I got it even while testing your PRs.
      • 2017-05-29 14940, 2017

      • bitmap
        "Couldn't connect to the renderer." right?
      • 2017-05-29 14951, 2017

      • yvanzo
        Well... seems to have vanished here as well...
      • 2017-05-29 14914, 2017

      • bitmap
        crazy
      • 2017-05-29 14953, 2017

      • yvanzo
        I just checked latest master, recompiled scripts, recreated test db and relaunched probe.
      • 2017-05-29 14910, 2017

      • bitmap
        I see an issue where it doesn't kill the renderer after the tests die
      • 2017-05-29 14929, 2017

      • bitmap
        but it still works even if I run them again with that renderer still running; it just connects to that one
      • 2017-05-29 14953, 2017

      • yvanzo
        By the way, did you remove 'tests' target for resources compiling script for good?
      • 2017-05-29 14942, 2017

      • bitmap
        hmm, that wasn't intentional, sorry
      • 2017-05-29 14958, 2017

      • bitmap
        it still works via ./node_modules/.bin/gulp tests
      • 2017-05-29 14926, 2017

      • SothoTalKer
        ohhh, look. there are just the 2 people i was looking for :D
      • 2017-05-29 14938, 2017

      • bitmap
        I don't really use the browser tests ever
      • 2017-05-29 14938, 2017

      • samj1912 has quit
      • 2017-05-29 14906, 2017

      • SothoTalKer
        bitmap, yvanzo: what do you make of this? https://pastebin.com/RLKg1LuA
      • 2017-05-29 14926, 2017

      • bitmap
        I mean I run them with t/js.t
      • 2017-05-29 14911, 2017

      • bitmap
        SothoTalKer: is that for a specific edit?
      • 2017-05-29 14917, 2017

      • SothoTalKer
        yes
      • 2017-05-29 14917, 2017

      • yvanzo
        SothoTalKer: how did you get this? did you report it?
      • 2017-05-29 14931, 2017

      • bitmap
        which edit?
      • 2017-05-29 14910, 2017

      • bitmap
        or is this trying to submit it
      • 2017-05-29 14934, 2017

      • SothoTalKer
      • 2017-05-29 14958, 2017

      • bitmap
        yeah, never mind
      • 2017-05-29 14914, 2017

      • SothoTalKer
        While submitting this edit there where 2 other edits by me, moving the medium numbers from 2 to 1, and from 3 to 2.
      • 2017-05-29 14916, 2017

      • bitmap
        well, the exception you linked was for a ReorderMediums edit
      • 2017-05-29 14919, 2017

      • SothoTalKer
        Those failed
      • 2017-05-29 14946, 2017

      • bitmap
        I've seen this error before, I don't know what causes it yet
      • 2017-05-29 14930, 2017

      • SothoTalKer
        i guess that happens when you try to remove the first medium of a release, and the positions of the others try to automatically change. but since the removal of the first medium is a destructive edit, it needs to get voted on, so the other edits (changing medium number) fail.
      • 2017-05-29 14902, 2017

      • SothoTalKer
        because medium 1 still does exist.
      • 2017-05-29 14916, 2017

      • bitmap
        it tries to be smart and change the position of the removed medium before it removes it, but perhaps that fails
      • 2017-05-29 14935, 2017

      • SothoTalKer
        I think it is easy to recreate. just set up a release with 2 or more mediums and try to remove the first one. That should trigger it.
      • 2017-05-29 14938, 2017

      • yvanzo
        bitmap: No worry about browser tests. I just noticed the change because I was used to compile it (acc. to HACKING doc), back then my server machine was much slower than my client. I don't use them anymore either.
      • 2017-05-29 14916, 2017

      • bitmap
        okay 👍
      • 2017-05-29 14933, 2017

      • bitmap
        I only use them when I'm debugging a browser-specific issue
      • 2017-05-29 14912, 2017

      • Sophist_UK
        bitmap: Change to gh:MBS#505 made.
      • 2017-05-29 14912, 2017

      • BrainzBot
        MBS-9323: Fix long URLs distorting page layout.: https://github.com/metabrainz/musicbrainz-server/…
      • 2017-05-29 14920, 2017

      • github joined the channel
      • 2017-05-29 14920, 2017

      • github
        [musicbrainz-server] yvanzo opened pull request #517: MBS-9362: Restore work language edit data (beta...mbs-9362-beta-fix) https://git.io/vHWGb
      • 2017-05-29 14920, 2017

      • github has left the channel
      • 2017-05-29 14944, 2017

      • ruaok
        reosarevok: lets see about getting the contract done tomorrow. ping me during the day
      • 2017-05-29 14950, 2017

      • Mineo has quit
      • 2017-05-29 14911, 2017

      • m0n0g0n has quit
      • 2017-05-29 14941, 2017

      • lazka has quit
      • 2017-05-29 14927, 2017

      • Bellami joined the channel
      • 2017-05-29 14929, 2017

      • suhas2go has quit
      • 2017-05-29 14933, 2017

      • D4RK-PH0ENiX has quit
      • 2017-05-29 14901, 2017

      • D4RK-PH0ENiX joined the channel
      • 2017-05-29 14947, 2017

      • D4RK-PH0ENiX has quit
      • 2017-05-29 14936, 2017

      • D4RK-PH0ENiX joined the channel
      • 2017-05-29 14957, 2017

      • SkullTech joined the channel
      • 2017-05-29 14911, 2017

      • SkullTech
        In the issue PW-41, the tables need to be sorted. Can someone tell me what would be the preferred way of doing this, server-side or client-side (using Javascript)?
      • 2017-05-29 14911, 2017

      • BrainzBot
        PW-41: Plugins table sortable https://tickets.metabrainz.org/browse/PW-41