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

Added Kotlin Shnips for PEX #1199

Merged
merged 18 commits into from
Feb 6, 2024
Merged

Added Kotlin Shnips for PEX #1199

merged 18 commits into from
Feb 6, 2024

Conversation

blackgirlbytes
Copy link
Contributor

@blackgirlbytes blackgirlbytes commented Feb 5, 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 tests and kotlin snippets for Presentation Exchange. The last two cannot be converted to Kotlin because they're not implemented yet. See issue for validate submission decentralized-identity/web5-kt#133

Related Tickets & Documents

Resolves #

Mobile & Desktop Screenshots/Recordings

Added code snippets?

  • 👍 yes
  • 🙅 no, because they aren't needed

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

No tests? Add a note

Added to documentation?

  • 📜 readme
  • 📜 contributing.md
  • 📓 general documentation
  • 🙅 no documentation needed

No docs? Add a note

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?


Copy link

codesandbox bot commented Feb 5, 2024

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

Copy link

netlify bot commented Feb 5, 2024

Deploy Preview for tbd-website-developer ready!

Name Link
🔨 Latest commit dc15f28
🔍 Latest deploy log https://app.netlify.com/sites/tbd-website-developer/deploys/65c29c3453fd3100083a4e9e
😎 Deploy Preview https://deploy-preview-1199--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.

@blackgirlbytes blackgirlbytes requested a review from acekyd February 5, 2024 12:12
@blackgirlbytes
Copy link
Contributor Author

Ready for review!

@blackgirlbytes
Copy link
Contributor Author

Comment on lines +19 to +20
val signedEmploymentVcJwt = "eyJhbGciOiJFZERTQSIsInR5cCI6IkpXVCIsImtpZCI6ImRpZDprZXk6ejZNa2VyNDlDbnVnN2hzdkhEZ3Y0NHl2cGR2dE1oNHlMaURYeFM2N2huclVodHQyI3o2TWtlcjQ5Q251Zzdoc3ZIRGd2NDR5dnBkdnRNaDR5TGlEWHhTNjdobnJVaHR0MiJ9.eyJpc3MiOiJkaWQ6a2V5Ono2TWtlcjQ5Q251Zzdoc3ZIRGd2NDR5dnBkdnRNaDR5TGlEWHhTNjdobnJVaHR0MiIsInZjIjp7IkBjb250ZXh0IjpbImh0dHBzOi8vd3d3LnczLm9yZy8yMDE4L2NyZWRlbnRpYWxzL3YxIl0sInR5cGUiOlsiVmVyaWZpYWJsZUNyZWRlbnRpYWwiLCJFbXBsb3ltZW50Q3JlZGVudGlhbCJdLCJpZCI6InVybjp1dWlkOjcyNDhiOTkyLTkwOTYtNDk2NS1hMGVjLTc3ZDhhODNhMWRmYiIsImlzc3VlciI6ImRpZDprZXk6ejZNa2VyNDlDbnVnN2hzdkhEZ3Y0NHl2cGR2dE1oNHlMaURYeFM2N2huclVodHQyIiwiaXNzdWFuY2VEYXRlIjoiMjAyMy0xMi0yMVQyMDoxMToyNVoiLCJjcmVkZW50aWFsU3ViamVjdCI6eyJpZCI6ImRpZDppb246RWlEMTR4UmY0cTJNWlh1ZWY2X2ZXYnBGbVlTUG94dGFxTkp1SmdEMG96Wl84UTpleUprWld4MFlTSTZleUp3WVhSamFHVnpJanBiZXlKaFkzUnBiMjRpT2lKeVpYQnNZV05sSWl3aVpHOWpkVzFsYm5RaU9uc2ljSFZpYkdsalMyVjVjeUk2VzNzaWFXUWlPaUprZDI0dGMybG5JaXdpY0hWaWJHbGpTMlY1U25kcklqcDdJbU55ZGlJNklrVmtNalUxTVRraUxDSnJkSGtpT2lKUFMxQWlMQ0o0SWpvaWVubGFNbVYzTlhKeVVXdFVjbUV3WlZsVk16WlBTblJzTURCbFJWZHhhalZhV0dkNmNEZFpSVTVKUVNKOUxDSndkWEp3YjNObGN5STZXeUpoZFhSb1pXNTBhV05oZEdsdmJpSmRMQ0owZVhCbElqb2lTbk52YmxkbFlrdGxlVEl3TWpBaWZTeDdJbWxrSWpvaVpIZHVMV1Z1WXlJc0luQjFZbXhwWTB0bGVVcDNheUk2ZXlKamNuWWlPaUp6WldOd01qVTJhekVpTENKcmRIa2lPaUpGUXlJc0luZ2lPaUpQZDJZMFQyMUViamxKWm5SNFdYWnBkRTFHWm1jMVVXeDVMVVV6VWs1b1dsUkdPVlpFTWtnNVQzVjNJaXdpZVNJNkltUnZjVmxtV2s1c1NtRlRNVll4U201bU9HdEZObEF6VkRsd2QzaDNla3hFVTJWc1ZqTlRUa2s1U2xFaWZTd2ljSFZ5Y0c5elpYTWlPbHNpYTJWNVFXZHlaV1Z0Wlc1MElsMHNJblI1Y0dVaU9pSktjMjl1VjJWaVMyVjVNakF5TUNKOVhTd2ljMlZ5ZG1salpYTWlPbHQ3SW1sa0lqb2laSGR1SWl3aWMyVnlkbWxqWlVWdVpIQnZhVzUwSWpwN0ltVnVZM0o1Y0hScGIyNUxaWGx6SWpwYklpTmtkMjR0Wlc1aklsMHNJbTV2WkdWeklqcGJJbWgwZEhCek9pOHZaSGR1TG5SaVpHUmxkaTV2Y21jdlpIZHVOaUlzSW1oMGRIQnpPaTh2WkhkdUxuUmlaR1JsZGk1dmNtY3ZaSGR1TUNKZExDSnphV2R1YVc1blMyVjVjeUk2V3lJalpIZHVMWE5wWnlKZGZTd2lkSGx3WlNJNklrUmxZMlZ1ZEhKaGJHbDZaV1JYWldKT2IyUmxJbjFkZlgxZExDSjFjR1JoZEdWRGIyMXRhWFJ0Wlc1MElqb2lSV2xEWm05bVFUQkpVbU5uY2tWdVVHZHdRbU5RV1ZsV2VFWlliR0pTYjJRd2RVNWZRVkJwTkVrNUxVRmZRU0o5TENKemRXWm1hWGhFWVhSaElqcDdJbVJsYkhSaFNHRnphQ0k2SWtWcFFtd3pWWG80VldGT2REZGxlREJKYjJJMFJFNXNhbFJGVmpaelQwTmtjbFJ3TWxvNE5FTkJPVFJPUWtFaUxDSnlaV052ZG1WeWVVTnZiVzFwZEcxbGJuUWlPaUpGYVVOWk9WRldZbWRKYkUxemRraEZYMVJtTld4a1MxQjBkR3d3WVV4blNrdHNSbmt6Vms0d2QzQTJhVFpSSW4xOSIsImVtcGxveW1lbnRTdGF0dXMiOiJlbXBsb3llZCJ9fX0.Sazc8Ndhs-NKjxvtVMKeC9dxjEkI26fVsp2kFNWM-SYLtxMzKvl5ffeWd81ysHgPmBBSk2ar4dMqGgUsyM4gAQ"
val signedNameAndDobVcJwt = "eyJhbGciOiJFZERTQSIsInR5cCI6IkpXVCIsImtpZCI6ImRpZDprZXk6ejZNa2pwUzRHVUFoYmdCSmg2azJnZTZvWTQ0UUxyRXA3NXJadHNqYVRLb3JSRGR0I3o2TWtqcFM0R1VBaGJnQkpoNmsyZ2U2b1k0NFFMckVwNzVyWnRzamFUS29yUkRkdCJ9.eyJpc3MiOiJkaWQ6a2V5Ono2TWtqcFM0R1VBaGJnQkpoNmsyZ2U2b1k0NFFMckVwNzVyWnRzamFUS29yUkRkdCIsInZjIjp7IkBjb250ZXh0IjpbImh0dHBzOi8vd3d3LnczLm9yZy8yMDE4L2NyZWRlbnRpYWxzL3YxIl0sInR5cGUiOlsiVmVyaWZpYWJsZUNyZWRlbnRpYWwiLCJOYW1lQW5kRG9iQ3JlZGVudGlhbCJdLCJpZCI6InVybjp1dWlkOjliZjM2YzY5LTI0ODAtNDllZC1iMTYyLTRlZDEwOWE3MTc3NyIsImlzc3VlciI6ImRpZDprZXk6ejZNa2pwUzRHVUFoYmdCSmg2azJnZTZvWTQ0UUxyRXA3NXJadHNqYVRLb3JSRGR0IiwiaXNzdWFuY2VEYXRlIjoiMjAyMy0xMi0yMVQyMDowNjowMVoiLCJjcmVkZW50aWFsU3ViamVjdCI6eyJpZCI6ImRpZDppb246RWlDS2o2M0FyZlBGcEpsb2lTd3gxQUhxVWtpWlNoSDZGdnZoSzRaTl9fZDFtQTpleUprWld4MFlTSTZleUp3WVhSamFHVnpJanBiZXlKaFkzUnBiMjRpT2lKeVpYQnNZV05sSWl3aVpHOWpkVzFsYm5RaU9uc2ljSFZpYkdsalMyVjVjeUk2VzNzaWFXUWlPaUprZDI0dGMybG5JaXdpY0hWaWJHbGpTMlY1U25kcklqcDdJbU55ZGlJNklrVmtNalUxTVRraUxDSnJkSGtpT2lKUFMxQWlMQ0o0SWpvaWNscFdXbTVJVkVrNWFEWkJUVmxVV0dwT01HcFhTVkYwTTI5ak4xTnJTeTF4Y2kxcVVuSTBUalEzUlNKOUxDSndkWEp3YjNObGN5STZXeUpoZFhSb1pXNTBhV05oZEdsdmJpSmRMQ0owZVhCbElqb2lTbk52YmxkbFlrdGxlVEl3TWpBaWZTeDdJbWxrSWpvaVpIZHVMV1Z1WXlJc0luQjFZbXhwWTB0bGVVcDNheUk2ZXlKamNuWWlPaUp6WldOd01qVTJhekVpTENKcmRIa2lPaUpGUXlJc0luZ2lPaUpaVDFwRE5WSmlUMHQ1T0dadVVUWTJVWEZPUkc5aldFMXZPVXhUZEdNNVYyOWthMHd0ZFZCZlExQnZJaXdpZVNJNklsWnZZM0UxVERodFozQlhXVTFrYjFwS1JrWlJUa1ZDT0hsR0xXTndkRWQzZFdkcFRWVm5hR2t6Y21jaWZTd2ljSFZ5Y0c5elpYTWlPbHNpYTJWNVFXZHlaV1Z0Wlc1MElsMHNJblI1Y0dVaU9pSktjMjl1VjJWaVMyVjVNakF5TUNKOVhTd2ljMlZ5ZG1salpYTWlPbHQ3SW1sa0lqb2laSGR1SWl3aWMyVnlkbWxqWlVWdVpIQnZhVzUwSWpwN0ltVnVZM0o1Y0hScGIyNUxaWGx6SWpwYklpTmtkMjR0Wlc1aklsMHNJbTV2WkdWeklqcGJJbWgwZEhCek9pOHZaSGR1TG5SaVpHUmxkaTV2Y21jdlpIZHVOaUlzSW1oMGRIQnpPaTh2WkhkdUxuUmlaR1JsZGk1dmNtY3ZaSGR1TUNKZExDSnphV2R1YVc1blMyVjVjeUk2V3lJalpIZHVMWE5wWnlKZGZTd2lkSGx3WlNJNklrUmxZMlZ1ZEhKaGJHbDZaV1JYWldKT2IyUmxJbjFkZlgxZExDSjFjR1JoZEdWRGIyMXRhWFJ0Wlc1MElqb2lSV2xCTXpSMlMzb3llVmswZVV4dGRDMUdabkJuYWpWbGFFRm1ZWFI1YzFOa2MwNVNWbVpMYkhwUWRqTjVkeUo5TENKemRXWm1hWGhFWVhSaElqcDdJbVJsYkhSaFNHRnphQ0k2SWtWcFF6ZGZjMXBzTW1wMVVXNUdhRVJIV1RSb2NFVTRiMlF4YVU5MWRuZG1PVFJ5TVVkbk9HMWFWbVJCVmxFaUxDSnlaV052ZG1WeWVVTnZiVzFwZEcxbGJuUWlPaUpGYVVKdU5sTnJiSEpWYzNKdVFuaFJPVXBqVXkxTlNVaGtWelYwTXpRM1MxWjNaMXBwVEZwMFQwcDRRVkYzSW4xOSIsIm5hbWUiOiJhbGljZSBib2IiLCJkYXRlT2ZCaXJ0aCI6IjEwLTAxLTE5OTAifX19.mNCDv_JntH-wZpYONKNL58UbOWaYXCYJO_HPI_WVlSgwzo6dhYmV_9qtpFKd_exFb-aaEYPeSE43twWlrJeSBg"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I need to put this in a separate file because this file takes particularly long to load. I just need to figure out how to import these variables from a file into another file in Kotlin

