We will need config keys (+fork keys when adding GitLab) in a few different locations (see #96, final comment). It didn't seem to necessarily fit in src/utils/defineApiUrls.js or src/utils/defineRibbonVals.js, though both of these are just getting various platform-specific values (each has only one function).
I'm thinking it would be better to consolidate to a single definePlatformValues.js utility with those two and a getPlatformKeys function to define platformKeys (instead of configs as used here). Would you agree with that move?
Originally posted by @egrace479 in #121 (comment)
We will need config keys (+fork keys when adding GitLab) in a few different locations (see #96, final comment). It didn't seem to necessarily fit in
src/utils/defineApiUrls.jsorsrc/utils/defineRibbonVals.js, though both of these are just getting various platform-specific values (each has only one function).I'm thinking it would be better to consolidate to a single
definePlatformValues.jsutility with those two and agetPlatformKeysfunction to defineplatformKeys(instead of configs as used here). Would you agree with that move?Originally posted by @egrace479 in #121 (comment)