-
Notifications
You must be signed in to change notification settings - Fork 20
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
Feedback implementation for the Workshop #318
Conversation
Signed-off-by: Chris Chinchilla <[email protected]> # Conflicts: # docs/develop/08_opendid/01_overview.md # docs/develop/08_opendid/02_opendid_flow.md # docs/develop/08_opendid/03_opendid_service.md # docs/develop/08_opendid/04_integrate_opendid.md # docs/develop/08_opendid/05_demo_project.md # docs/develop/08_opendid/06_advanced.md
@MarkusGarmeister Fixed merge conflicts, I think you started work before I made changes. Will now look at the code side. |
Signed-off-by: Chris Chinchilla <[email protected]>
code_examples/sdk_examples/src/workshop/attester/generateKeypairs.ts
Outdated
Show resolved
Hide resolved
Also seems to be a lot of text editor / IDE formatting differences in the PR, markdown doesn't really have standards, so I don't mind allowing these in, but they're not really necessary and someone else's changes will probably change them again. I use |
Signed-off-by: Chris Chinchilla <[email protected]>
@MarkusGarmeister Something is wrong with our CI tests, I ran them locally and everything worked OK, so will approve this now |
Signed-off-by: Chris Chinchilla <[email protected]>
This pull request is automatically being deployed by Amplify Hosting (learn more). |
fixes KILTProtocol/ticket#3455
I made several changes to improve the project:
Updated the JS code for ES6 compatibility:
-Added file extensions to the imported files.
-Changed the require statements to ES6 import syntax.
Added a generateKeypairs file to the attester module.
Included terminal commands for quick creation of folder structures and files.
Modified the output of the DID to make it less confusing.
Checklist: