-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Policy upload to Filebase/IPFS #639
base: master
Are you sure you want to change the base?
Conversation
Params10
commented
Mar 18, 2023
- fix for Storage migration from Estuary to Filebase #588 .
- styling and typing issue solved.
- contract calls commented, can be executed when ready.
- all format supported for netlify function including pdfs.
… to the filebase exchange
✅ Deploy Preview for kleros-v2 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
contracts/scripts/policyUpdate.ts
Outdated
} | ||
const json = JSON.parse(jsonString); | ||
courtsV1 = json.map((courtDetails) => ({ | ||
...courtDetails, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the format of courtDetails
? If you are destructuring it there is no need to repeat
name: courtDetails.name
as it will already be assigned.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed extra step of destructuring.
Code Climate has analyzed commit 1bc7176 and detected 13 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
Kudos, SonarCloud Quality Gate passed!
|
8409ab7
to
8ea2709
Compare
✅ Deploy Preview for kleros-v2-contracts ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
8ea2709
to
3f8ad18
Compare
Kudos, SonarCloud Quality Gate passed!
|