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.

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

feat: Accept contentType params for DID URL Resolution [DEV-4722] #325

Open
wants to merge 14 commits into
base: develop
Choose a base branch
from
Open

feat: Accept contentType params for DID URL Resolution [DEV-4722] #325

wants to merge 14 commits into from
+847 −157

Conversation

DaevMithran
Copy link
Contributor

No description provided.

@ankurdotb
Copy link
Contributor

Task linked: DEV-4722 Apply Accept header logic for DID URL Dereferencing

@DaevMithran DaevMithran marked this pull request as draft January 31, 2025 09:21
@ankurdotb ankurdotb changed the title feat: Accept contetType params for DID URL Resolution feat: Accept contentType params for DID URL Resolution Jan 31, 2025
@ankurdotb ankurdotb changed the title feat: Accept contentType params for DID URL Resolution feat: Accept contentType params for DID URL Resolution [DEV-4722] Jan 31, 2025
@DaevMithran DaevMithran marked this pull request as ready for review February 6, 2025 10:14
@DaevMithran DaevMithran force-pushed the DEV-4722 branch 7 times, most recently from fa1dcad to 35be15c Compare February 6, 2025 13:26
@DaevMithran DaevMithran force-pushed the DEV-4722 branch 2 times, most recently from df4017b to 35169aa Compare February 6, 2025 13:36
ankurdotb
ankurdotb requested changes Feb 7, 2025
View reviewed changes
services/helpers.go Outdated Show resolved Hide resolved
services/resource/echo_handlers.go Outdated Show resolved Hide resolved
types/constants.go Show resolved Hide resolved
types/resource_dereferencing.go Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Eengineer1 Eengineer1 Awaiting requested review from Eengineer1

@ankurdotb ankurdotb Awaiting requested review from ankurdotb

@sownak sownak Awaiting requested review from sownak

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@DaevMithran @ankurdotb @sownak @Eengineer1