SaveFamily/sf-app-platform
11
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

added recoverPassword endpoints, providers and states #10

Merged
julianalcala merged 6 commits from auth-recover-password into develop 2025-12-19 08:46:56 +00:00
Conversation 1 Commits 6 Files Changed 36 +1497 -481
aitorarana commented 2025-12-12 10:33:18 +00:00
Member
Copy Link
No description provided.
aitorarana added 1 commit 2025-12-12 10:33:18 +00:00
added recoverPassword endpoints, providers and states 098217f47a
Raul requested review from julianalcala 2025-12-12 10:51:56 +00:00
aitorarana added 1 commit 2025-12-12 12:42:17 +00:00
added size utils to recoverPassword 2e3681d36d
aitorarana added 1 commit 2025-12-17 11:39:57 +00:00
Added error message text to new password screen ce062ec6e8
julianalcala requested changes 2025-12-17 13:51:16 +00:00
modules/auth/lib/src/core/data/datasource/auth_remote_datasource.dart
@@ -3,2 +3,4 @@
Future<void> verifyPhoneCode({required String phone, required String code});
Future<String> requestPasswordReset({String? phone, String? email});
julianalcala commented 2025-12-17 13:42:35 +00:00
Member
Copy Link

Are these parameters none required?

Are these parameters none required?
aitorarana commented 2025-12-18 09:26:45 +00:00
Author
Member
Copy Link

The user can use either the phone number or the email, so none is strictly required.

The user can use either the phone number or the email, so none is strictly required.
julianalcala marked this conversation as resolved
modules/auth/lib/src/core/data/datasource/auth_remote_datasource_impl.dart
@@ -56,0 +70,4 @@
return 'ec14b7e7-58dd-4a59-9f41-0da86eaabf14';
} else {
body = {'phone': phone!};
return 'ec14b7e7-58dd-4a59-9f41-0da86eaabf14';
julianalcala commented 2025-12-17 13:44:23 +00:00
Member
Copy Link

why are you returning a static token?

why are you returning a static token?
aitorarana commented 2025-12-18 09:44:26 +00:00
Author
Member
Copy Link

As a placeholder before having actual accounts for testing.

As a placeholder before having actual accounts for testing.
julianalcala marked this conversation as resolved
modules/auth/lib/src/core/data/repositories/auth_repository_impl.dart
@@ -17,1 +17,4 @@
}
@override
Future<String> requestPasswordReset({String? phone, String? email}) {
julianalcala commented 2025-12-17 13:45:46 +00:00
Member
Copy Link

Are these parameters none required?

Are these parameters none required?
julianalcala marked this conversation as resolved
modules/auth/lib/src/core/domain/repositories/auth_repository.dart
@@ -3,2 +3,4 @@
Future<void> verifyPhoneCode({required String phone, required String code});
Future<String> requestPasswordReset({String phone, String email});
julianalcala commented 2025-12-17 13:46:00 +00:00
Member
Copy Link

Are these parameters none required?

Are these parameters none required?
julianalcala marked this conversation as resolved
modules/auth/lib/src/features/recover_password/presentation/new_password/new_password_screen.dart Outdated
@@ -0,0 +7,4 @@
import 'package:sf_localizations/sf_localizations.dart';
import 'package:utils/utils.dart';
class NewPasswordScreen extends ConsumerStatefulWidget {
julianalcala commented 2025-12-17 13:49:57 +00:00
Member
Copy Link

use ConsumerWidget not ConsumerStatefulWidget

use ConsumerWidget not ConsumerStatefulWidget
aitorarana marked this conversation as resolved
modules/auth/lib/src/features/recover_password/presentation/new_password/new_password_screen.dart Outdated
@@ -0,0 +40,4 @@
mainAxisAlignment: MainAxisAlignment.center,
children: [
Text(
'Recuperar contraseña',
julianalcala commented 2025-12-17 13:49:12 +00:00
Member
Copy Link

add easy_localization in dev_dependencies section of pubspec.yaml inside sf_localization in src level run:

dart pub get
dart run easy_localization:generate
-S assets/l10n
-O lib/generated
-o i18n.g.dart
-f keys

use translates like context.translate(I18n.example)

add easy_localization in dev_dependencies section of pubspec.yaml inside sf_localization in src level run: dart pub get dart run easy_localization:generate \ -S assets/l10n \ -O lib/generated \ -o i18n.g.dart \ -f keys use translates like context.translate(I18n.example)
aitorarana marked this conversation as resolved
aitorarana added 2 commits 2025-12-18 10:14:05 +00:00
Added recover password texts to i18n f6e07192dc
Updated locale keys ea6a9741ee
aitorarana added 1 commit 2025-12-18 15:46:56 +00:00
clean warnings 6a9d68e945
julianalcala merged commit 0d8cb4bd37 into develop 2025-12-19 08:46:56 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
julianalcala
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
aitorarana aitzibersolis alvarsanmartin bea10labs hector ivanmartinez jenkins jonanderelvira jorgebareas julianalcala nataliaexposito palomaibanez Raul unaimartinez
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SaveFamily/sf-app-platform#10
Delete Branch "auth-recover-password"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?