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

Get current user #473

Merged
merged 53 commits into from
Apr 22, 2022
Merged

Get current user #473

merged 53 commits into from
Apr 22, 2022

Conversation

blagoev
Copy link
Contributor

@blagoev blagoev commented Apr 17, 2022

fixes #406

nielsenko and others added 30 commits March 29, 2022 09:49
@nielsenko
Rollback Dart_WeakHandle finalizer hack
453e018
@nielsenko
Use dart main/flutter beta in github actions.
8430d42 
Drop once stable is updated with finalizer fix.
Don't merge before that!
@nielsenko
Add dart specific support for realm_http_transport_new
e847de2 
Update build
@nielsenko
Wire up realm_core
6a5c9c4
@nielsenko @blagoev
Apply suggestions from code review
fca9182 
Co-authored-by: blagoev <lubo@blagoev.com>
@nielsenko
Copy all info from realm_http_request before starting async request
435e484
@nielsenko
Added ApplicationConfiguration
d7207d9
@nielsenko
Added Application class with async logIn method (includes stubs for U…
ea8d7ef 
…ser and Credentials)
@nielsenko
Use common GCHandle class for userdata on all callbacks
a54b1a8
@nielsenko
Support hard gc handles as well, as suggested by Yavor. Ie. handles t…
6ae70e2 
…hat will keep the corresponding dart object alive.
@nielsenko
Backport to dart 2.16 where we have to work around a finalization iss…
c81e0c0 
…ue on isolate shutdown: dart-lang/sdk#48321
@nielsenko
Add support for http_transport
6899dc8
@nielsenko
Added ApplicationConfiguration
9a17d45
@nielsenko
Update docs on public interface
73e2e4c
@nielsenko
Address PR feedback
0baa62f
@nielsenko
Added Application class with async logIn method (includes stubs for U…
20e95af 
…ser)
@nielsenko
Add testWithBass function
d63f205
@desistefanova
Merge branch 'master' into realm_app_config
0cdf4b7
@desistefanova
Update CHANGELOG.md
b9cc163
@desistefanova
Merge branch 'realm_app_config' into realm_app
8448ffd
@desistefanova
Commit after merge
7ff02ef
@desistefanova
Fixes after merge
fc5e481
@desistefanova
Merge branch 'realm_app'
c1cb35e
@desistefanova
Fixes after merge
e07be97
@desistefanova
Remove user and login
b7ddc96
@desistefanova
size of handles
938a168
@desistefanova
Set size of handles
d447fe0
@desistefanova
workflow chanel = stable
ef644c5
@desistefanova
Delete realm_dart_http_transport
a132bf4
@desistefanova
remove import dart:async from tests
4e15ea7
desistefanova and others added 14 commits April 12, 2022 15:38
@desistefanova
Merge branch 'master' into realm_app
1489eee
@desistefanova
Fix after merge
9fd8fec
@desistefanova
Fix after merge
07be0ae
@desistefanova
Merge branch 'realm_app' into login_user
b85cbb4
@desistefanova
Updarte CHANGELOG.md
18ec3ca
@blagoev
Merge branch 'master' into login_user
a69d013 
# Conflicts:
#	CHANGELOG.md
#	lib/src/application.dart
#	lib/src/native/realm_core.dart
#	test/application_test.dart
#	test/test.dart
@blagoev
fix build
60b665c
@blagoev
rename method
4e26d96
@blagoev
fix login invocation
b94d62c
@blagoev
fix GC hints
bad719b 
return error code from windows build script
@blagoev
[test] build in debug
333b99d
@blagoev
application holds the HttpTransport, AppConfig and SyncClient handles
e8aa395 
fix header enumeration bug
use invokeGetBool to check the app_log_in_with_credentials result
@blagoev
Merge branch 'master' into login_user
5f3ace8 
# Conflicts:
#	CHANGELOG.md
#	lib/src/realm_class.dart
@blagoev
enable app.getCurrentUser
d5a782d
@cla-bot cla-bot bot added the cla: yes label Apr 17, 2022
@blagoev blagoev changed the base branch from master to login_user April 17, 2022 22:15
@blagoev blagoev self-assigned this Apr 20, 2022
@blagoev blagoev marked this pull request as draft April 20, 2022 17:40
Base automatically changed from login_user to master April 21, 2022 19:11
@blagoev
Merge branch 'master' into get-current-user
f14f985 
# Conflicts:
#	CHANGELOG.md
#	lib/src/app.dart
#	lib/src/native/realm_core.dart
#	lib/src/realm_class.dart
#	lib/src/user.dart
#	test/test.dart
@coveralls
Copy link

coveralls commented Apr 22, 2022 •
edited
Loading

Pull Request Test Coverage Report for Build 2207594320

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 91.784%
Totals Coverage Status
Change from base Build 2207383539: 0.0%
Covered Lines: 391
Relevant Lines: 426
💛 - Coveralls

@blagoev blagoev marked this pull request as ready for review April 22, 2022 12:00
nielsenko
nielsenko approved these changes Apr 22, 2022
View reviewed changes
lib/src/native/realm_core.dart Show resolved Hide resolved
@blagoev blagoev merged commit ce53756 into master Apr 22, 2022
@blagoev blagoev deleted the get-current-user branch April 22, 2022 12:22
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@nielsenko nielsenko nielsenko approved these changes

@nirinchev nirinchev Awaiting requested review from nirinchev

Assignees

@blagoev blagoev

Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

App: get current user
4 participants
@blagoev @coveralls @nielsenko @desistefanova