Skip to content

fix: presentation now selects the proper key #207

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 14, 2025
Merged

fix: presentation now selects the proper key #207

merged 1 commit into from
Apr 14, 2025

Conversation

goncalo-frade-iohk
Copy link
Contributor

@goncalo-frade-iohk goncalo-frade-iohk commented Apr 14, 2025 •
edited
Loading

Description:

There was an issue because master key should not be available with prism did document. This is a fix for the moment further refactoring is needed.

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

@goncalo-frade-iohk goncalo-frade-iohk requested a review from a team as a code owner April 14, 2025 11:23
@goncalo-frade-iohk goncalo-frade-iohk force-pushed the fix/issue-205 branch 2 times, most recently from ce16774 to 8d3875e Compare April 14, 2025 11:34
amagyar-iohk
amagyar-iohk approved these changes Apr 14, 2025
View reviewed changes
Copy link
Contributor

@amagyar-iohk amagyar-iohk left a comment

Choose a reason for hiding this comment

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

lgtm

@goncalo-frade-iohk goncalo-frade-iohk merged commit f202f9e into main Apr 14, 2025
4 checks passed
@goncalo-frade-iohk goncalo-frade-iohk deleted the fix/issue-205 branch April 14, 2025 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amagyar-iohk amagyar-iohk amagyar-iohk approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@goncalo-frade-iohk @amagyar-iohk