Skip to content

Android Auto-aware Assist Triggering#6710

Draft
lowlyocean wants to merge 3 commits intohome-assistant:mainfrom
lowlyocean:android_auto_assist
Draft

Android Auto-aware Assist Triggering#6710
lowlyocean wants to merge 3 commits intohome-assistant:mainfrom
lowlyocean:android_auto_assist

Conversation

@lowlyocean
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @lowlyocean

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

@jpelgrom
Copy link
Copy Markdown
Member

Hi 👋

I noticed you recently submitted a similar PR to the iOS app: home-assistant/iOS#4496

Did you fully prepare this one using AI systems as well? Did you review and test the changes, and consider Android Auto(motive) policy requirements?

@lowlyocean
Copy link
Copy Markdown
Author

Hello again! I failed to realize both projects likely have the same reviewers

This one was AI-assisted as well but unlike the iOS PR it is something I've been testing on a real device connectes to Device Head Unit emulator. It's still very much a draft despite being functional in a crude sense.

I think the policy requirements are somewhat similar to iOS and unlike the CarPlay version it's not clear how to trigger directly into an Assist screen without navigating from a MainVehicleScreen. ln this case, an argument for being compliant with policy might involve separating Android Auto Assist into its own "App"

Copy link
Copy Markdown

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @lowlyocean

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants