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

Update https-outcalls-get.md #2342

Closed
wants to merge 2 commits into from

Conversation

dipanshuhappy
Copy link

@dipanshuhappy dipanshuhappy commented Jan 13, 2024

Update the rust http get call for latest ic_cdk ^ 0.9.0 ,added cycles and context changes

Thank you for your contribution to the IC Developer Portal.
Before submitting your Pull Request, please make sure that:

Update the rust http get call for latest ic_cdk ^ 0.9.0 ,added cycles and context changes
@sa-github-api
Copy link

Dear @dipanshuhappy,

In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA.

If you decide to agree with it, please visit this issue and read the instructions there. Once you have signed it, re-trigger the workflow on this PR to see if your code can be merged.

— The DFINITY Foundation

@letmejustputthishere
Copy link
Member

i'll close this and merge this PR instead that added a couple more fixes. thank you for your contributions and making us aware of this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants