-
Notifications
You must be signed in to change notification settings - Fork 32
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
Capability to Unbake Open Badge 3.0 #649
Comments
The embedded VC from above:
|
Review error messaging w/ @alexfigtree for:
|
Current error message for uploading a file is: Unable to Add Credentials Ensure the file contains one or more credentials, and is a supported file type. @alexfigtree - could you help look into what the testing is and what triggers this message? I think we may be able to re-use it but would edit to be more helpful based on what you learn. Thanks! related to VerifierPlus issue: digitalcredentials/web-verifier-plus#142 Note that the VerifierPlus error message is: "JSON cannot be parsed" so will need to do this research for this implementation too. Will make a note in that issue. |
Error messaging reviewed, will need to perform the following checks:
|
@alexfigtree - I asked James about step #4 and he said that the verification core code should handle that because it needs to check if it is a VC which is good enough for us for now. So we can skip #4 and go to #5. Will add a new comment with the error message for the file type here soon. |
Updated my comment from yesterday |
I think this message should work if the file upload type is not .json or .png: File Upload Error Will add this to digitalcredentials/web-verifier-plus#142 |
Credly makes it possible for their users to download a baked badge image. This means that the .png image is encoded with an open badge 3.0. This 1EdTech reference explains the baking specification: https://www.imsglobal.org/spec/ob/v3p0
This issue is to add functionality to the LCW on the Add Credential screen. Users should be able to click on "Add from file" and upload their badge image. The code should:
In this issue at 1EdTech: there are some libraries mentioned that will perform the extraction: 1EdTech/openbadges-specification#586
including: https://www.npmjs.com/package/png-itxt
(Note offer for assistance from Credly in the 1EdTech issue if we need a hand).
The Mozilla badge bakery is outdated so we don't want to use that one.
This tool is a browser interface that can be used to manually extract the information: https://www.nayuki.io/page/png-file-chunk-inspector
Note: Out of scope are any image formats that aren't .png. The 1EdTech spec does allow for .svg but we will only be handling .pngs at this time.
Here is one badge to test with. Can provide an open badges 2.0 version if needed for testing
Related issues to link to once they are created:
The text was updated successfully, but these errors were encountered: