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

ICRC-39: extract icrc-39_batch_calls #105

Merged
merged 1 commit into from
Jan 4, 2024

Conversation

plitzenberger
Copy link
Collaborator

@plitzenberger plitzenberger commented Nov 22, 2023

This PR intends to extract the following standard extensions:

  • ICRC-39: Batch Calling

This is a collaborative change by @frederikrothenberger and @plitzenberger

@plitzenberger plitzenberger changed the title ICRC-25: remove batch calls ICRC-34: extract icrc-34_batch_calls Nov 30, 2023
@plitzenberger plitzenberger force-pushed the feature/icrc-25-remove-batch-calls branch from e60440a to 25cb837 Compare December 7, 2023 16:14
@plitzenberger plitzenberger changed the base branch from main to feature/icrc-25-separate-canister-calls December 7, 2023 16:14
@plitzenberger plitzenberger marked this pull request as ready for review December 7, 2023 16:15
topics/icrc_34_batch_calling.md Outdated Show resolved Hide resolved
topics/icrc_34_batch_calling.md Outdated Show resolved Hide resolved
@plitzenberger plitzenberger force-pushed the feature/icrc-25-remove-batch-calls branch from 780cb53 to fc1e8d3 Compare December 8, 2023 13:19
@plitzenberger plitzenberger force-pushed the feature/icrc-25-separate-canister-calls branch from e3f8264 to 57be97b Compare December 8, 2023 13:26
Base automatically changed from feature/icrc-25-separate-canister-calls to main December 8, 2023 13:27
@plitzenberger plitzenberger force-pushed the feature/icrc-25-remove-batch-calls branch from fc1e8d3 to d2ffe73 Compare December 8, 2023 13:28
@frederikrothenberger frederikrothenberger changed the title ICRC-34: extract icrc-34_batch_calls ICRC-3x: extract icrc-3x_batch_calls Dec 15, 2023
@frederikrothenberger
Copy link
Member

@plitzenberger : This PR now needs updating with a different number (because of #116 ).

The processing of additional requests after a failure is also changed
to avoid footguns resulting from partially successful batches.
@frederikrothenberger frederikrothenberger changed the title ICRC-3x: extract icrc-3x_batch_calls ICRC-39: extract icrc-39_batch_calls Jan 4, 2024
@frederikrothenberger
Copy link
Member

@plitzenberger : This PR now needs updating with a different number (because of #116 ).

I assigned it ICRC-39.

@frederikrothenberger
Copy link
Member

@plitzenberger: I changed the processing of additional requests after a failure to avoid footguns resulting from partially successful batches.

Copy link
Collaborator Author

@plitzenberger plitzenberger left a comment

Choose a reason for hiding this comment

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

Looks good to me. But I cannot approve as I'm the original author of the PR.

@frederikrothenberger frederikrothenberger merged commit 000e6eb into main Jan 4, 2024
2 checks passed
@frederikrothenberger frederikrothenberger deleted the feature/icrc-25-remove-batch-calls branch January 4, 2024 13:27
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.

2 participants