-
Notifications
You must be signed in to change notification settings - Fork 12
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
Feature/real time flags (#5) #21
Merged
matthewelwell
merged 9 commits into
Flagsmith:feat/real-time
from
foresightmobile:feature/real-time-flags-squashed
Sep 11, 2023
Merged
Feature/real time flags (#5) #21
matthewelwell
merged 9 commits into
Flagsmith:feat/real-time
from
foresightmobile:feature/real-time-flags-squashed
Sep 11, 2023
Commits on Aug 25, 2023
-
* Compiles and test, need to add some tests then get it into a sample app * Should be testing but not getting the errors through Fuse so can't run into the logic * Probably gone as far as I can with 2.x fuel, let's try the 3.x * Move to Retrofit - seems to be going well so far, test runs * Tidying up, setTrait test not working * All the tests are passing so will finish the retrofit migration * Updated some of the logic and added setTraits * Checkpoint commit before trying generic converter * Generics working fine * All passing for flags and such with the new generic caching * Mostly swapped to Retrofit, now need to do the analytics * Analytics now over to retrofit * Add caching for the getFlags endpoint * Get rid of the last of Fuel * Another clear-out and all working fine on the tests * Now using Retrofit cache, remove the old stuff * Now just using HTTP caching * Delete the old caching logic * Finishing off, should be done for defaults and caching * Remove unneeded todo * Remove some more code * Still just playing around with it * Move cache configuration to its own data class * Tidy up the cache config and the tests * Update the comments * Now covers the caching tests * Tidy up some more of the tests * Some more tidying up * Default to caching disabled * Last few PR comments * Split the read and write timeout for HTTP * Initial basic implementation, let's try to get things hooked up to the server * Seems to be generally working * Checkpoint commit, seems to be generally working now just need to get the flags on update * Checkpoint commit before making the changes OK'd by Matthew to move the update clock into the event service * Ensure that the event source just reconnects if it loses the connection * Events and timers now all hooked-up and working in the manual integration test * Got the integration test working * Tidy everything up and move sensitive data to environment variables * Add a new test to cover the event stream going through a reconnect cycle * Added test for the live stream of flags, tidied up the imports and various thing, changed the logic a bit for when we need to do updates from events
Configuration menu - View commit details
-
Copy full SHA for 6061dff - Browse repository at this point
Copy the full SHA 6061dffView commit details
Commits on Sep 8, 2023
-
Update FlagsmithClient/src/test/java/com/flagsmith/RealTimeUpdatesInt…
…egrationTests.kt Co-authored-by: Matthew Elwell <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for a9a5a2e - Browse repository at this point
Copy the full SHA a9a5a2eView commit details -
Configuration menu - View commit details
-
Copy full SHA for bb51d7e - Browse repository at this point
Copy the full SHA bb51d7eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 2468611 - Browse repository at this point
Copy the full SHA 2468611View commit details -
Add some error checking on the environment variables so it's a bit mo…
…re obvious what's going on if we don't configure properly
Configuration menu - View commit details
-
Copy full SHA for 832f568 - Browse repository at this point
Copy the full SHA 832f568View commit details -
Merge branch 'feature/real-time-flags-squashed' of https://github.com…
…/foresightmobile/flagsmith-kotlin-android-client into feature/real-time-flags-squashed
Configuration menu - View commit details
-
Copy full SHA for f2c288d - Browse repository at this point
Copy the full SHA f2c288dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 7dfb107 - Browse repository at this point
Copy the full SHA 7dfb107View commit details -
Configuration menu - View commit details
-
Copy full SHA for c845a44 - Browse repository at this point
Copy the full SHA c845a44View commit details
Commits on Sep 9, 2023
-
Configuration menu - View commit details
-
Copy full SHA for a2abab6 - Browse repository at this point
Copy the full SHA a2abab6View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.