#metabrainz

/

      • shivam-kapila
        Thanks ishaanshah[m] :D. Surely will share it asap.
      • 2020-03-12 07235, 2020

      • _lucifer
        ruaok: yvanzo: what's a good way to store keys to sign the app ?
      • 2020-03-12 07232, 2020

      • _lucifer
        i mean i have a local copy but it should be stored somewhere with the org as well
      • 2020-03-12 07247, 2020

      • Endurance21 has quit
      • 2020-03-12 07229, 2020

      • D4RK-PH0ENiX has quit
      • 2020-03-12 07203, 2020

      • Endurance21 joined the channel
      • 2020-03-12 07222, 2020

      • MusicbrainzB0T joined the channel
      • 2020-03-12 07254, 2020

      • MusicbrainzB0T1 has quit
      • 2020-03-12 07234, 2020

      • gr0uch0mars
        yes, we should store copies of the keys…
      • 2020-03-12 07231, 2020

      • D4RK-PH0ENiX joined the channel
      • 2020-03-12 07234, 2020

      • yvanzo
        _lucifer, gr0uch0mars: I can copy it to our private store.
      • 2020-03-12 07243, 2020

      • ruaok
        _lucifer: yes, we have a repo for such things. mail them to me and I'll add them to the repo.
      • 2020-03-12 07200, 2020

      • yvanzo
        reosarevok: it seems all set for beta now, unless you are about to update 1408?
      • 2020-03-12 07253, 2020

      • _lucifer
        thanks yvanzo, ruaok I'll do that
      • 2020-03-12 07212, 2020

      • P23
        ruaok updated the .proto file and added a key val based struct method for unknown/optional things by the user aswell
      • 2020-03-12 07232, 2020

      • P23
      • 2020-03-12 07239, 2020

      • P23
        would love more feedback
      • 2020-03-12 07250, 2020

      • yvanzo
        _lucifer: how is this related to Picard? https://github.com/metabrainz/musicbrainz-android…
      • 2020-03-12 07255, 2020

      • _lucifer
        yvanzo: that must be a typo, it should tagger instead since we do not have a name yet
      • 2020-03-12 07258, 2020

      • _lucifer
        i'll fix that
      • 2020-03-12 07210, 2020

      • reosarevok
        I'm not home yet, so put beta out if you want
      • 2020-03-12 07217, 2020

      • reosarevok
        yvanzo ^
      • 2020-03-12 07240, 2020

      • _lucifer
        ruaok: since we transferred the app to the new account, we need to fill out a target audience and policy compliance questionnaire
      • 2020-03-12 07247, 2020

      • _lucifer
        do you want to take a look at it?
      • 2020-03-12 07246, 2020

      • ruaok
        Sure. Got link?
      • 2020-03-12 07252, 2020

      • yvanzo
        _lucifer: thanks, fwiw, I came across this while updating translation.
      • 2020-03-12 07207, 2020

      • yvanzo
        reosarevok: ok, will do
      • 2020-03-12 07208, 2020

      • prabal has quit
      • 2020-03-12 07214, 2020

      • _lucifer
      • 2020-03-12 07255, 2020

      • _lucifer
        if it does not work, i can tell you the steps to open it
      • 2020-03-12 07211, 2020

      • prabal joined the channel
      • 2020-03-12 07207, 2020

      • supersandro2000
      • 2020-03-12 07236, 2020

      • BrainzGit
        [bookbrainz-site] MonkeyDo merged pull request #381 (master…editorpageredesign): removed disambiguation button and added disambiguation input as a def… https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2020-03-12 07225, 2020

      • travis-ci joined the channel
      • 2020-03-12 07225, 2020

      • travis-ci
        Project bookbrainz-site build #2715: passed in 2 min 11 sec: https://travis-ci.org/bookbrainz/bookbrainz-site/…
      • 2020-03-12 07225, 2020

      • travis-ci has left the channel
      • 2020-03-12 07231, 2020

      • Pac23m joined the channel
      • 2020-03-12 07229, 2020

      • yvanzo
        supersandro2000: thanks, I will rebase my mbs-dev stuff on that
      • 2020-03-12 07253, 2020

      • dpmittal_ joined the channel
      • 2020-03-12 07202, 2020

      • supersandro2000
        if you find another problem let me know
      • 2020-03-12 07200, 2020

      • BrainzGit
        [musicbrainz-docker] yvanzo merged pull request #123 (mbvm-38-dev…fix-build-and-update): Fix build, update base images and software, update readme https://github.com/metabrainz/musicbrainz-docker/…
      • 2020-03-12 07249, 2020

      • yvanzo
        I did already test with that fix, so hopefully that was the last bug.
      • 2020-03-12 07243, 2020

      • dpmittal_ has quit
      • 2020-03-12 07251, 2020

      • yvanzo
        updating beta.mb.o
      • 2020-03-12 07212, 2020

      • yvanzo
        done
      • 2020-03-12 07224, 2020

      • yvanzo
        bitmap, reosarevok: there are a lot of unassigned tickets in beta, likely yours.
      • 2020-03-12 07243, 2020

      • Endurance21
        Mr_Monkey: i am having hard time understanding how props are passed down to entity-editor. Can you please give a rough sketch of that?. I am currently working on https://tickets.metabrainz.org/browse/BB-296
      • 2020-03-12 07244, 2020

      • BrainzBot
        BB-296: Save creation form state on (or before) page navigation
      • 2020-03-12 07209, 2020

      • Endurance21
        Mr_Monkey: if I get the way how data of forms are passed then it will be easy for me to save them in localstorage as discussed.
      • 2020-03-12 07213, 2020

      • Mr_Monkey
        Endurance21: We use Redux as the state for the entity editor. Are you familiar with Redux?
      • 2020-03-12 07214, 2020

      • Endurance21
        Mr_Monkey: yes I am familiar with that.
      • 2020-03-12 07212, 2020

      • Mr_Monkey
        Each section of the form has actions that can be dispatched, which trigger something in the accompanying reducer that modifies the state
      • 2020-03-12 07247, 2020

      • Mr_Monkey
        For example the submit section light be the right place if you're thinking of having save/clear buttons: https://github.com/bookbrainz/bookbrainz-site/tre…
      • 2020-03-12 07252, 2020

      • Mr_Monkey
        might*
      • 2020-03-12 07231, 2020

      • Endurance21
        Mr_Monkey:i am currently on that section,
      • 2020-03-12 07236, 2020

      • Endurance21
        I need to ask, saving form data in localstorage should be a part of reducer function?. Or I have to do that explicitly.
      • 2020-03-12 07229, 2020

      • Mr_Monkey
        Some other option that would automatically save every time the state changes (it can be throttled too: https://medium.com/@jrcreencia/persisting-redux-s…
      • 2020-03-12 07246, 2020

      • Endurance21
        Mr_Monkey: bcz reducer functions are meant to change the state only, and I am using localstorage thing. So I think I should not include that in reducers.
      • 2020-03-12 07214, 2020

      • Mr_Monkey
        I haven't thought of where the best place would be to put this. What's your suggestion?
      • 2020-03-12 07200, 2020

      • Mr_Monkey
      • 2020-03-12 07244, 2020

      • Pac23m has quit
      • 2020-03-12 07247, 2020

      • Endurance21
        Mr_Monkey: i think this must be placed inside submission-section.js itself
      • 2020-03-12 07226, 2020

      • Endurance21
        Is it possible to get app store in submission-section.js
      • 2020-03-12 07250, 2020

      • _lucifer
        ruaok: let me know when you are done with that form, I'll then push an update to verify releases are working are fine.
      • 2020-03-12 07201, 2020

      • ruaok
        sorry, just got back from lunch.
      • 2020-03-12 07215, 2020

      • _lucifer
        no hurry
      • 2020-03-12 07220, 2020

      • _lucifer
        gr0uch0mars: how do you suggest I do the further development on the app, do I work on my fork of the official repo and submit a PR or work on a new branch in the official repo ?
      • 2020-03-12 07203, 2020

      • Endurance21
        Mr_Monkey: i have a another idea! Why to use save button? Whenever there is state change our form data automatically should be saved In local storage. Only we need. A clear button. To clear the saved data in forms.
      • 2020-03-12 07216, 2020

      • Mr_Monkey
        Endurance21: I think the best solution would be to subscribe to store changes: https://egghead.io/lessons/javascript-redux-persi…
      • 2020-03-12 07219, 2020

      • Mr_Monkey
        Yes, that.
      • 2020-03-12 07226, 2020

      • Endurance21
        As it's simple to make it.
      • 2020-03-12 07237, 2020

      • Endurance21
        Yes i was going through the same blog.
      • 2020-03-12 07253, 2020

      • Endurance21
        I was mesmerized by that..
      • 2020-03-12 07239, 2020

      • Endurance21
        Mr_Monkey: do you agree with not having a save button?.
      • 2020-03-12 07245, 2020

      • Mr_Monkey
        yes
      • 2020-03-12 07200, 2020

      • Endurance21
        Okay! Thanks.
      • 2020-03-12 07207, 2020

      • Mr_Monkey
        But there need to be clear rules about when we clear the state
      • 2020-03-12 07233, 2020

      • Endurance21
        We will be clearing the form data of state. Not the whole state..
      • 2020-03-12 07258, 2020

      • Endurance21
        And on button click
      • 2020-03-12 07258, 2020

      • Mr_Monkey
        For example if I submit an edit and it goes through, you would need to clear the saved state. However if there's an error when submitting, don't clear the state.
      • 2020-03-12 07213, 2020

      • Endurance21
        Ya that's error handling. I will take care of that..
      • 2020-03-12 07245, 2020

      • Endurance21
        There will be a active or inactive state of clear button, it will be active only when if either data is submitted successfully or the submit button is not yet hit.
      • 2020-03-12 07211, 2020

      • Endurance21
        And it will be inactive for all other cases
      • 2020-03-12 07216, 2020

      • Mr_Monkey
        There should also be a mechanism to automatically clear the saved form state if succesfully submitted.
      • 2020-03-12 07207, 2020

      • leonardo
        looks like i cannot import anymore releases from bandcamp using the greasemonkey script on firefox, is just me or something is changed in the musicbrainz site and the script is now broken?
      • 2020-03-12 07222, 2020

      • Mr_Monkey
        Endurance21: Here's how to react to specific redux actions using middleware, could be a good option: https://medium.com/@Dane_s/persisting-data-with-l…
      • 2020-03-12 07201, 2020

      • reosarevok
        leonardo: latest version works fine for me in Chrome at least
      • 2020-03-12 07223, 2020

      • leonardo
        i see
      • 2020-03-12 07228, 2020

      • leonardo
        so is definitely me
      • 2020-03-12 07203, 2020

      • leonardo
        problem is, even when i'm logged in, the system redirect me on the login page
      • 2020-03-12 07224, 2020

      • leonardo
        i submit once again my credentials, but then i lost every other parameter
      • 2020-03-12 07203, 2020

      • Pac23m joined the channel
      • 2020-03-12 07240, 2020

      • leonardo
        probably some session cookie is not sent correctly
      • 2020-03-12 07259, 2020

      • Endurance21
        Mr_Monkey: ya i agree about automatically clearing the data if it is submitted successfully! So that if in any case user need to update the same he/she will revise it from edit section..
      • 2020-03-12 07258, 2020

      • leonardo
        which is strance, since i whitelisted *.musicbrainz.org on my ublock extension
      • 2020-03-12 07252, 2020

      • reosarevok
        leonardo: huh. Are you maybe logged in in beta and the script send you to the normal server or the other way around?
      • 2020-03-12 07254, 2020

      • leonardo
        mmm nope iwas not even aware of a beta version
      • 2020-03-12 07220, 2020

      • leonardo
        interestingly, the "import" button redirects me to the login page, while the "search" button shows me the search results and shows me that i am logged in
      • 2020-03-12 07205, 2020

      • reosarevok
        Huh
      • 2020-03-12 07209, 2020

      • reosarevok
        What's your version?
      • 2020-03-12 07258, 2020

      • leonardo
        using 2020.2.14.2 on Tampèermonkey on Firefox Nightly
      • 2020-03-12 07249, 2020

      • leonardo
        Tampermonkey*
      • 2020-03-12 07221, 2020

      • BrainzGit
        [bookbrainz-site] MonkeyDo merged pull request #383 (master…SearchResult-newtab-prabal): Don’t open search results in new tab https://github.com/bookbrainz/bookbrainz-site/pul…
      • 2020-03-12 07238, 2020

      • Pac23m has quit
      • 2020-03-12 07248, 2020

      • Pac23m joined the channel
      • 2020-03-12 07213, 2020

      • travis-ci joined the channel
      • 2020-03-12 07213, 2020

      • travis-ci
        Project bookbrainz-site build #2716: passed in 2 min 16 sec: https://travis-ci.org/bookbrainz/bookbrainz-site/…
      • 2020-03-12 07213, 2020

      • travis-ci has left the channel
      • 2020-03-12 07244, 2020

      • reosarevok
        I have 2020.2.14.2 as well. But I seem to remember someone else had an issue with it recently?
      • 2020-03-12 07204, 2020

      • reosarevok
        Yeah, they did, also firefox
      • 2020-03-12 07221, 2020

      • reosarevok
        So something probably broke with recent FF or tampermonkey updates
      • 2020-03-12 07226, 2020

      • leonardo
        i haven't tried with chromium, maybe is just a firefox issue
      • 2020-03-12 07252, 2020

      • leonardo
        lol we reached the same conclusions
      • 2020-03-12 07225, 2020

      • leonardo
        i'll try chromium and maybe some alternative to tampermonkey and see what happen
      • 2020-03-12 07202, 2020

      • leonardo
        speaking of which... is there any alternatives to tampermonkey?
      • 2020-03-12 07239, 2020

      • gr0uch0mars
        _lucifer: I think creating branches and PRs on the official repo would be enough. I will add templates for PRs so it's easier to review
      • 2020-03-12 07250, 2020

      • gr0uch0mars
        _lucifer: you can start by creating an issue describing what you're doing, and create a branch like: “feature/#4-my-awesome-thing”, that way we know it's related to issue #4
      • 2020-03-12 07216, 2020

      • gr0uch0mars
        later we'll try to migrate schedule to tasks in MB's JIRA, but let's start easy
      • 2020-03-12 07246, 2020

      • _lucifer
        👍 gr0uch0mars
      • 2020-03-12 07227, 2020

      • SothoTalKer_ joined the channel
      • 2020-03-12 07251, 2020

      • SothoTalKer has quit
      • 2020-03-12 07226, 2020

      • leonardo
        reosarevok: installed greasemonkey, tried again, still the same result
      • 2020-03-12 07236, 2020

      • leonardo
        so, firefox is the problem
      • 2020-03-12 07211, 2020

      • Lotheric
        leonardo: there's violentmonkey on chrome
      • 2020-03-12 07250, 2020

      • Lotheric
        seems to have a version for firefox as well
      • 2020-03-12 07256, 2020

      • Lotheric
        never used it though
      • 2020-03-12 07210, 2020

      • leonardo
        i'll try that too
      • 2020-03-12 07215, 2020

      • leonardo
        but i'm pretty sure it won't change anything
      • 2020-03-12 07226, 2020

      • Pac23m has quit
      • 2020-03-12 07227, 2020

      • MusicbrainzB0T1 joined the channel
      • 2020-03-12 07256, 2020

      • MusicbrainzB0T has quit
      • 2020-03-12 07212, 2020

      • BrainzGit
        [musicbrainz-server] reosarevok merged pull request #1156 (master…MBS-4090): MBS-4090: Report: non-matching recording/track titles https://github.com/metabrainz/musicbrainz-server/…
      • 2020-03-12 07213, 2020

      • BrainzBot
        MBS-4090: Report for recordings with a single track but with different spellings https://tickets.metabrainz.org/browse/MBS-4090
      • 2020-03-12 07222, 2020

      • dseomn1 joined the channel
      • 2020-03-12 07222, 2020

      • dseomn has quit
      • 2020-03-12 07227, 2020

      • dseomn1 has quit
      • 2020-03-12 07201, 2020

      • Lotheric has quit
      • 2020-03-12 07218, 2020

      • sumedh joined the channel
      • 2020-03-12 07213, 2020

      • Lotheric joined the channel
      • 2020-03-12 07259, 2020

      • dseomn joined the channel
      • 2020-03-12 07241, 2020

      • Cyna
      • 2020-03-12 07246, 2020

      • Cyna
        bitmap: yvanzo : Any idea why I face this only on a docker container of ubuntu when making it run ./scripts/compile_resources.sh ?
      • 2020-03-12 07257, 2020

      • yvanzo
        Cyna: isn't it due to the way you mount submodule as a volume in docker?
      • 2020-03-12 07229, 2020

      • Cyna
        I dont think that should be the reason should it? submoduling in based on my local machine... for the container the it should simply be a folder bound as a volume.
      • 2020-03-12 07200, 2020

      • Cyna
        When running from docker-compose from my mac everything runs fine.. It submoduling was the case, it should also have occurred on mac, right ?
      • 2020-03-12 07207, 2020

      • yvanzo
        Cyna: because of “fatal: Not a git repository: ../.git/modules/musicbrainz-server”
      • 2020-03-12 07234, 2020

      • yvanzo
        ỳour 'mb-docker' repo has a submodule named 'musicbrainz-server'
      • 2020-03-12 07256, 2020

      • Cyna
        Yea which is also a git repository right
      • 2020-03-12 07247, 2020

      • yvanzo
        Yes which probably references 'mb-docker' as parent repo
      • 2020-03-12 07228, 2020

      • yvanzo
        So when trying to get git info from 'musicbrainz-server', it also tries to get git info from 'mb-docker' which is not mounted into docker.