Skip to content
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

feat: connectionless credential offer #207

Merged
merged 4 commits into from
Sep 19, 2024
Merged

feat: connectionless credential offer #207

merged 4 commits into from
Sep 19, 2024

Conversation

cristianIOHK
Copy link
Contributor

Description:

This PR adds support for connectionless credential offer. It adds logic to extract all relevan data to form a message than it is used for any connectionless operation.

Checklist:

  • My PR follows the contribution guidelines of this project
  • My PR is free of third-party dependencies that don't comply with the Allowlist
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked the PR title to follow the conventional commit specification

Copy link

github-actions bot commented Sep 17, 2024

Unit Test Results

152 tests  +1   149 ✅ +1   7s ⏱️ ±0s
 26 suites ±0     3 💤 ±0 
 26 files   ±0     0 ❌ ±0 

Results for commit 8f2f03c. ± Comparison against base commit e967a21.

♻️ This comment has been updated with latest results.

Signed-off-by: Cristian G <[email protected]>
Signed-off-by: Cristian G <[email protected]>
)

pluto.storeMessage(requestPresentation.makeMessage())
connectionlessInvitation(ownDID, jsonString)
Copy link
Contributor

Choose a reason for hiding this comment

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

As we were talking yesterday this is not needed

Copy link
Contributor

Choose a reason for hiding this comment

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

You need to parse and return the Credential Offer message

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 have changed the implementation. Please take another look.

@cristianIOHK cristianIOHK merged commit fce300e into main Sep 19, 2024
6 checks passed
@cristianIOHK cristianIOHK deleted the feature/ATL-7492 branch September 19, 2024 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants