Skip to content

feat: add endpoint to get snap logs#40

Merged
st3v3nmw merged 1 commit intocanonical:mainfrom
aliihsancengiz:support-logs-endpoint
Mar 25, 2026
Merged

feat: add endpoint to get snap logs#40
st3v3nmw merged 1 commit intocanonical:mainfrom
aliihsancengiz:support-logs-endpoint

Conversation

@aliihsancengiz
Copy link
Contributor

@aliihsancengiz aliihsancengiz commented Mar 18, 2026

Closes #39

@aliihsancengiz aliihsancengiz force-pushed the support-logs-endpoint branch from ff679fc to d4bd3dd Compare March 18, 2026 12:20
@aliihsancengiz
Copy link
Contributor Author

Would be cool if @Perfect5th and @st3v3nmw have a look.

@aliihsancengiz aliihsancengiz force-pushed the support-logs-endpoint branch from d4bd3dd to b5cd500 Compare March 23, 2026 10:23
@st3v3nmw
Copy link
Member

Hi @aliihsancengiz, I'll try review this today.

Copy link
Member

@st3v3nmw st3v3nmw 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, some small nitpicks below.

Also, please change the PR title to feat: add endpoint to get snap logs.

@aliihsancengiz aliihsancengiz force-pushed the support-logs-endpoint branch from b5cd500 to 2c4fe5a Compare March 24, 2026 08:41
@aliihsancengiz aliihsancengiz changed the title Add logs endpoint for snaps feat: add endpoint to get snap logs Mar 24, 2026
@aliihsancengiz aliihsancengiz requested a review from st3v3nmw March 24, 2026 08:41
Copy link
Member

@st3v3nmw st3v3nmw left a comment

Choose a reason for hiding this comment

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

Tested this locally and it's working as expected.

One thing we can do next is parse the application/x-ndjson result into a list of objects, I'll look into this before the next release.

LGTM.

@st3v3nmw st3v3nmw merged commit cbe8b26 into canonical:main Mar 25, 2026
4 checks passed
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.

Retrieve snap logs

2 participants