chore: merged repo and org config

[devpanther]

Resolves #648

QA: one-two-test/one#3

[netlify]

✅ Deploy Preview for ubiquibot-staging ready!

Name	Link
🔨 Latest commit	2f0631c
🔍 Latest deploy log	https://app.netlify.com/sites/ubiquibot-staging/deploys/64f9809e12cce50008bcd308
😎 Deploy Preview	https://deploy-preview-691--ubiquibot-staging.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[whilefoo]

If it wasn't clear, the spec is to have nested inheritance for the entire config, not just for incentives. Ideally I'd like to get rid of every function in helpers.ts and just merge repo, org and default config using lodash - that way we don't need to keep adding new functions for every property.

[0xcodercrane]

we can get the nested inheritance for the entire config without using the lodash simply like this.

const config = {...parsedOrgConfig, ...parsedRepoConfig}

What's the benefit to use lodash here? @whilefoo

[whilefoo]

we can get the nested inheritance for the entire config without using the lodash simply like this.

const config = {...parsedOrgConfig, ...parsedRepoConfig}

What's the benefit to use lodash here? @whilefoo

that would work only for top-level properties not nested, example: https://jsfiddle.net/8qhbj7r2/

[devpanther]

that way we don't need to keep adding new functions for every property.

Got that, I'm pushing a new commit with helpers function removed and lodash used in merging all objs

[devpanther]

All configs merged: one-two-test/one#3 (comment)

[whilefoo]

@devpanther please resolve conflicts

[devpanther]

@devpanther please resolve conflicts

resolved

[devpanther]

@whilefoo @Draeieg This should be QA'd and merged asap.. there are new configs been added everyday and that's merge conflicts for me

[devpanther]

@whilefoo @Draeieg ping

[whilefoo]

QA: ubiquity-whilefoo#84