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

NTV-556 : Migrate Creator Dashboard viewmodels classes to Kotlin #1639

Conversation

hadia
Copy link
Contributor

@hadia hadia commented Jun 8, 2022

📲 What

Migrate classes from java to kotlin

  1. DashboardRewardStatsRowHolderViewModel
  2. CreatorDashboardReferrerBreakdownHolderViewModel
  3. CreatorDashboardReferrerStatsHolderViewModel
  4. CreatorDashboardReferrerStatsRowHolderViewModel
  5. CreatorDashboardRewardStatsHolderViewModel
  6. CreatorDashboardViewModel

🤔 Why

Migrate All classes to java.

📋 QA

test Creator dashboard screen

Story 📖

https://kickstarter.atlassian.net/browse/NTV-556

@hadia hadia changed the title Hadia/ntv 556 migrate creator dashboard viewmodels classes to kotlin NTV-556 : Migrate Creator Dashboard viewmodels classes to Kotlin Jun 8, 2022
@codecov
Copy link

codecov bot commented Jun 8, 2022

Codecov Report

Merging #1639 (3f917bb) into master (7aafc25) will increase coverage by 0.04%.
The diff coverage is 97.92%.

@@             Coverage Diff              @@
##             master    #1639      +/-   ##
============================================
+ Coverage     78.38%   78.42%   +0.04%     
+ Complexity     1862     1861       -1     
============================================
  Files           353      353              
  Lines         16521    16573      +52     
  Branches       2034     2039       +5     
============================================
+ Hits          12950    12998      +48     
+ Misses         2408     2406       -2     
- Partials       1163     1169       +6     
Impacted Files Coverage Δ
...ava/com/kickstarter/viewmodels/BackingViewModel.kt 93.75% <ø> (ø)
...dels/CreatorDashboardBottomSheetHolderViewModel.kt 88.88% <ø> (ø)
...iewmodels/CreatorDashboardHeaderHolderViewModel.kt 97.33% <ø> (ø)
...ickstarter/viewmodels/CreatorDashboardViewModel.kt 96.38% <96.38%> (ø)
...CreatorDashboardReferrerStatsRowHolderViewModel.kt 96.55% <96.55%> (ø)
...dels/CreatorDashboardRewardStatsHolderViewModel.kt 97.91% <97.91%> (ø)
...reatorDashboardReferrerBreakdownHolderViewModel.kt 98.03% <98.03%> (ø)
...ls/CreatorDashboardReferrerStatsHolderViewModel.kt 100.00% <100.00%> (ø)
...ewmodels/DashboardRewardStatsRowHolderViewModel.kt 100.00% <100.00%> (ø)
...pp/src/main/java/com/kickstarter/models/Message.kt 79.16% <0.00%> (-2.09%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7aafc25...3f917bb. Read the comment docs.

@hadia hadia requested review from Arkariang and leighdouglas June 9, 2022 11:13
@hadia hadia added the migration Migration Java to Kotlin label Jun 9, 2022
@hadia hadia marked this pull request as ready for review June 9, 2022 11:14
Copy link
Contributor

@Arkariang Arkariang left a comment

Choose a reason for hiding this comment

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

lgtm! just a small comment, all the @JvmField annotations can be removed

@hadia hadia merged commit 95a7e4f into master Jun 15, 2022
@hadia hadia deleted the hadia/NTV-556_Migrate_Creator_Dashboard_viewmodels_classes_to_Kotlin branch June 15, 2022 13:48
Arkariang added a commit that referenced this pull request Jun 15, 2022
…tin/new-architecture

* 'master' of github.com:kickstarter/android-oss:
  NTV-556 :  Migrate Creator Dashboard viewmodels classes to Kotlin (#1639)
  WEB-532 : [Android] Canceled backer: Remove "show comment" link from comment tombstones (#1641)
  [no-jira]: remove redundant observable subscription (#1640)
  PAY-1692: Shipping label when changing rewards to Local pickup reward (#1638)
Arkariang added a commit that referenced this pull request Jul 6, 2022
…tin/PAY-1693-integrate-paymentsheet

* 'master' of github.com:kickstarter/android-oss:
  WEB-605: Update string for flagged comments (#1650)
  NTV-563 : Migrate Message related viewmodels classes to Kotlin (#1649)
  WEB-567: Show flagged comment UI/state when pending T&S review (#1648)
  - Release 3.3.0
  Comments from canceled backers with no replies do not appear in the list of comments (#1645)
  NTV-561: Update dependencies  (#1646)
  WEB-566: [Android] UI for flagged comment (#1643)
  NTV-560: Updated gems (#1644)
  PAY-1716: Update stripe SDK (#1642)
  NTV-556 :  Migrate Creator Dashboard viewmodels classes to Kotlin (#1639)
  WEB-532 : [Android] Canceled backer: Remove "show comment" link from comment tombstones (#1641)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
migration Migration Java to Kotlin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants