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

MBL-1739: Support Android 15 #2143

Draft
wants to merge 13 commits into
base: master
Choose a base branch
from
Draft

Conversation

jlplks
Copy link
Contributor

@jlplks jlplks commented Oct 3, 2024

📲 What

Spike for the task

🤔 Why

Some changes are needed for the release of android 15 like handling edge to edge

)

* Android sdk 35

- Change compile and target SDK to version 35
- change tag for CircleCI from 2024.01.1 to 2024.09.1

* downgrade tag to 2024.08.1

* 2024.07.1

* update gradle version

* add JDK_VERSION to executor environment

* UPGRADE VERSIONS

Kotlin version from 1.8.10 1.8.22
Jacoco_version from 0.8.8 to 0.8.11
compose version from 1.4.3 to 1.4.8


setted java 17 on the circle ci environment

* add validate java version on circle ci pipeline

* force java 17 on circle ci

* fix typo

* explicity install java 17

* specify jvm 17 on gradle

* force assembleExternalRelease to use java 17

* Fix lint external release

* change name of intent on ThreadView model

Changed the name for function intent and PublishSubject intent so it be more clear to read

* code more clean


* fix ktlint
… enforcement (#2131)

* Manage edge to edege

- Created utility to manage edge to edge on different screens
- fixed composable screens
- fixed other screens with the utility

* Manage edge to edge in all the screens

- Manage edge to edge behavior for all the screens
- modify solution

* Fix last screens

* Fix ktlint tests

* fix KTLint
@codecov-commenter
Copy link

codecov-commenter commented Oct 3, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 23.28767% with 56 lines in your changes missing coverage. Please review.

Project coverage is 68.76%. Comparing base (ff46cac) to head (19a0340).

Files with missing lines Patch % Lines
...arter/ui/activities/compose/ReportProjectScreen.kt 0.00% 36 Missing ⚠️
...ain/java/com/kickstarter/utils/WindowInsetsUtil.kt 0.00% 11 Missing ⚠️
.../ui/activities/compose/projectpage/AddOnsScreen.kt 0.00% 3 Missing ⚠️
...vities/compose/projectpage/RewardCarouselScreen.kt 0.00% 3 Missing ⚠️
...ckstarter/ui/activities/compose/FormularyScreen.kt 0.00% 1 Missing ⚠️
...i/activities/compose/projectpage/CheckoutScreen.kt 0.00% 1 Missing ⚠️
...java/com/kickstarter/viewmodels/ThreadViewModel.kt 90.00% 0 Missing and 1 partial ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2143      +/-   ##
============================================
- Coverage     68.88%   68.76%   -0.13%     
  Complexity     2224     2224              
============================================
  Files           365      366       +1     
  Lines         22794    22847      +53     
  Branches       3262     3261       -1     
============================================
+ Hits          15701    15710       +9     
- Misses         5320     5364      +44     
  Partials       1773     1773              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Arkariang
Copy link
Contributor

fyi @jlplks add to the title of the PR the Jira Epic ticket, and in the what/why sections you can link the spike -> https://kickstarter.atlassian.net/wiki/spaces/NT/pages/3075538945/Android+15+Spike

@jlplks jlplks changed the title Android 15 to master MBL-1739: Support Android 15 Oct 7, 2024
Arkariang
Arkariang previously approved these changes Oct 29, 2024
…2159)

* Fix pledge_container UI height bug on android 15

* fix ktlint

* set fitsSystemWindows to false on AppBarLayout on activity_proyect_page

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

Successfully merging this pull request may close these issues.

3 participants