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

Data source: Shared Preferences #17

Closed
1 task done
JoaquimLey opened this issue Mar 26, 2018 · 0 comments
Closed
1 task done

Data source: Shared Preferences #17

JoaquimLey opened this issue Mar 26, 2018 · 0 comments

Comments

@JoaquimLey
Copy link
Owner

JoaquimLey commented Mar 26, 2018

Subtask of #69

Tasks ⚙

  • Implement with SharedPreferences for first version
@JoaquimLey JoaquimLey added this to the MVP milestone Mar 26, 2018
@JoaquimLey JoaquimLey mentioned this issue Mar 26, 2018
6 tasks
JoaquimLey added a commit that referenced this issue Sep 30, 2018
- Start wiring up with the domain layer, FavoritesrepositoryImpl and tests
- Scaffold TransportDataStore (still no implementation or sources)
  - Still missing tests for these

#62
See also: #63

Local cache starting to creep in ;) #17
@JoaquimLey JoaquimLey self-assigned this Oct 13, 2018
@JoaquimLey JoaquimLey mentioned this issue Oct 13, 2018
8 tasks
@JoaquimLey JoaquimLey changed the title MVP: Local cache Data source: Shared Preferences Oct 13, 2018
JoaquimLey added a commit that referenced this issue Oct 14, 2018
Some basic ideas just so I know what I'm supposed to continue, a lot of TODOs
so lint warns me.

For #17
Ref #69
JoaquimLey added a commit that referenced this issue Oct 16, 2018
This will be included in the stblib from Kotlin 1.3.

#17, #69
JoaquimLey added a commit that referenced this issue Oct 16, 2018
Also, fixes and issue where the String was default to  which made the serialization fail:
 - Kotlin/kotlinx.serialization#115

#17, #69
JoaquimLey added a commit that referenced this issue Oct 16, 2018
Had some small conflits, nothing major.

#17 #19
JoaquimLey added a commit that referenced this issue Oct 17, 2018
- Created necessary factory/mock data classes for the SharedPref module

Named the mock classes with the SharedPref prefix for #66

#17 #69
JoaquimLey added a commit that referenced this issue Oct 17, 2018
Lacking SharedPreferences test setup
  - Will probably need to move all the Android test libs to androidx variant.

#17 #69
JoaquimLey added a commit that referenced this issue Oct 17, 2018
)

* Scaffold sharedpreferences module

- Attach to store implemetnation
- Mock data currently being returned

Some housekeeping.

* Small configurations for CI

Still not working, wanted to make mock work with debug but variants still compain.

* Test removing ViewModelModule

* Ci why you no work?

* Fix lints on CI

* Working stuff for CI

* housekeeping mobile-ui/build.gradle

Had to make some changtes while debuging CI fails. This change still makes sense IMO.

#69

* Rename data package to include project name
#69

* Remove unecessary .gitignore from other packages

Maintenece commit, not really related to this branch feature.

* Manifest cleanup and format

* Scaffold stuff for stream episode 17

Some basic ideas just so I know what I'm supposed to continue, a lot of TODOs
so lint warns me.

For #17
Ref #69

* Housekeeping from previous commit

* Fix typo in readme

Changed the copy on the stream log section

* Include kotlinx.serialization experimental library for sharedPrefs

This will be included in the stblib from Kotlin 1.3.

#17, #69

* Make SharedPrefTransport model @kotlinx.serialization.Serializable

Also, fixes and issue where the String was default to  which made the serialization fail:
 - Kotlin/kotlinx.serialization#115

#17, #69

* SharedPrefMapper tests

- Created necessary factory/mock data classes for the SharedPref module

Named the mock classes with the SharedPref prefix for #66

#17 #69

* Initial tests for FrameworkLocalStorageImpl

Lacking SharedPreferences test setup
  - Will probably need to move all the Android test libs to androidx variant.

#17 #69

* Include stream episode PR to README
JoaquimLey added a commit that referenced this issue Oct 22, 2018
* Scaffold sharedpreferences module

- Attach to store implemetnation
- Mock data currently being returned

Some housekeeping.

* Small configurations for CI

Still not working, wanted to make mock work with debug but variants still compain.

* Test removing ViewModelModule

* Ci why you no work?

* Fix lints on CI

* Working stuff for CI

* housekeeping mobile-ui/build.gradle

Had to make some changtes while debuging CI fails. This change still makes sense IMO.

#69

* Rename data package to include project name
#69

* Remove unecessary .gitignore from other packages

Maintenece commit, not really related to this branch feature.

* Manifest cleanup and format

* Scaffold stuff for stream episode 17

Some basic ideas just so I know what I'm supposed to continue, a lot of TODOs
so lint warns me.

For #17
Ref #69

* Housekeeping from previous commit

* Fix typo in readme

Changed the copy on the stream log section

* Include kotlinx.serialization experimental library for sharedPrefs

This will be included in the stblib from Kotlin 1.3.

#17, #69

* Make SharedPrefTransport model @kotlinx.serialization.Serializable

Also, fixes and issue where the String was default to  which made the serialization fail:
 - Kotlin/kotlinx.serialization#115

#17, #69

* Implement initial SharedPref slot save logic

#17, #69

* Add stream ep to README, ignore failing test on shared pref

#17, #69
JoaquimLey added a commit that referenced this issue Oct 28, 2018
Still missing clear all, getAll and make sure observeSharedPreferencesChanges
emitts for any changes (missing tests basically).

#17, #69
JoaquimLey added a commit that referenced this issue Oct 28, 2018
* Final touches on frameworkLocalStorageTest

Still missing clear all, getAll and make sure observeSharedPreferencesChanges
emitts for any changes (missing tests basically).

#17, #69

* Add episode to README

#84
JoaquimLey added a commit that referenced this issue Oct 30, 2018
* Scaffold sharedpreferences module

- Attach to store implemetnation
- Mock data currently being returned

Some housekeeping.

* Small configurations for CI

Still not working, wanted to make mock work with debug but variants still compain.

* Test removing ViewModelModule

* Ci why you no work?

* Fix lints on CI

* Working stuff for CI

* housekeeping mobile-ui/build.gradle

Had to make some changtes while debuging CI fails. This change still makes sense IMO.

#69

* Rename data package to include project name
#69

* Remove unecessary .gitignore from other packages

Maintenece commit, not really related to this branch feature.

* Manifest cleanup and format

* Scaffold stuff for stream episode 17

Some basic ideas just so I know what I'm supposed to continue, a lot of TODOs
so lint warns me.

For #17
Ref #69

* Housekeeping from previous commit

* Fix typo in readme

Changed the copy on the stream log section

* Include kotlinx.serialization experimental library for sharedPrefs

This will be included in the stblib from Kotlin 1.3.

#17, #69

* Make SharedPrefTransport model @kotlinx.serialization.Serializable

Also, fixes and issue where the String was default to  which made the serialization fail:
 - Kotlin/kotlinx.serialization#115

#17, #69

* Test for SharedPreferences all slots free, saves to Slot.ONE

#17, #69

* Final touches on frameworkLocalStorageTest

Still missing clear all, getAll and make sure observeSharedPreferencesChanges
emitts for any changes (missing tests basically).

#17, #69

* Add episode to README

#84

* Stream Episode 19 (#84)

* Final touches on frameworkLocalStorageTest

Still missing clear all, getAll and make sure observeSharedPreferencesChanges
emitts for any changes (missing tests basically).

#17, #69

* Add episode to README

#84

* Add klint link

* Create stub test class on :presentation

This is a reminder for eventual instrumentation tests, stll not sure if makes
sense to use though.
JoaquimLey added a commit that referenced this issue Feb 14, 2019
Feature: Data source from Android `SharedPreferences`

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

No branches or pull requests

1 participant