Skip to content

Improve multi-canister skill#31

Merged
alin-at-dfinity merged 1 commit into
mainfrom
alin/multi-canister
Feb 27, 2026
Merged

Improve multi-canister skill#31
alin-at-dfinity merged 1 commit into
mainfrom
alin/multi-canister

Conversation

@alin-at-dfinity

Copy link
Copy Markdown
Contributor

Drop a couple of false statements. Add a How It Works section. Add a bunch of Mistakes That Break Your Build entries.

Drop a couple of fals statements. Add a How It Works section. Add a bunch of Mistakes That Break Your Build entries.
@alin-at-dfinity

Copy link
Copy Markdown
Contributor Author

I have no experience with this whatsoever, so I don't know if my text isn't too verbose or too vague. But I'm pretty sure that the points themselves are highly relevant.

Comment thread public/llms-full.txt
Comment thread public/llms-full.txt

@derlerd-dfinity derlerd-dfinity left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks a lot. LGTM.

One general thing I'm wondering is whether we should recommend using bounded wait calls more strongly? For example we could adapt the code of the content canister so that it makes a bounded wait call to the user canister.

Comment thread public/llms-full.txt
Comment thread public/llms-full.txt
Comment thread public/llms-full.txt

@alin-at-dfinity alin-at-dfinity left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

One general thing I'm wondering is whether we should recommend using bounded wait calls more strongly? For example we could adapt the code of the content canister so that it makes a bounded wait call to the user canister.

Yeah, I didn't actually get as far as the code. (o:

But it would definitely make more sense to have bounded wait as the default.

Comment thread public/llms-full.txt
@alin-at-dfinity alin-at-dfinity merged commit 9817801 into main Feb 27, 2026
6 checks passed
@marc0olo marc0olo deleted the alin/multi-canister branch February 27, 2026 14:46
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