Skip to content

Some bug and compile error fixes#686

Open
vdods wants to merge 1 commit intospruceid:mainfrom
LedgerDomain:some-bug-and-compile-error-fixes
Open

Some bug and compile error fixes#686
vdods wants to merge 1 commit intospruceid:mainfrom
LedgerDomain:some-bug-and-compile-error-fixes

Conversation

@vdods
Copy link
Contributor

@vdods vdods commented Nov 12, 2025

Description

  • Fixed RelativeDIDURL::fragment, implemented unit test to check its path, query, and fragment methods.
  • Implemented analogous unit test for DIDURL.

Other changes

None

Optional section

  • Reviewers, please pay special attention to...

Nothing special here.

  • If this PR exceeds 500 lines, please explain why

Tested

How did you test these changes? Provide instructions so reviewers can reproduce if desired. Please also list any relevant details for your test configuration.

cargo test -p ssi-dids-core runs the added unit tests and verifies fixed behavior.

cargo test --workspace --all-features also passes.

@vdods vdods changed the base branch from main to organize-jwk November 12, 2025 09:15
…ragment methods and analogous ones for DIDURL.
@vdods vdods force-pushed the some-bug-and-compile-error-fixes branch from eec7189 to 6969afa Compare November 13, 2025 00:39
@vdods vdods changed the base branch from organize-jwk to main November 13, 2025 00:39
@vdods
Copy link
Contributor Author

vdods commented Nov 13, 2025

I had originally made this change against the organize-jwk feature branch, but I have now rebased the change against main.

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.

1 participant