Skip to content
This repository has been archived by the owner on Dec 12, 2024. It is now read-only.

VC workflow with Kotlin Code snippets #1173

Closed
wants to merge 45 commits into from
Closed

Conversation

blackgirlbytes
Copy link
Contributor

@blackgirlbytes blackgirlbytes commented Jan 29, 2024

What type of PR is this? (check all applicable)

  • ♻️ Refactor
  • ✨ New Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 👷 Example Application
  • 🧑‍💻 Code Snippet
  • 🎨 Design
  • 📖 Content
  • 🧪 Tests
  • 🔖 Release
  • 🚩 Other

Description

This PR adds code snippets for VC workflow. Added code snippets; no tests

Related ticket

https://app.asana.com/0/1204316590307635/1206268818053074

Copy link

codesandbox bot commented Jan 29, 2024

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

Copy link

netlify bot commented Jan 29, 2024

Deploy Preview for tbd-website-developer ready!

Name Link
🔨 Latest commit 00d5db7
🔍 Latest deploy log https://app.netlify.com/sites/tbd-website-developer/deploys/65c37eb1f0a8d40008a1e5c4
😎 Deploy Preview https://deploy-preview-1173--tbd-website-developer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@angiejones
Copy link
Member

@blackgirlbytes let me know the error you're getting with the validation. I ran into some problems early on too. wondering if yours is the same

@blackgirlbytes blackgirlbytes marked this pull request as ready for review January 30, 2024 15:13
@blackgirlbytes
Copy link
Contributor Author

blackgirlbytes commented Jan 30, 2024

@blackgirlbytes let me know the error you're getting with the validation. I ran into some problems early on too. wondering if yours is the same

@angiejones I'm getting the error below. I could be completely wrong, so if you have a different issue let me know..but I think validateDefinition is something that Neal added to web5-kt AFTER the most recent release of the SDK. The version I have (which I think is the most recent release) only does satisifiesPresentation from what I read in the source code.Maybe we can make a ticket to update the guides after they do the release.

I think what I have should work for now.

Screenshot 2024-01-30 at 10 13 07 AM

@blackgirlbytes blackgirlbytes requested a review from acekyd January 30, 2024 15:25
@blackgirlbytes blackgirlbytes changed the title VC workflow with Kotlin Code snippets (WIP) VC workflow with Kotlin Code snippets Jan 30, 2024
@blackgirlbytes
Copy link
Contributor Author

This is ready for review!

@blackgirlbytes
Copy link
Contributor Author

angiejones and others added 8 commits February 7, 2024 07:41
* Kotlin: PFI Allowlist

* dont publish test dids

* ocd
* kotlin shnips

* adding new line

* kotlin shnip wasnt generating

* removing extra code snippet

* enabling switch content

* adding clarity for presentation submission with Kotlin

* showing language switcher at the top

* adding back title

* adding back title

* hardcode imports

* make verifiable presentation more obvious

* put language switcher at the top

* remove unnnecessary import

* possible note for missing method

* PR #1199 - Fix jackson.core to a more compatible version

* handle all imports with model.*

* add in as snippet

---------

Co-authored-by: Andrew Lee Rubinger <[email protected]>
* add replyTo option for RFQs

* Update site/docs/tbdex/wallet/send-rfq.mdx

Co-authored-by: Angie Jones <[email protected]>

---------

Co-authored-by: Angie Jones <[email protected]>
@blackgirlbytes
Copy link
Contributor Author

I rebased and pulled in unnecessary POM changes. I'm going to just close this PR and create a new one to reduce any chances of messing up our dependency management

@angiejones angiejones deleted the kotlin-vc-workflow branch February 7, 2024 16:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants