Skip to content

add OperationInfoRunFacet as required parameter to TeleopSession - #12

Merged
satoshi-kondo merged 1 commit into
mainfrom
feature/issue-136
Jan 27, 2026
Merged

add OperationInfoRunFacet as required parameter to TeleopSession#12
satoshi-kondo merged 1 commit into
mainfrom
feature/issue-136

Conversation

@satoshi-kondo

Copy link
Copy Markdown
Contributor

Summary

  • Add new OperationInfoRunFacet run facet with fields: recordId, operatorId, organization, episodeTemplateId
  • Integrate OperationInfoRunFacet as a required parameter in TeleopSession
  • Update examples and documentation

Breaking Changes

TeleopSession constructor now requires operation_info_facet parameter:

# Before
session = TeleopSession(
    namespace="...",
    common_facet=common_facet,
    device_facet=device_facet,
)

# After
session = TeleopSession(
    namespace="...",
    common_facet=common_facet,
    device_facet=device_facet,
    operation_info_facet=operation_info_facet,  # NEW - required
)

@yuya-haruna yuya-haruna 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.

LGTM

@satoshi-kondo
satoshi-kondo merged commit a8cbc72 into main Jan 27, 2026
1 check passed
@yuya-haruna
yuya-haruna deleted the feature/issue-136 branch January 27, 2026 05:21
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