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

Updated dependencies to allow usage of Kotlin 1.9, AGP 8+ and latest … #234

Conversation

nunofvalente
Copy link

@nunofvalente nunofvalente commented Sep 7, 2023

…Compose version

Issue

issue#208

Proposed changes

Frames SDK contains a lot of dependencies that are old and are an obstacle for companies to upgrade their codebases to use the latest dependencies as it is usually not supported by this SDK.

As we want to migrate to the newest Kotlin, AGP and Compose version we need Frames to be able to handle these versions as it was crashing our app. There is already an issue where people are not able to update their dependencies due to conflicts.

Migrated following dependecies:

  • Dagger
  • AGP 8.1.0
  • Kotlin to 1.9
  • Compose dependencies
  • Mockk
  • Other dependencies that were affected by these changes.

The application now compiles with Java 17 instead of Java 8 as it is a requirement for AGP 8.1.0.

Test Steps

All the tests have passed. Didn't introduce new code, only changed what was not supported or deprecated.
I have tested the apps and they were working fine.

Checklist

  • Reviewers assigned
  • I have performed a self-review of my code and manual testing
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if applicable)

Further comments

Most of the files were probably due to me optimising the imports.

MonochromeLauncherIcon lint issue is due to not having a monochromatic icon.
More information here: MonochromeLauncherIcon

@nunofvalente nunofvalente requested a review from a team as a code owner September 7, 2023 15:56
@nunofvalente nunofvalente marked this pull request as draft September 7, 2023 16:10
@nunofvalente nunofvalente marked this pull request as ready for review September 7, 2023 16:10
@nunofvalente nunofvalente force-pushed the feature/update-kotlin-to-1.9-and-compose branch from b9f437d to 10496d5 Compare September 8, 2023 09:24
@chintan-soni-cko
Copy link
Contributor

@nunofvalente Thanks for raising this PR. We will review it, and prioritise this tech stack upgrade soon for our future releases.

@PuhalschiAlexandru
Copy link

Sorry to bother you @chintan-soni-cko, Any updates on this PR? We have problems with SDK after migrating to a new Kotlin, AGP and Compose versions on our project, will help a lot if there is an ETA on the merge for this PR. Thanks!

@patrick-hoban-cko
Copy link
Contributor

Hey @PuhalschiAlexandru, we do plan to start making this upgrade next week once we finish some other priority work.

I'll come back on here once we have more concrete timelines for releasing a new version of our Frames Android SDK. Thank you for your patience in the meantime!

@fabio-insolia-cko
Copy link
Contributor

This PR is done as part of #245

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

Successfully merging this pull request may close these issues.

5 participants