Skip to content
This repository has been archived by the owner on Nov 19, 2024. It is now read-only.

feat: [IC-272] canister logging #263

Merged
merged 45 commits into from
May 23, 2024
Merged

feat: [IC-272] canister logging #263

merged 45 commits into from
May 23, 2024

Conversation

maksymar
Copy link
Contributor

@maksymar maksymar commented Dec 14, 2023

This PR introduces the API changes needed to support MVP for canister logging.

@maksymar maksymar marked this pull request as ready for review December 14, 2023 14:54
@maksymar maksymar requested a review from a team as a code owner December 14, 2023 14:54
@maksymar maksymar requested review from ielashi and mraszyk December 14, 2023 14:54
@maksymar maksymar changed the title [IC-272] canister logging feat: [IC-272] canister logging Dec 14, 2023
spec/_attachments/ic.did Outdated Show resolved Hide resolved
spec/_attachments/ic.did Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/_attachments/ic.did Outdated Show resolved Hide resolved
spec/_attachments/ic.did Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
mraszyk

This comment was marked as resolved.

spec/_attachments/ic.did Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
@maksymar maksymar requested review from ielashi and mraszyk January 15, 2024 14:54
Copy link
Contributor

@mraszyk mraszyk left a comment

Choose a reason for hiding this comment

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

Thank you! LGTM (I'll approve once the feature is ready to be released so that we don't accidentally merge the spec changes earlier.)

@maksymar

This comment was marked as resolved.

spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
@maksymar
Copy link
Contributor Author

maksymar commented Mar 18, 2024

hi all, the feature is going to be available on the next dfx release, I suppose we can move forward with merging this PR.

please take another look and LGTM!

spec/index.md Outdated Show resolved Hide resolved
@Dfinity-Bjoern

This comment was marked as resolved.

Copy link
Member

@Dfinity-Bjoern Dfinity-Bjoern left a comment

Choose a reason for hiding this comment

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

Please merge in sync with release in prod

@Stefs-2142
Copy link

Really looking forward to this feature!

spec/index.md Outdated Show resolved Hide resolved
@mraszyk

This comment was marked as resolved.

spec/index.md Show resolved Hide resolved
@maksymar

This comment was marked as resolved.

@maksymar

This comment was marked as resolved.

@mraszyk mraszyk merged commit 17ae77a into master May 23, 2024
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants