GodotNuts / GodotFirebase

Implementations of Firebase for Godot using GDScript
MIT License
554 stars 79 forks source link

Firebase compliant Authorization Signals #81

Closed fenix-hub closed 3 years ago

fenix-hub commented 3 years ago

Before this PR, each authorization function implemented by the Authorization module @Firebase.Auth would emit the login_succeeded signal upon successful sign-up and sign-in. With this PR, two different signals will be now emitted:

In this way, a client-side logic to differentiate sign-up from sign-in would be easier to implement. Also, this change won't force users to change their signal connection logics: it is much more easier to connect two different signals (login_succeeded and signup_succeeded) to the same function (example _on_login_signup_succeeded()) rather than connecting the same signal to two different functions and write custom logics.

NOTE please remember to add the signal connection to the new signal in already implemented projects.