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

Refactor of BLoC & Sensing | Fix authentication flow | Polar disconnection exception handling #219

Merged
merged 32 commits into from
Jan 29, 2024
Merged

Refactor of BLoC & Sensing | Fix authentication flow | Polar disconnection exception handling #219

merged 32 commits into from
Jan 29, 2024

Conversation

bardram
Copy link
Contributor

@bardram bardram commented Jan 26, 2024

Major refactor of entire app - cleaning up a lot of things, especially everything that has to do with CAMS and sensing, incl. the BloC, Sensing, and Backend classes. Addresses issues;

The core CAMS framework is also updated, which should address #201

bardram and others added 26 commits January 6, 2024 14:44
@bardram @LarsRefsgaard
Lars refsgaard/health as a service (#206)
5ff3de1 
* Use and register health package

* Good start for this feature

* Update dependencies in Podfile.lock and pubspec.lock

---------

Co-authored-by: Lars Refsgaard <[email protected]>
@bardram
Fix of DataViz & Service connectivity
458f4dd 
* fix of #204
* fix of #186
* misc. refactor and update to docs
@bardram
Added support for health in manifest + plist files
7d5524e
@bardram
fix of bug in data viz #204
39228cc
@bardram
cleanup in hr dataviz
00ecbcf
@bardram
Fix of #207
4c0ef06
@bardram
adding linter and fixing all linter problems (quite a few) - except f…
b20aba2 
…or WillPopScope deprecation
@bardram
fix of imports
40768ed
@bardram
fix of data viz initialization (again)
6af9ea6
@bardram
colors added to devices and services icons
2244333
@bardram
general clean up of dead code and state enums
ba86818
@bardram
Update project.pbxproj
2ef4527
@bardram
adding route names to each page (to avoid typos in route naming)
2841127
@bardram
Using ChangeNofifier in ViewModel
618dce1
@bardram
refactor of devices page
129d272
@bardram
small fixes to LoginPage
14ba5f7
@bardram
fix of ACTIVITY_RECOGNITION permission on Android API level <28
519d346
@bardram
refactor and documentation of CarpBackend
72c32e7
@bardram
More refactor of CarpBackend and StudyAppBLoC
e234cc4
@bardram
fix of invitation view models and view to follow MVVM
732973d
@bardram
fix of StudyPage and its model to follow MVVM
ddd932f
@bardram
refactor of device page + login
9953e31 
addressing issues #214 and #215
@bardram
fix of #215
41e3df4
@bardram
more refactor of the bloc
1ce05c5
@bardram
improvement to root route + documentation
0e576e9
@bardram
update of flutter plugins in pubspec
5afe12b
@LarsRefsgaard
Copy link
Contributor

@bardram Please do git merge origin/master and resolve the conflicts :)

@bardram bardram changed the title Bardram/responsive refactor #212 Refactor of BLoC & Sensing | Fix authentication flow | Polar disconnection exception handling Jan 28, 2024
@bardram
Copy link
Contributor Author

bardram commented Jan 28, 2024

@bardram Also the title of the PR needs to be updated to reflect what is happening more precisely

👍

@cph-cachet cph-cachet deleted a comment from LarsRefsgaard Jan 28, 2024
@bardram bardram mentioned this pull request Jan 28, 2024
Closed
LarsRefsgaard
LarsRefsgaard requested changes Jan 29, 2024
View reviewed changes
lib/blocs/app_bloc.dart Show resolved Hide resolved
lib/blocs/app_bloc.dart Outdated Show resolved Hide resolved
lib/carp_study_app.dart Show resolved Hide resolved
android/app/src/main/AndroidManifest.xml Show resolved Hide resolved
ios/Runner.xcodeproj/project.pbxproj Outdated Show resolved Hide resolved
lib/data/carp_backend.dart Show resolved Hide resolved
lib/data/carp_backend.dart Outdated Show resolved Hide resolved
lib/data/carp_backend.dart Outdated Show resolved Hide resolved
lib/ui/pages/loading_page.dart Outdated Show resolved Hide resolved
pubspec.yaml Show resolved Hide resolved
@LarsRefsgaard LarsRefsgaard enabled auto-merge (squash) January 29, 2024 09:26
@LarsRefsgaard LarsRefsgaard merged commit 6ef9415 into master Jan 29, 2024
4 checks passed
@LarsRefsgaard LarsRefsgaard deleted the bardram/responsive-refactor-#212 branch January 29, 2024 11:00
Panosfunk pushed a commit that referenced this pull request Feb 7, 2024
@bardram
Refactor of BLoC & Sensing | Fix authentication flow | Polar disconne…
a967ba4 
…ction exception handling (#219)

Co-authored-by: Lars Refsgaard <[email protected]>
fixes #204, #186, #204, #207, #214, #215

Update version to 1.4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LarsRefsgaard LarsRefsgaard LarsRefsgaard approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bardram @LarsRefsgaard