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: migrate http request and response types from ic-response-verification to ic-http-certification #241

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

nathanosdev
Copy link
Member

@nathanosdev nathanosdev commented Dec 4, 2023

This PR moves (and renames) the Request and Response types from ic-response-verification to ic-http-certification as HttpRequest and HttpResponse. The WASM conversion that was associated with these types has been moved to ic-response-verification-wasm.

@nathanosdev nathanosdev self-assigned this Dec 4, 2023
@nathanosdev nathanosdev force-pushed the nathan/move-request-response-types branch 2 times, most recently from 7e59525 to 4027ef4 Compare December 4, 2023 09:55
…cation to ic-http-certification

BREAKING CHANGE:
@nathanosdev nathanosdev force-pushed the nathan/move-request-response-types branch from 4027ef4 to 78e7b4a Compare December 4, 2023 10:03
@nathanosdev nathanosdev marked this pull request as ready for review December 4, 2023 10:19
@nathanosdev nathanosdev requested a review from a team as a code owner December 4, 2023 10:19
Copy link
Member

@keplervital keplervital left a comment

Choose a reason for hiding this comment

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

LGTM

@nathanosdev nathanosdev merged commit 8155414 into main Dec 4, 2023
6 checks passed
@nathanosdev nathanosdev deleted the nathan/move-request-response-types branch December 4, 2023 12:58
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.

2 participants