-
Notifications
You must be signed in to change notification settings - Fork 24
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
fix: Update create-claim script to use createACDClaim #624
Conversation
What are the package.json and package-lock.json files in here for? |
@warrenv I had to add |
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.
- Tested changes manually
- Checked accidental architectural/style changes
- Reviewed entire diff
- [NA] Unit tests
- [NA] Documentation
- [NA] Filenames and locations
Nice cleanup. Remove the ;
and it looks good to me.
scripts/CreateClaim.ts
Outdated
|
||
const main = async () => { | ||
const attributes = { | ||
name: process.argv[2] || 'testing claim', | ||
author: process.argv[3] || 'the tester', | ||
} | ||
const client = new Client() | ||
const claim = await createClaim(PrivateKeyACD, ClaimType.Work, attributes) | ||
const claim = await createACDClaim(attributes); |
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.
remove ;
🎉 This PR is included in version 2.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
PR Process - PR Review Checklist
Release
Semantic release is enabled for this repository. Make sure you follow the right commit message convention.
We're using semantic-release's default — Angular Commit Message Conventions.
Description of Changes
Fixes #625
The create-claim script broke when updating to poet-js v5.0.5.
Moved create[Type of Claim]Claim functions to tests/helpers/utils so that create-claim script can use them without triggering the setUpClaims script.