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

support EmailPassword confirm user #478

Merged
merged 95 commits into from
Apr 22, 2022
Merged

support EmailPassword confirm user #478

merged 95 commits into from
Apr 22, 2022

Conversation

blagoev
Copy link
Contributor

@blagoev blagoev commented Apr 19, 2022

closes #427

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 <[email protected]>
@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)
@desistefanova
Implement email/password provider
b4eaf29
@desistefanova
Implement RegisterUser in Application emailPasswordProvider
5e887a0
@desistefanova
test commented
b034700
@desistefanova
CHANGELOG.md
44625d4
@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
@blagoev blagoev marked this pull request as draft April 20, 2022 17:40
@blagoev
Merge branch 'login_user' into auth_provider_register_email
588fe1d 
# Conflicts:
#	CHANGELOG.md
#	lib/src/application.dart
#	lib/src/native/realm_core.dart
#	lib/src/realm_class.dart
#	test/test.dart
@blagoev
use persistent handle for register user call
c8848bf
@blagoev
await inside realm_core login
577b79f 
rename to _deletePersistentHandleFuncPtr to be more clear what that is
@blagoev
revert to non async method
59ef6cd
@blagoev
Merge branch 'auth_provider_register_email' into emailpassword-confir…
e3e6278 
…m-user

# Conflicts:
#	CHANGELOG.md
#	lib/src/realm_class.dart
#	test/credentials_test.dart
@blagoev
use persistent handle
3438f38
@blagoev
use named arg
5a1523e
@blagoev
rename methods
ef5e89d 
remove dead env vars usage
@blagoev
use string skip if url not present in test env
8c0deb4
@blagoev
move creation of httpTransportHandle, appConfigHandle and sycnClientC…
b37247d 
…onfigHandle inside getApp since they are not needed anywhere else. This allows the GC to collect them when out of scope. Native instances are preserved by Core
@blagoev
rename TO BAAS
9c7127d
@blagoev
rename to BAAS
7cc27d0
@blagoev
use non async login method
ec217bb
@blagoev
set skip to error if url is null
6392968
@blagoev
Merge branch 'login_user' into auth_provider_register_email
912e7f2 
# Conflicts:
#	lib/src/native/realm_core.dart
@blagoev
Merge branch 'auth_provider_register_email' into emailpassword-confir…
7bc1377 
…m-user
@blagoev
fix
2507a00
@blagoev
Merge branch 'auth_provider_register_email' into emailpassword-confir…
5d1ce11 
…m-user
@blagoev
remove dead file
622ad31
@blagoev
remove dead code
892d9c1
Base automatically changed from auth_provider_register_email to master April 21, 2022 19:36
@blagoev blagoev marked this pull request as ready for review April 21, 2022 19:41
@coveralls
Copy link

Pull Request Test Coverage Report for Build 2203979201

  • 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 2203961504: 0.0%
Covered Lines: 391
Relevant Lines: 426
💛 - Coveralls

@blagoev blagoev merged commit a13fea2 into master Apr 22, 2022
@blagoev blagoev deleted the emailpassword-confirm-user branch April 22, 2022 07:36
@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 nirinchev approved these changes

Assignees

@blagoev blagoev

Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Auth Provider: confirm user
5 participants
@blagoev @coveralls @nirinchev @nielsenko @desistefanova