Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[examples/tv-app] Response handling missing for Account Login cluster for Content Apps #23550

Closed
tomasn-spotify opened this issue Nov 8, 2022 · 1 comment · Fixed by #24628
Closed

[examples/tv-app] Response handling missing for Account Login cluster for Content Apps #23550

tomasn-spotify opened this issue Nov 8, 2022 · 1 comment · Fixed by #24628
Assignees
@amitnj
Labels
tv TV-related features

Comments

@tomasn-spotify
Copy link

tomasn-spotify commented Nov 8, 2022 •
edited
Loading

Feature description

Response handling for the Account Login cluster for Content Apps is currently missing in the example tv-app.

See https://github.com/project-chip/connectedhomeip/blob/master/examples/tv-app/android/java/ContentAppCommandDelegate.cpp#L190:

// case app::Clusters::AccountLogin::Id:

Would be great to get that added to complete the TV example code.

Platform

android
@bzbarsky-apple bzbarsky-apple added the tv TV-related features label Nov 10, 2022
@tomasn-spotify
Copy link
Author

@amitnj - Here is the issue for missing Account Login cluster handling, as discussed on Slack.

@amitnj amitnj self-assigned this Jan 24, 2023
@amitnj amitnj mentioned this issue Jan 24, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@amitnj amitnj

Labels
tv TV-related features
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for account login cluster for content apps. amitnj/connectedhomeip
3 participants
@bzbarsky-apple @amitnj @tomasn-spotify