Anthony Lavado@lemmy.caM to Jellyfin: The Free Software Media System@lemmy.mlEnglish · 1 year agoA Call for Developersjellyfin.orgexternal-linkmessage-square13fedilinkarrow-up114arrow-down10
arrow-up114arrow-down1external-linkA Call for Developersjellyfin.orgAnthony Lavado@lemmy.caM to Jellyfin: The Free Software Media System@lemmy.mlEnglish · 1 year agomessage-square13fedilink
minus-squaregardner@lemmy.nzlinkfedilinkEnglisharrow-up1arrow-down1·edit-21 year agoI submitted a pull request that was shut down and the only feedback was “I don’t think this is the way we’d do it.” I know managing community contributions is a big task but if you want people to volunteer, a baseline amount of effort should be made to enable it. It’s also helpful to have automated tests to increase confidence that a change won’t break existing features.
minus-squarelud@lemm.eelinkfedilinkEnglisharrow-up1·1 year agoThey are probably talking about this one: https://github.com/jellyfin/jellyfin-web/pull/3522
minus-squareredfellow@sopuli.xyzlinkfedilinkEnglisharrow-up1·1 year agoThere was a baseline effort there. Your PR wasn’t a proper fix, but more of a hack which also isn’t in line with how a proper UI would display things.
I submitted a pull request that was shut down and the only feedback was “I don’t think this is the way we’d do it.”
I know managing community contributions is a big task but if you want people to volunteer, a baseline amount of effort should be made to enable it.
It’s also helpful to have automated tests to increase confidence that a change won’t break existing features.
Link?
They are probably talking about this one: https://github.com/jellyfin/jellyfin-web/pull/3522
There was a baseline effort there. Your PR wasn’t a proper fix, but more of a hack which also isn’t in line with how a proper UI would display things.