@blackgirlbytes blackgirlbytes requested a review from acekyd February 6, 2024 20:07
@blackgirlbytes blackgirlbytes merged commit 6f4d371 into main Feb 6, 2024
8 of 10 checks passed
@angiejones angiejones deleted the kotlin-pex-shnips branch February 6, 2024 21:53
blackgirlbytes added a commit that referenced this pull request Feb 7, 2024
* 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]>
acekyd pushed a commit that referenced this pull request Feb 12, 2024
* 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]>
acekyd added a commit that referenced this pull request Feb 20, 2024
* Add DidJwk and update other DID methods

* Issue #1197 - Add tbdex-kt dependencies (PR #1198)

* Issue #1197 - Add tbdex-kt dependencies to Kotlin TestSuite

With a WHOLE LOT of Dark Arts Evil Nonsense to make it work:

* SNAPSHOT dependency
* Addition of a SNAPSHOTs repo
* dependencyManagement to force version resolution

This may unblock us temporarily but will not work sustainably. The SNAPSHOT dependency will go away and due to version changes from upstream projects the dependencyManagement guards will need to be updated with them.

* nuxt transpile error doc (#1200)

* Kotlin: PFI Allowlist (#1208)

* Kotlin: PFI Allowlist

* dont publish test dids

* ocd

* Added Kotlin Shnips for PEX (#1199)

* 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]>

* Removing sidebar link to JS API Guide (#1211)

* Issue #1197 - Update to locked tbdex-kt:0.9.0-beta off former SNAPSHOT

* add replyTo option for RFQs (#1207)

* 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]>

* Fan Club VC with Kotlin snippets (#1212)

* Fan Club VC with Kotlin snippets

* wrapping in a try catch

* OSS Project Dashboard: Avoid orange, yellow for badges that don't indicate failure (#1210)

tweak a few badges to avoid yellow, orange colors

this will make failures/warnings more obvious

* Presentation Definition kotlin updates (#1213)

* Presentation Definition kotlin updates

* Fix tests

* removed submission requirements

* aligned validation snippets

---------

Co-authored-by: angiejones <[email protected]>

* Removing console logs (#1217)

* update docs with references for KMS (#1216)

* Add Kotlin Snippets to PFI/Structuring a PFI App  (#1181)

* Use shnip system, still need code snippets for some parts of exchanges and offerings apis pending convo with Jiyoon

* add all snippets

* Update snippet names and use Shnip system

* fixing code

* add Kotlin code snippets

* update js server name

* update tests to run out of new testsuites directories

* renaming kotlin test file

* Update Kotlin code

* Use shnip system, still need code snippets for some parts of exchanges and offerings apis pending convo with Jiyoon

* add all snippets

* Update snippet names and use Shnip system

* fixing code

* add Kotlin code snippets

* update js server name

* update tests to run out of new testsuites directories

* renaming kotlin test file

* Update Kotlin code

* Make io.ktor.server available on the classpath

* Try to fix Kotlin compilation errors

* try to fix build

* conforming title to the rest of the guides on the site

* pushing up some changes, still addressing comments

* separate out Kotlin classes

* fix build issue with package names

* fix build

* update PFI did creation, sync names of sever var

* Apply suggestions from code review

Apply suggestions

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

* add imports

* removing dup imports

---------

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

* switching to use Shnip in JWT to VC (#1218)

* switching to Shnips in JWT to VC

* add headers

* fix path

* Adding padding on Shnip content (#1219)

* Upgrade to xyz.block:tbdex:0.10.0-beta

* update merge

* PFI/Orders Kotlin Code Snippets (#1220)

* first push for orders page to write snippets

* fix kotlin snippet and imports

* update markdown

* Adding Close Message snippets

* fixed js Close snippet

* address comments to resolve them, still need to test after wiping out most of them

* missing js ordermessage rename

* add language links

---------

Co-authored-by: Rizel Scarlett <[email protected]>
Co-authored-by: angiejones <[email protected]>

* Mock PFI Servers (#1223)

* Wallet: Get Offerings - js/kt

* added mock server and tests for Kotlin

* test data

* JS mock PFI server and tests

* Publish PFI DIDs used for tbDEX

* Changed PFI example service endpoints (#1224)

* Update DID variable names

* Updating tests and snippets

* added options to DID create

* removing pfi-structure.test.js file, doesnt compile

* tbdex version update

* fixing tests

* remove console.log

* formatting

* removing DidKey example from Create a DID guide

* removing DidKey test

* Update import snippets

* renaming variables for better readability

* update tests

---------

Co-authored-by: Andrew Lee Rubinger <[email protected]>
Co-authored-by: Angie Jones <[email protected]>
Co-authored-by: Rizel Scarlett <[email protected]>
Co-authored-by: Finn <[email protected]>
Co-authored-by: chris-tbd <[email protected]>
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