Skip to content

Commit 65409d7

Browse files
committed
Update a bit
1 parent 7595e56 commit 65409d7

File tree

2 files changed

+27
-19
lines changed

2 files changed

+27
-19
lines changed

app/src/main/java/org/wikipedia/yearinreview/YearInReviewSlides.kt

Lines changed: 26 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,12 @@ package org.wikipedia.yearinreview
22

33
import org.wikipedia.R
44
import org.wikipedia.WikipediaApp
5+
import org.wikipedia.auth.AccountUtil
56

67
object YearInReviewSlides {
8+
9+
private val isEnglishWiki = WikipediaApp.instance.appOrSystemLanguageCode == "en"
10+
711
fun spentReadingHoursScreen(vararg params: Int): YearInReviewScreenData.StandardScreen {
812
// TODO: yir123
913
return YearInReviewScreenData.StandardScreen(
@@ -208,7 +212,7 @@ object YearInReviewSlides {
208212
"You saved 25 articles",
209213
"You edited Wikipedia 150 times"
210214
),
211-
headlineText = if (isLoggedIn) "Your 2025 highlights" else "2025 highlights"
215+
headlineText = "2025 highlights"
212216
)
213217
}
214218

@@ -219,19 +223,18 @@ object YearInReviewSlides {
219223
editedTimesScreen(),
220224
editedViewsScreen()
221225
)
222-
!isEditor && WikipediaApp.instance.appOrSystemLanguageCode != "en" -> {
226+
!isEditor && !isEnglishWiki -> {
223227
listOf(
224228
editorsEditsScreen(),
225229
editedPerMinuteScreen()
226230
)
227231
}
228-
!isEditor && WikipediaApp.instance.appOrSystemLanguageCode == "en" -> {
232+
else -> {
229233
listOf(
230234
editorsChangesScreen(),
231235
addedBytesScreen()
232236
)
233237
}
234-
else -> emptyList()
235238
}
236239
}
237240

@@ -310,4 +313,23 @@ object YearInReviewSlides {
310313
isEnglishWiki = true
311314
)
312315
}
316+
317+
// TODO: send all required data to this function
318+
fun finalSlides(): List<YearInReviewScreenData> {
319+
return when {
320+
AccountUtil.isLoggedIn && isEnglishWiki -> {
321+
loggedInEnglishSlides()
322+
}
323+
324+
AccountUtil.isLoggedIn && !isEnglishWiki -> {
325+
loggedInGeneralSlides()
326+
}
327+
328+
!AccountUtil.isLoggedIn && isEnglishWiki -> {
329+
nonLoggedInEnglishGeneralSlides()
330+
}
331+
332+
else -> nonLoggedInGeneralSlides()
333+
}
334+
}
313335
}

app/src/main/java/org/wikipedia/yearinreview/YearInReviewViewModel.kt

Lines changed: 1 addition & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -125,21 +125,7 @@ class YearInReviewViewModel() : ViewModel() {
125125
}
126126

127127
// TODO: build slides based on conditions: logged in, non-logged in, en vs non-en, and also based on the data we have.
128-
val finalRoute = when {
129-
AccountUtil.isLoggedIn && WikipediaApp.instance.appOrSystemLanguageCode == "en" -> {
130-
YearInReviewSlides.loggedInEnglishSlides()
131-
}
132-
133-
AccountUtil.isLoggedIn && WikipediaApp.instance.appOrSystemLanguageCode != "en" -> {
134-
YearInReviewSlides.loggedInGeneralSlides()
135-
}
136-
137-
!AccountUtil.isLoggedIn && WikipediaApp.instance.appOrSystemLanguageCode == "en" -> {
138-
YearInReviewSlides.nonLoggedInEnglishGeneralSlides()
139-
}
140-
141-
else -> YearInReviewSlides.nonLoggedInGeneralSlides()
142-
}
128+
val finalRoute = YearInReviewSlides.finalSlides()
143129

144130
// TODO: make sure return enough slides here
145131
_uiScreenListState.value = UiState.Success(

0 commit comments

Comments
 (0)