Skip to content

Conversation

calren
Copy link
Collaborator

@calren calren commented Jun 5, 2020

Internal CL: 153272

• Update non-cameraX dependencis
• Replace deprecated use of ViewModelProviders.of() with viewModels()
property delegate
• Remove unnecessary Companion references
• Replace use of this as lifecycle owner with viewLifecycleOwner

• Update non-cameraX dependencis
• Replace deprecated use of ViewModelProviders.of() with viewModels()
  property delegate
• Remove unnecessary Companion references
• Replace use of this as lifecycle owner with viewLifecycleOwner
@googlebot googlebot added the cla: yes Contributor License Agreement signed label Jun 5, 2020
@calren calren requested a review from sheepmaster June 6, 2020 15:36
Copy link
Collaborator

@sheepmaster sheepmaster left a comment

Choose a reason for hiding this comment

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

Thanks!

Is it okay to add a reference to the internal CL? That might make tracking easier.

@calren
Copy link
Collaborator Author

calren commented Jun 8, 2020

Sure! Added internal cl number in description

@calren calren merged commit 61a4fe7 into master Jun 8, 2020
@calren calren deleted the caren/merge_translate_showcase_improvements branch June 9, 2020 16:53
@sheepmaster sheepmaster added the translate-showcase Translate showcase app label Jul 13, 2020
ta-tomschell pushed a commit to ta-tomschell/mlkit that referenced this pull request Apr 4, 2025
…anslate_showcase_improvements

Translate showcase cleanups
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Contributor License Agreement signed translate-showcase Translate showcase app
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants