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: Allow DID fragment de-referencing [DEV-1172] #4

Merged
merged 21 commits into from
May 19, 2022

Conversation

Toktar
Copy link
Contributor

@Toktar Toktar commented Apr 20, 2022

No description provided.

@askolesov askolesov self-requested a review May 13, 2022 12:20
askolesov
askolesov previously approved these changes May 13, 2022
@askolesov
Copy link
Contributor

Great work!

Copy link
Contributor

@ankurdotb ankurdotb left a comment

Choose a reason for hiding this comment

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

@Toktar There are a few minor changes here but I'm also don't expect you to fix this before you go on holiday. Will pick it up with Alex K while you're out.

I'm also going to set this to a draft PR and block merging until we resolve DEV-1322 since we have zero Go linting and quality checks at the moment on this repo. Will plan that out with Alex.

config.yaml Outdated Show resolved Hide resolved
config.yaml Show resolved Hide resolved
config.yaml Outdated Show resolved Hide resolved
services/request_service_test.go Outdated Show resolved Hide resolved
types/dereverecing_metadata.go Outdated Show resolved Hide resolved
@ankurdotb ankurdotb marked this pull request as draft May 13, 2022 15:03
@askolesov
Copy link
Contributor

Implemented this task in scope of the PR:
https://app.clickup.com/t/6600954/DEV-1327

@askolesov askolesov marked this pull request as ready for review May 16, 2022 17:26
@ankurdotb ankurdotb marked this pull request as draft May 17, 2022 07:39
@askolesov askolesov requested a review from ankurdotb May 17, 2022 08:09
@ankurdotb ankurdotb marked this pull request as ready for review May 18, 2022 21:53
Copy link
Contributor

@ankurdotb ankurdotb left a comment

Choose a reason for hiding this comment

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

Waiting for linter errors to be resolved

Andrew Nikitin and others added 3 commits May 19, 2022 18:29
Signed-off-by: Andrew Nikitin <andrew.nikitin@cheqd.io>
Signed-off-by: Andrew Nikitin <andrew.nikitin@cheqd.io>
@ankurdotb ankurdotb changed the title Dev-1172: dereferencing feat: Allow DID fragment de-referencing [DEV-1172] May 19, 2022
@lampkin-diet lampkin-diet merged commit faa7a47 into main May 19, 2022
@lampkin-diet lampkin-diet deleted the dev-1172-dereferencing branch May 19, 2022 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants