engineer-dao / platform

The EngineerDAO dApp for interacting with job contracts
engineer-dao-platform.vercel.app
6 stars 4 forks source link

Smart contract reporting fixes and updates #151

Closed deweller closed 2 years ago

deweller commented 2 years ago

Fixes #144. Closes #77. Also closes #53.

Adds a Reported state and a FinalDelisted state.

Successful job reporters are rewarded with their report deposit plus a percentage of the job value.

Sets the following constraints on configurable values:

MAX_DAO_FEE = 25%
MAX_RESOLUTION_FEE_PERCENTAGE = 25%
MAX_REPORT_DEPOSIT = 200
MAX_REPORT_REWARD_PERCENT = 25%

Removes the metadataCid from the contract. This is emitted as a Log Event.

Removes unnecessary requiresApproval modifier.

Removes unused requiresOneOfJobStates modifier.

Completes 100% smart contract test coverage.

image
vercel[bot] commented 2 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

platform – ./frontend

🔍 Inspect: https://vercel.com/engineerdao/platform/972oj2sS4hrfcj8QNunN7Q8HgG4S
✅ Preview: https://platform-git-issue-144-contract-fixes-engineerdao.vercel.app

backend – ./backend

🔍 Inspect: https://vercel.com/engineerdao/backend/BCvMcvXv8N2x8cBXRxMzEHLDT6oc
✅ Preview: https://backend-git-issue-144-contract-fixes-engineerdao.vercel.app

vercel[bot] commented 2 years ago

Deployment failed with the following error:

The total size of all Environment Variables (4.13KB) exceeds 4KB.

Learn More: https://vercel.link/env-vars-limit

deweller commented 2 years ago

Note that these changes will require a new contract deployment on testnet.