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.

Sign up for GitHub

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

Jump to bottom

Added logic and UI for current streak #52

Merged
merged 57 commits into from
Nov 5, 2022
Merged

Added logic and UI for current streak #52

merged 57 commits into from
Nov 5, 2022

Conversation

Jacob3075
Copy link
Owner

Part of #39

Jacob3075 and others added 30 commits October 17, 2022 13:56
@Jacob3075
WTA #39: Updated how home page loaded state data is represented.
1b6559c
@Jacob3075
WTA #39: Updated cache and GetLast7DaysStatsUC to send correct data.
822cda8
@Jacob3075
WTA #39: Updated view model to handle new data representation.
c8864b3
@Jacob3075
WTA #39: Updated tests.
d94c0bd
@Jacob3075
WTA #39: Updated HomePageScreen to use new data representation.
cf74812
@Jacob3075
WTA #39: Updated HomePageCache to store current streak data.
8e6830d
@Jacob3075
WTA #39: Added setup for tests.
7df0f50
@Jacob3075
WTA #39: Added simple test case and code for it.
bd1ecf3
@Jacob3075
WTA #44: Added use case to get cached data.
d300324
@Jacob3075
WTA #44: Updated how last request time is calculated.
10bb8c0
@Jacob3075
WTA #44: Updated GetLast7DaysStatsUC to not have cache validity check…
de666a6 
…ing logic in it.
@Jacob3075
WTA #44: Updated use cases to only return errors.
ede6994
@Jacob3075
WTA #44: Updated how view model handles use case data.
d1254be
@Jacob3075
WTA #44: Updated use cases to not return flows.
9b582a4
@Jacob3075
WTA #44: Updated HomePageViewModel to not be an android view model.
b582230
@Jacob3075
WTA #44: Fixed GetLast7DaysStatsUC tests.
825a938
@Jacob3075
WTA #44: Updated how cached data is sent and how view model handles it.
1e637d6
@Jacob3075
WTA #44: Added separate use case to update caches.
3839e37
@Jacob3075
WTA #44: Updated GetCachedHomePageUiDataTests.
9534492
@Jacob3075
WTA #44: Started adding tests for GetCachedHomePageUiData.
7b75120
@Jacob3075
WTA #44: Added some more tests and added turbine for testing flows an…
09f8531 
…d enabled context receivers.
@Jacob3075
WTA #44: Added some more tests.
f840208
@Jacob3075
WTA #44: Added context receivers to entire app.
6c8939d
@Jacob3075
WTA #44: Updated where is last request on previous day is checked.
6729a71
@Jacob3075
WTA #44: Added test for multiple emissions and updated how mocks are …
7cea6ce 
…handled.
@Jacob3075
WTA #44: Cleaned up tests.
7cfe663
@Jacob3075
WTA #44: Renamed classes.
bbff41d
@Jacob3075
Merge pull request #45
55ae467 
Moved caching logic to separate use-cases.
@Jacob3075
WTA #44: Updated CalculateCurrentStreakUC to handle some base cases a…
c4ac24c 
…nd added tests.
@Jacob3075
WTA #44: Fixed test case.
4999884
Jacob3075 and others added 27 commits October 25, 2022 11:30
@Jacob3075
WTA #44: Added endpoint to get stats for given range.
cb112f3
@Jacob3075
WTA #44: WIP, updated CalculateCurrentStreakUC to call api to get sta…
cb66140 
…ts of previous days to calculate streak.
@Jacob3075
WTA #44: Created helper class to re-calculate streak.
1316ab4
@Jacob3075
WTA #44: Updated use case to use new helper class and updated when case.
9a61f9b
@Jacob3075
WTA #44: Fixed streak addition logic and added tests.
482b26e
@Jacob3075
WTA #44: Cleaned up CalculateCurrentStreakUC.
83ce7d7
@Jacob3075
WTA #44: Moved files around.
7b855f4
@Jacob3075
WTA #44: Added some tests for RecalculateLatestStreakService.
6732d41
@Jacob3075
WTA #44: Fixed some bugs in RecalculateLatestStreakService and update…
01c5f37 
…d to return an either.
@Jacob3075
WTA #44: Fixed some tests.
c1d4513
@Jacob3075
WTA #44: Fixed test that checks for streak across multiple months.
56d606a
@Jacob3075
WTA #44: Moved RecalculateLatestStreakService to use-cases package.
4d50496
@Jacob3075
WTA #44: Changed re-calculation start value.
6aa35e1
@Jacob3075
Merge pull request #46 from Jacob3075/WTA-39/current-streak
4935ad5 
Added logic to calculate current streak
@Jacob3075
WTA #44: Removed unused fragment navigation libs.
92c7cf1
@Jacob3075
WTA #44: Updated AGP version.
98f3db1
@Jacob3075
WTA #44: Updated lib versions and fixed previews not loading.
0f485e9
@Jacob3075
WTA #44: Updated build files to use compose bom.
69dbe2c
@Jacob3075
WTA #44: Started working on streaks card.
bfcc9e8
@Jacob3075
Added tests for bug where no data in last 7 days stats cache is not h…
6afcde3 
…andled properly.
@Jacob3075
Fixed bug where no data in last 7 days cache is not handled properly.
d6c8e09
@Jacob3075
Fixed bug where wrong parameters where passed to UserDetailsSection.
99aa996
@Jacob3075
Updated AGP version.
1a55d2f
@Jacob3075
WTA #39: Added component for current streak.
aa814cc
@Jacob3075
WTA #39: Added streak components to OtherDailyStatsSection of home page.
70ca271
@Jacob3075
Fixed bug in getting cached data where UI would not update if state d…
961c777 
…ata was available.
@Jacob3075
Merge pull request #51 from Jacob3075/WTA-39/current-streak-ui
fa563ac 
Added UI elements for current streak
@Jacob3075 Jacob3075 marked this pull request as ready for review November 5, 2022 07:35
@Jacob3075 Jacob3075 linked an issue Nov 5, 2022 that may be closed by this pull request
Add more items to home page #39
Closed
@Jacob3075 Jacob3075 merged commit f521091 into master Nov 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add more items to home page
1 participant
@Jacob3075