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.

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

[Android Auto] Allow for downstream upgrades #1683

Merged
merged 1 commit into from
Sep 23, 2022

Conversation

kmadsen
Copy link
Contributor

@kmadsen kmadsen commented Sep 21, 2022

Summary of changes

Adding an example of onClick requires a compileVersion 33 and a kotlin upgrade. This pull request is adding support to add onClick downstread #1682, except this change is not blocked by the repo upgrades.

Please see the full description and video in the example #1682. This change creates no functional changes, but adds the ability to customize the SurfaceCallback.

Pull request checklist:

  • Briefly describe the changes in this PR.
  • Include before/after visuals or gifs if this PR includes visual changes.
  • Write tests for all new functionality. If tests were not written, please explain why.
  • Optimize code for java consumption (@JvmOverloads, @file:JvmName, etc).
  • Add example if relevant.
  • Document any changes to public APIs.
  • Run make update-api to update generated api files, if there's public API changes, otherwise the verify-api-* CI steps might fail.
  • Update CHANGELOG.md or use the label 'skip changelog', otherwise check changelog CI step will fail.
  • If this PR is a v10.[version] release branch fix / enhancement, merge it to main firstly and then port to v10.[version] release branch.

Fixes: < Link to related issues that will be fixed by this pull request, if they exist >

PRs must be submitted under the terms of our Contributor License Agreement CLA.

@kmadsen kmadsen requested a review from a team as a code owner September 21, 2022 19:16
@kmadsen kmadsen mentioned this pull request Sep 21, 2022
9 tasks
@kmadsen kmadsen force-pushed the km-allow-callback-surface-override branch from 7ea29d8 to 334ef1d Compare September 21, 2022 19:34
@kmadsen kmadsen requested a review from a team as a code owner September 21, 2022 19:34
Copy link
Contributor

@yunikkk yunikkk left a comment

Choose a reason for hiding this comment

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

LGTM, one small q

@kmadsen kmadsen force-pushed the km-allow-callback-surface-override branch 2 times, most recently from 3613a29 to b12ba1c Compare September 22, 2022 17:18
@kmadsen kmadsen force-pushed the km-allow-callback-surface-override branch from 712ec5f to 4f1eb41 Compare September 22, 2022 17:51
Copy link
Member

@pengdev pengdev left a comment

Choose a reason for hiding this comment

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

Overall I'm ok with this as a temporary solution, dropped a few notes but in general 🟢

@kmadsen kmadsen merged commit 3f51e3f into main Sep 23, 2022
@kmadsen kmadsen deleted the km-allow-callback-surface-override branch September 23, 2022 16:02
mapbox-github-ci-writer-public-1 bot pushed a commit that referenced this pull request May 5, 2023
* bump gl-native to 10.13.0 and commo to 23.5.0
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.

4 participants