I have traced this back to crate.js#openCrate and lootbox.js#openMultiple, wherein the latter does not take the opportunity to consolidate common logic and/or simply delegate to the existing #openBox method (which is currently used only on execute of no-args!). I haven't done much js but I'd like to try either 1) writing conditional message code or 2) refactoring lootbox.js execute to more closely resemble crate.js execute and deduplicate common code in the process 🙂
#openCrate
and lootbox.js#openMultiple
, wherein the latter does not take the opportunity to consolidate common logic and/or simply delegate to the existing#openBox
method (which is currently used only on execute of no-args!). I haven't done much js but I'd like to try either 1) writing conditional message code or 2) refactoringlootbox.js
execute to more closely resemblecrate.js
execute and deduplicate common code in the process 🙂