ubiquity / ubiquibot

Putting the 'A' in 'DAO'
https://github.com/marketplace/ubiquibot
MIT License
17 stars 61 forks source link

Fix/database functions #849

Closed 0x4007 closed 12 months ago

0x4007 commented 1 year ago

Still hacking away at this. Overtaking https://github.com/ubiquity/ubiquibot/pull/644

netlify[bot] commented 1 year ago

Deploy Preview for ubiquibot-staging ready!

Name Link
Latest commit 503c337c3ffaa6669ea8569f0773cb1169a49cea
Latest deploy log https://app.netlify.com/sites/ubiquibot-staging/deploys/6522aaa28fae0a00081940fd
Deploy Preview https://deploy-preview-849--ubiquibot-staging.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

0x4007 commented 1 year ago

Still need to convert the last of the database client functions:

src/handlers/comment/handlers/authorize.ts(40,30): error TS2552: Cannot find name 'getLabelChanges'. Did you mean 'labelChanges'?
src/handlers/comment/handlers/authorize.ts(42,9): error TS2552: Cannot find name '_approveLabelChange'. Did you mean 'approveLabelChange'?
src/handlers/comment/handlers/index.ts(224,15): error TS2304: Cannot find name 'addPenalty'.
src/handlers/comment/handlers/multiplier.ts(79,11): error TS2304: Cannot find name 'upsertWalletMultiplier'.
src/handlers/comment/handlers/query.ts(29,22): error TS2304: Cannot find name 'getAllAccessLevels'.
src/handlers/comment/handlers/query.ts(33,15): error TS2304: Cannot find name 'upsertAccessControl'.
src/handlers/comment/handlers/query.ts(45,30): error TS2304: Cannot find name 'getWalletInfo'.
src/handlers/comment/handlers/wallet.ts(85,11): error TS2304: Cannot find name 'upsertWalletAddress'.
src/handlers/label/index.ts(28,9): error TS2304: Cannot find name 'saveLabelChange'.
src/handlers/payout/action.ts(19,1): error TS6133: 'shortenEthAddress' is declared but its value is never read.
src/handlers/payout/action.ts(25,1): error TS6133: 'isEmpty' is declared but its value is never read.
src/handlers/payout/action.ts(137,13): error TS2304: Cannot find name 'removePenalty'.
src/handlers/payout/action.ts(202,30): error TS2552: Cannot find name 'getLabelChanges'. Did you mean 'labelChanges'?
src/handlers/payout/action.ts(230,39): error TS2554: Expected 2 arguments, but got 1.
src/handlers/payout/action.ts(291,31): error TS2304: Cannot find name 'getPenalty'.
src/handlers/payout/action.ts(302,13): error TS2304: Cannot find name 'removePenalty'.
src/handlers/payout/action.ts(443,13): error TS2304: Cannot find name 'removePenalty'.
src/handlers/payout/action.ts(491,37): error TS2304: Cannot find name 'getWalletMultiplier'.
src/handlers/payout/calculate-issue-conversation-reward.ts(87,30): error TS2322: Type '{ account: string; priceInBigNumber: Decimal; userId: string; user: string; penaltyAmount: BigNumber; }[]' is not assignable to type '{ account: string; priceInBigNumber: Decimal; penaltyAmount: BigNumber; user: string; userId: number; debug: Record<string, { count: number; reward: Decimal; }>; }[]'.
  Property 'debug' is missing in type '{ account: string; priceInBigNumber: Decimal; userId: string; user: string; penaltyAmount: BigNumber; }' but required in type '{ account: string; priceInBigNumber: Decimal; penaltyAmount: BigNumber; user: string; userId: number; debug: Record<string, { count: number; reward: Decimal; }>; }'.
src/handlers/payout/calculate-issue-creator-reward.ts(64,5): error TS2322: Type 'string' is not assignable to type 'number'.
src/handlers/payout/calculate-issue-creator-reward.ts(70,9): error TS2322: Type 'string' is not assignable to type 'number'.
src/handlers/payout/calculate-review-contributor-rewards.ts(113,30): error TS2322: Type '{ account: string; priceInBigNumber: Decimal; userId: string; user: string; penaltyAmount: BigNumber; }[]' is not assignable to type '{ account: string; priceInBigNumber: Decimal; penaltyAmount: BigNumber; user: string; userId: number; debug: Record<string, { count: number; reward: Decimal; }>; }[]'.
  Property 'debug' is missing in type '{ account: string; priceInBigNumber: Decimal; userId: string; user: string; penaltyAmount: BigNumber; }' but required in type '{ account: string; priceInBigNumber: Decimal; penaltyAmount: BigNumber; user: string; userId: number; debug: Record<string, { count: number; reward: Decimal; }>; }'.
src/helpers/payout.ts(63,30): error TS2304: Cannot find name 'getAccessLevel'.
src/helpers/permit.ts(140,29): error TS2304: Cannot find name 'savePermit'.
src/tests/after-all-handler.ts(1,27): error TS2307: Cannot find module './commands-test' or its corresponding type declarations.
src/tests/before-all-handler.ts(13,8): error TS2307: Cannot find module './commands-test' or its corresponding type declarations.
src/tests/before-all-handler.ts(16,94): error TS2307: Cannot find module './commands-test' or its corresponding type declarations.
src/tests/test-suite.ts(14,8): error TS2307: Cannot find module './commands-test' or its corresponding type declarations.
src/utils/private.ts(169,32): error TS2339: Property 'botDelay' does not exist on type '{ openAIKey?: string | undefined; openAITokenLimit?: number | undefined; privateKeyEncrypted?: string | undefined; issueCreatorMultiplier: number; timeLabels: { name: string; }[]; priorityLabels: { ...; }[]; ... 20 more ...; staleBountyTime: string; }'.
0x4007 commented 1 year ago

todo:

  1. finish redoing all the database client functions
  2. make sure it builds
  3. figure out how to rearchitect labels to dynamically store location and handle repo/org scoping
  4. set up final migrations for database
  5. update all secrets
  6. migrate database
0x4007 commented 12 months ago

Database backup and schema backup: Archive.zip

Will do some due diligence and verify the code to be mostly functional before pushing this through.