Skip to content

Return episode recording locations - #164

Merged
Mo-Elshenawy1 merged 1 commit into
mainfrom
mostafa/dc-1801-episode-recording-location
Sep 29, 2026
Merged

Mo-Elshenawy1 merged 1 commit into
mainfrom
mostafa/dc-1801-episode-recording-location

Conversation

@Mo-Elshenawy1

@Mo-Elshenawy1 Mo-Elshenawy1 commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Changelog

Episode recordings returned with include_recordings=True now include their location when available.

Docs

None

Description

Passes each episode recording's location through, so methods that return episode recordings include it. It's None for recordings without a location.

Part of DC-1801

The Foxglove API now includes `location` (bucket, object key, and Azure
storage account when set) on episode recordings requested with
`include=recordings`, for recordings indexed in place in a customer-managed
bucket. `_episode_recording_dict` copies known fields only, so pass
`location` through, as `None` when the API omits it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019TegazNERbHFBWDR5CFJ6m
@linear-code

linear-code Bot commented Sep 28, 2026

Copy link
Copy Markdown

DC-1801

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Mo-Elshenawy1
Mo-Elshenawy1 merged commit 18b61e0 into main Sep 29, 2026
2 checks passed
@Mo-Elshenawy1
Mo-Elshenawy1 deleted the mostafa/dc-1801-episode-recording-location branch September 29, 2026 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants