fix for resend, cooldown and a few small fixes #491

Merged
tech merged 1 commits from SEND052 into main 2026-02-12 23:13:02 +00:00
24 changed files with 550 additions and 227 deletions

View File

@@ -0,0 +1,5 @@
enum AuthProbeResult {
authorized,
notVerified,
error,
}

View File

@@ -7,10 +7,12 @@ import 'package:share_plus/share_plus.dart';
import 'package:pshared/api/errors/unauthorized.dart'; import 'package:pshared/api/errors/unauthorized.dart';
import 'package:pshared/api/requests/signup.dart'; import 'package:pshared/api/requests/signup.dart';
import 'package:pshared/api/requests/login_data.dart'; import 'package:pshared/api/requests/login_data.dart';
import 'package:pshared/api/responses/error/server.dart';
import 'package:pshared/api/responses/verification/response.dart'; import 'package:pshared/api/responses/verification/response.dart';
import 'package:pshared/config/constants.dart'; import 'package:pshared/config/constants.dart';
import 'package:pshared/models/account/account.dart'; import 'package:pshared/models/account/account.dart';
import 'package:pshared/models/auth/login_outcome.dart'; import 'package:pshared/models/auth/login_outcome.dart';
import 'package:pshared/models/auth/probe_result.dart';
import 'package:pshared/models/auth/pending_login.dart'; import 'package:pshared/models/auth/pending_login.dart';
import 'package:pshared/models/auth/state.dart'; import 'package:pshared/models/auth/state.dart';
import 'package:pshared/models/describable.dart'; import 'package:pshared/models/describable.dart';
@@ -309,4 +311,31 @@ class AccountProvider extends ChangeNotifier {
} }
await restore(); await restore();
} }
Future<AuthProbeResult> probeAuthorization({
required String email,
required String password,
required String locale,
}) async {
try {
final outcome = await AccountService.login(LoginData.build(
login: email,
password: password,
locale: locale,
));
if (outcome.isCompleted) {
await AuthorizationService.logout();
return AuthProbeResult.authorized;
}
if (outcome.isPending) {
return AuthProbeResult.authorized;
}
return AuthProbeResult.error;
} catch (e) {
if (e is ErrorResponse && e.error == 'account_not_verified') {
return AuthProbeResult.notVerified;
}
return AuthProbeResult.error;
}
}
} }

View File

@@ -5,6 +5,7 @@ import 'package:pshared/provider/resource.dart';
import 'package:pshared/service/account.dart'; import 'package:pshared/service/account.dart';
import 'package:pshared/utils/exception.dart'; import 'package:pshared/utils/exception.dart';
class EmailVerificationProvider extends ChangeNotifier { class EmailVerificationProvider extends ChangeNotifier {
Resource<bool> _resource = Resource(data: null, isLoading: false); Resource<bool> _resource = Resource(data: null, isLoading: false);
String? _token; String? _token;
@@ -16,6 +17,14 @@ class EmailVerificationProvider extends ChangeNotifier {
ErrorResponse? get errorResponse => ErrorResponse? get errorResponse =>
_resource.error is ErrorResponse ? _resource.error as ErrorResponse : null; _resource.error is ErrorResponse ? _resource.error as ErrorResponse : null;
int? get errorCode => errorResponse?.code; int? get errorCode => errorResponse?.code;
bool get isTokenAlreadyUsed {
final response = errorResponse;
if (response == null) return false;
if (response.code != 409 || response.error != 'data_conflict') {
return false;
}
return response.details.contains('verification token has already been used');
}
bool get canResendVerification => bool get canResendVerification =>
errorCode == 400 || errorCode == 410 || errorCode == 500; errorCode == 400 || errorCode == 410 || errorCode == 500;

View File

@@ -45,6 +45,9 @@ GoRouter createRouter() => GoRouter(
email: state.extra is SignupConfirmationArgs email: state.extra is SignupConfirmationArgs
? (state.extra as SignupConfirmationArgs).email ? (state.extra as SignupConfirmationArgs).email
: null, : null,
password: state.extra is SignupConfirmationArgs
? (state.extra as SignupConfirmationArgs).password
: null,
), ),
), ),
GoRoute( GoRoute(

View File

@@ -0,0 +1,91 @@
import 'dart:async';
import 'package:flutter/foundation.dart';
import 'package:pshared/models/auth/probe_result.dart';
import 'package:pshared/provider/account.dart';
class SignupConfirmationController extends ChangeNotifier {
SignupConfirmationController({
required AccountProvider accountProvider,
Duration pollInterval = const Duration(seconds: 10),
}) : _accountProvider = accountProvider,
_pollInterval = pollInterval;
final AccountProvider _accountProvider;
final Duration _pollInterval;
Timer? _pollTimer;
bool _isChecking = false;
bool _isAuthorized = false;
String? _email;
String? _password;
String? _locale;
bool get isAuthorized => _isAuthorized;
bool get isChecking => _isChecking;
void startPolling({
required String email,
required String password,
required String locale,
}) {
final trimmedEmail = email.trim();
final trimmedPassword = password.trim();
final trimmedLocale = locale.trim();
if (trimmedEmail.isEmpty || trimmedPassword.isEmpty || trimmedLocale.isEmpty) {
return;
}
_email = trimmedEmail;
_password = trimmedPassword;
_locale = trimmedLocale;
_pollTimer?.cancel();
_pollTimer = Timer.periodic(_pollInterval, (_) => _probeAuthorization());
_probeAuthorization();
}
void stopPolling() {
_pollTimer?.cancel();
_pollTimer = null;
}
@override
void dispose() {
_pollTimer?.cancel();
super.dispose();
}
Future<void> _probeAuthorization() async {
if (_isChecking || _isAuthorized) return;
final email = _email;
final password = _password;
final locale = _locale;
if (email == null || password == null || locale == null) return;
_setChecking(true);
try {
final result = await _accountProvider.probeAuthorization(
email: email,
password: password,
locale: locale,
);
if (result == AuthProbeResult.authorized) {
_isAuthorized = true;
stopPolling();
notifyListeners();
}
} finally {
_setChecking(false);
}
}
void _setChecking(bool value) {
if (_isChecking == value) return;
_isChecking = value;
notifyListeners();
}
}

View File

@@ -0,0 +1,132 @@
import 'dart:async';
import 'package:flutter/foundation.dart';
import 'package:pshared/provider/account.dart';
import 'package:pweb/models/flow_status.dart';
import 'package:pweb/models/resend/action_result.dart';
import 'package:pweb/models/resend/avaliability.dart';
class SignupConfirmationCardController extends ChangeNotifier {
SignupConfirmationCardController({
required AccountProvider accountProvider,
Duration defaultCooldown = const Duration(seconds: 60),
}) : _accountProvider = accountProvider,
_defaultCooldown = defaultCooldown;
final AccountProvider _accountProvider;
final Duration _defaultCooldown;
Timer? _cooldownTimer;
DateTime? _cooldownUntil;
int _cooldownRemainingSeconds = 0;
FlowStatus _resendState = FlowStatus.idle;
String? _email;
int get cooldownRemainingSeconds => _cooldownRemainingSeconds;
ResendAvailability get resendAvailability {
final email = _email;
if (email == null || email.isEmpty) {
return ResendAvailability.missingEmail;
}
if (_resendState == FlowStatus.submitting) {
return ResendAvailability.resending;
}
if (_cooldownRemainingSeconds > 0) {
return ResendAvailability.cooldown;
}
return ResendAvailability.available;
}
void updateEmail(String? email) {
final trimmed = email?.trim();
if (_email == trimmed) return;
_email = trimmed;
notifyListeners();
}
void initialize({String? email}) {
updateEmail(email);
startDefaultCooldown();
}
void startDefaultCooldown() {
_startCooldown(_defaultCooldown);
}
Future<ResendActionResult> resendVerificationEmail() async {
switch (resendAvailability) {
case ResendAvailability.missingEmail:
return ResendActionResult.missingEmail;
case ResendAvailability.cooldown:
return ResendActionResult.cooldown;
case ResendAvailability.resending:
return ResendActionResult.inProgress;
case ResendAvailability.available:
break;
}
_setResendState(FlowStatus.submitting);
try {
final email = _email;
if (email == null || email.isEmpty) {
_setResendState(FlowStatus.idle);
return ResendActionResult.missingEmail;
}
await _accountProvider.resendVerificationEmail(email);
_startCooldown(_defaultCooldown);
return ResendActionResult.sent;
} finally {
_setResendState(FlowStatus.idle);
}
}
@override
void dispose() {
_cooldownTimer?.cancel();
super.dispose();
}
void _startCooldown(Duration duration) {
_cooldownTimer?.cancel();
_cooldownUntil = DateTime.now().add(duration);
_syncRemaining();
if (_cooldownRemainingSeconds <= 0) {
_cooldownUntil = null;
notifyListeners();
return;
}
_cooldownTimer = Timer.periodic(const Duration(seconds: 1), (timer) {
_syncRemaining();
if (_cooldownRemainingSeconds <= 0) {
timer.cancel();
_cooldownUntil = null;
notifyListeners();
}
});
}
void _syncRemaining() {
final remaining = _cooldownRemaining();
if (remaining == _cooldownRemainingSeconds) return;
_cooldownRemainingSeconds = remaining;
notifyListeners();
}
int _cooldownRemaining() {
final until = _cooldownUntil;
if (until == null) return 0;
final remaining = until.difference(DateTime.now()).inSeconds;
return remaining < 0 ? 0 : remaining;
}
void _setResendState(FlowStatus state) {
if (_resendState == state) return;
_resendState = state;
notifyListeners();
}
}

View File

@@ -660,6 +660,8 @@
"verificationFailed": "Verification Failed", "verificationFailed": "Verification Failed",
"verificationStatusUnknown": "We couldn't determine the status of your verification. Please try again later.", "verificationStatusUnknown": "We couldn't determine the status of your verification. Please try again later.",
"verificationStatusErrorUnknown": "Unexpected error occurred while verification. Try once again or contact support", "verificationStatusErrorUnknown": "Unexpected error occurred while verification. Try once again or contact support",
"accountAlreadyVerified": "Your account has already been verified",
"accountAlreadyVerifiedDescription": "You can now log in to access your account.",
"accountVerified": "Account Verified!", "accountVerified": "Account Verified!",
"accountVerifiedDescription": "Your account has been successfully verified. You can now log in to access your account.", "accountVerifiedDescription": "Your account has been successfully verified. You can now log in to access your account.",
"retryVerification": "Retry Verification", "retryVerification": "Retry Verification",

View File

@@ -662,6 +662,8 @@
"verificationFailed": "Ошибка подтверждения", "verificationFailed": "Ошибка подтверждения",
"verificationStatusUnknown": "Не удалось определить статус подтверждения. Попробуйте позже", "verificationStatusUnknown": "Не удалось определить статус подтверждения. Попробуйте позже",
"verificationStatusErrorUnknown": "Произошла непредвиденная ошибка при подтверждении. Попробуйте еще раз или обратитесь в службу поддержки", "verificationStatusErrorUnknown": "Произошла непредвиденная ошибка при подтверждении. Попробуйте еще раз или обратитесь в службу поддержки",
"accountAlreadyVerified": "Ваш аккаунт уже подтвержден",
"accountAlreadyVerifiedDescription": "Теперь вы можете войти, чтобы получить доступ к аккаунту.",
"accountVerified": "Аккаунт подтвержден!", "accountVerified": "Аккаунт подтвержден!",
"accountVerifiedDescription": "Ваш аккаунт успешно подтвержден. Теперь вы можете войти, чтобы получить доступ к своему аккаунту", "accountVerifiedDescription": "Ваш аккаунт успешно подтвержден. Теперь вы можете войти, чтобы получить доступ к своему аккаунту",
"retryVerification": "Повторить подтверждение", "retryVerification": "Повторить подтверждение",

View File

@@ -0,0 +1,6 @@
enum ResendActionResult {
sent,
missingEmail,
cooldown,
inProgress,
}

View File

@@ -0,0 +1,6 @@
enum ResendAvailability {
available,
cooldown,
resending,
missingEmail,
}

View File

@@ -3,6 +3,8 @@ import 'package:flutter/material.dart';
import 'package:provider/provider.dart'; import 'package:provider/provider.dart';
import 'package:pweb/providers/two_factor.dart'; import 'package:pweb/providers/two_factor.dart';
import 'package:pweb/utils/cooldown_format.dart';
import 'package:pweb/widgets/resend_link.dart';
import 'package:pweb/generated/i18n/app_localizations.dart'; import 'package:pweb/generated/i18n/app_localizations.dart';
@@ -12,46 +14,20 @@ class ResendCodeButton extends StatelessWidget {
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
final theme = Theme.of(context);
final localizations = AppLocalizations.of(context)!; final localizations = AppLocalizations.of(context)!;
final provider = context.watch<TwoFactorProvider>(); final provider = context.watch<TwoFactorProvider>();
final isDisabled = provider.isCooldownActive || provider.isResending; final isDisabled = provider.isCooldownActive || provider.isResending;
final label = provider.isCooldownActive final label = provider.isCooldownActive
? '${localizations.twoFactorResend} (${_formatCooldown(provider.cooldownRemainingSeconds)})' ? '${localizations.twoFactorResend} (${formatCooldownSeconds(provider.cooldownRemainingSeconds)})'
: localizations.twoFactorResend; : localizations.twoFactorResend;
return TextButton( return ResendLink(
onPressed: isDisabled ? null : () => provider.resendCode(), label: label,
style: TextButton.styleFrom( onPressed: provider.resendCode,
padding: EdgeInsets.zero, isDisabled: isDisabled,
minimumSize: const Size(0, 0), isLoading: provider.isResending,
tapTargetSize: MaterialTapTargetSize.shrinkWrap,
alignment: Alignment.centerLeft,
foregroundColor: theme.colorScheme.primary,
textStyle: theme.textTheme.bodyMedium?.copyWith(
decoration: TextDecoration.underline,
),
),
child: provider.isResending
? SizedBox(
width: 16,
height: 16,
child: CircularProgressIndicator(
strokeWidth: 2,
color: theme.colorScheme.primary,
),
)
: Text(label),
); );
} }
String _formatCooldown(int seconds) {
final minutes = seconds ~/ 60;
final remainingSeconds = seconds % 60;
if (minutes > 0) {
return '$minutes:${remainingSeconds.toString().padLeft(2, '0')}';
}
return remainingSeconds.toString();
}
} }

View File

@@ -1,5 +1,6 @@
class SignupConfirmationArgs { class SignupConfirmationArgs {
final String? email; final String? email;
final String? password;
const SignupConfirmationArgs({this.email}); const SignupConfirmationArgs({this.email, this.password});
} }

View File

@@ -1,13 +1,16 @@
import 'dart:async';
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:provider/provider.dart'; import 'package:provider/provider.dart';
import 'package:pshared/provider/account.dart'; import 'package:pshared/provider/account.dart';
import 'package:pweb/models/resend/action_result.dart';
import 'package:pweb/models/resend/avaliability.dart';
import 'package:pweb/utils/snackbar.dart'; import 'package:pweb/utils/snackbar.dart';
import 'package:pweb/utils/error/snackbar.dart'; import 'package:pweb/utils/error/snackbar.dart';
import 'package:pweb/utils/cooldown_format.dart';
import 'package:pweb/controllers/signup/confirmation_card.dart';
import 'package:pweb/widgets/resend_link.dart';
import 'package:pweb/widgets/vspacer.dart'; import 'package:pweb/widgets/vspacer.dart';
import 'package:pweb/generated/i18n/app_localizations.dart'; import 'package:pweb/generated/i18n/app_localizations.dart';
@@ -15,7 +18,6 @@ import 'package:pweb/generated/i18n/app_localizations.dart';
part 'state.dart'; part 'state.dart';
part 'content.dart'; part 'content.dart';
part 'badge.dart'; part 'badge.dart';
part 'resend_button.dart';
class SignupConfirmationCard extends StatefulWidget { class SignupConfirmationCard extends StatefulWidget {

View File

@@ -63,11 +63,11 @@ class _SignupConfirmationContent extends StatelessWidget {
runSpacing: 12, runSpacing: 12,
alignment: WrapAlignment.center, alignment: WrapAlignment.center,
children: [ children: [
_SignupConfirmationResendButton( ResendLink(
canResend: canResend, label: resendLabel,
isResending: isResending, onPressed: onResend,
resendLabel: resendLabel, isDisabled: !canResend,
onResend: onResend, isLoading: isResending,
), ),
], ],
), ),

View File

@@ -1,36 +0,0 @@
part of 'card.dart';
class _SignupConfirmationResendButton extends StatelessWidget {
final bool canResend;
final bool isResending;
final String resendLabel;
final VoidCallback onResend;
const _SignupConfirmationResendButton({
required this.canResend,
required this.isResending,
required this.resendLabel,
required this.onResend,
});
@override
Widget build(BuildContext context) {
final theme = Theme.of(context);
return ElevatedButton.icon(
onPressed: canResend ? onResend : null,
icon: isResending
? SizedBox(
width: 16,
height: 16,
child: CircularProgressIndicator(
strokeWidth: 2,
color: theme.colorScheme.onPrimary,
),
)
: const Icon(Icons.mark_email_read_outlined),
label: Text(resendLabel),
);
}
}

View File

@@ -3,61 +3,45 @@ part of 'card.dart';
class _SignupConfirmationCardState extends State<SignupConfirmationCard> { class _SignupConfirmationCardState extends State<SignupConfirmationCard> {
static const int _defaultCooldownSeconds = 60; static const int _defaultCooldownSeconds = 60;
late final SignupConfirmationCardController _controller;
Timer? _cooldownTimer;
int _cooldownRemainingSeconds = 0;
bool _isResending = false;
@override @override
void initState() { void initState() {
super.initState(); super.initState();
_startCooldown(_defaultCooldownSeconds); _controller = SignupConfirmationCardController(
accountProvider: context.read<AccountProvider>(),
defaultCooldown: const Duration(seconds: _defaultCooldownSeconds),
);
_controller.initialize(email: widget.email);
} }
@override @override
void dispose() { void dispose() {
_cooldownTimer?.cancel(); _controller.dispose();
super.dispose(); super.dispose();
} }
bool get _isCooldownActive => _cooldownRemainingSeconds > 0; @override
void didUpdateWidget(covariant SignupConfirmationCard oldWidget) {
void _startCooldown(int seconds) { super.didUpdateWidget(oldWidget);
_cooldownTimer?.cancel(); if (oldWidget.email != widget.email) {
if (seconds <= 0) { _controller.updateEmail(widget.email);
setState(() => _cooldownRemainingSeconds = 0);
return;
} }
setState(() => _cooldownRemainingSeconds = seconds);
_cooldownTimer = Timer.periodic(const Duration(seconds: 1), (timer) {
if (!mounted) {
timer.cancel();
return;
}
if (_cooldownRemainingSeconds <= 1) {
timer.cancel();
setState(() => _cooldownRemainingSeconds = 0);
return;
}
setState(() => _cooldownRemainingSeconds -= 1);
});
} }
Future<void> _resendVerificationEmail() async { Future<void> _resendVerificationEmail() async {
final email = widget.email?.trim();
final locs = AppLocalizations.of(context)!; final locs = AppLocalizations.of(context)!;
if (email == null || email.isEmpty) { try {
final result = await _controller.resendVerificationEmail();
if (!mounted) return;
if (result == ResendActionResult.missingEmail) {
notifyUser(context, locs.errorEmailMissing); notifyUser(context, locs.errorEmailMissing);
return; return;
} }
if (_isResending || _isCooldownActive) return; if (result == ResendActionResult.sent) {
final email = widget.email?.trim() ?? '';
setState(() => _isResending = true);
try {
await context.read<AccountProvider>().resendVerificationEmail(email);
if (!mounted) return;
notifyUser(context, locs.signupConfirmationResent(email)); notifyUser(context, locs.signupConfirmationResent(email));
_startCooldown(_defaultCooldownSeconds); }
} catch (e) { } catch (e) {
if (!mounted) return; if (!mounted) return;
postNotifyUserOfErrorX( postNotifyUserOfErrorX(
@@ -65,21 +49,8 @@ class _SignupConfirmationCardState extends State<SignupConfirmationCard> {
errorSituation: locs.signupConfirmationResendError, errorSituation: locs.signupConfirmationResendError,
exception: e, exception: e,
); );
} finally {
if (mounted) {
setState(() => _isResending = false);
} }
} }
}
String _formatCooldown(int seconds) {
final minutes = seconds ~/ 60;
final remainingSeconds = seconds % 60;
if (minutes > 0) {
return '$minutes:${remainingSeconds.toString().padLeft(2, '0')}';
}
return remainingSeconds.toString();
}
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
@@ -89,9 +60,17 @@ class _SignupConfirmationCardState extends State<SignupConfirmationCard> {
final description = (email != null && email.isNotEmpty) final description = (email != null && email.isNotEmpty)
? locs.signupConfirmationDescription(email) ? locs.signupConfirmationDescription(email)
: locs.signupConfirmationDescriptionNoEmail; : locs.signupConfirmationDescriptionNoEmail;
final canResend = !_isResending && !_isCooldownActive && email != null && email.isNotEmpty;
final resendLabel = _isCooldownActive return AnimatedBuilder(
? locs.signupConfirmationResendCooldown(_formatCooldown(_cooldownRemainingSeconds)) animation: _controller,
builder: (context, _) {
final availability = _controller.resendAvailability;
final canResend = availability == ResendAvailability.available;
final isResending = availability == ResendAvailability.resending;
final resendLabel = availability == ResendAvailability.cooldown
? locs.signupConfirmationResendCooldown(
formatCooldownSeconds(_controller.cooldownRemainingSeconds),
)
: locs.signupConfirmationResend; : locs.signupConfirmationResend;
final content = _SignupConfirmationContent( final content = _SignupConfirmationContent(
@@ -99,7 +78,7 @@ class _SignupConfirmationCardState extends State<SignupConfirmationCard> {
description: description, description: description,
canResend: canResend, canResend: canResend,
resendLabel: resendLabel, resendLabel: resendLabel,
isResending: _isResending, isResending: isResending,
onResend: _resendVerificationEmail, onResend: _resendVerificationEmail,
); );
@@ -118,5 +97,7 @@ class _SignupConfirmationCardState extends State<SignupConfirmationCard> {
child: content, child: content,
), ),
); );
},
);
} }
} }

View File

@@ -1,39 +1,83 @@
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:provider/provider.dart';
import 'package:pshared/provider/account.dart';
import 'package:pweb/app/router/pages.dart';
import 'package:pweb/pages/login/app_bar.dart'; import 'package:pweb/pages/login/app_bar.dart';
import 'package:pweb/pages/signup/confirmation/card/card.dart'; import 'package:pweb/pages/signup/confirmation/card/card.dart';
import 'package:pweb/pages/signup/confirmation/login_prompt.dart'; import 'package:pweb/controllers/signup/confirmation.dart';
import 'package:pweb/pages/with_footer.dart'; import 'package:pweb/pages/with_footer.dart';
import 'package:pweb/widgets/vspacer.dart';
class SignUpConfirmationPage extends StatefulWidget { class SignUpConfirmationPage extends StatefulWidget {
final String? email; final String? email;
final String? password;
const SignUpConfirmationPage({super.key, this.email}); const SignUpConfirmationPage({
super.key,
this.email,
this.password,
});
@override @override
State<SignUpConfirmationPage> createState() => _SignUpConfirmationPageState(); State<SignUpConfirmationPage> createState() => _SignUpConfirmationPageState();
} }
class _SignUpConfirmationPageState extends State<SignUpConfirmationPage> { class _SignUpConfirmationPageState extends State<SignUpConfirmationPage> {
late final SignupConfirmationController _controller;
@override
void initState() {
super.initState();
_controller = SignupConfirmationController(
accountProvider: context.read<AccountProvider>(),
)..addListener(_handleAuthorizationStatus);
WidgetsBinding.instance.addPostFrameCallback((_) => _startPolling());
}
@override
void dispose() {
_controller.removeListener(_handleAuthorizationStatus);
_controller.dispose();
super.dispose();
}
void _startPolling() {
if (!mounted) return;
final email = widget.email?.trim();
final password = widget.password;
if (email == null || email.isEmpty || password == null || password.isEmpty) {
return;
}
_controller.startPolling(
email: email,
password: password,
locale: Localizations.localeOf(context).toLanguageTag(),
);
}
void _handleAuthorizationStatus() {
if (!_controller.isAuthorized || !mounted) return;
navigateAndReplace(context, Pages.login);
}
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
final email = widget.email?.trim(); final email = widget.email?.trim();
final width = MediaQuery.of(context).size.width;
final isWide = width >= 980;
return PageWithFooter( return PageWithFooter(
appBar: const LoginAppBar(), appBar: const LoginAppBar(),
child: LayoutBuilder( child: Padding(
builder: (context, constraints) {
final isWide = constraints.maxWidth >= 980;
return ListView(
padding: const EdgeInsets.symmetric(horizontal: 20, vertical: 32), padding: const EdgeInsets.symmetric(horizontal: 20, vertical: 32),
children: [ child: Center(
Center(
child: ConstrainedBox( child: ConstrainedBox(
constraints: BoxConstraints(maxWidth: isWide ? 980 : 720), constraints: BoxConstraints(maxWidth: isWide ? 980 : 720),
child: Column( child: Column(
mainAxisSize: MainAxisSize.min,
crossAxisAlignment: CrossAxisAlignment.center, crossAxisAlignment: CrossAxisAlignment.center,
children: [ children: [
Card( Card(
@@ -46,31 +90,17 @@ class _SignUpConfirmationPageState extends State<SignUpConfirmationPage> {
), ),
child: Padding( child: Padding(
padding: const EdgeInsets.all(28), padding: const EdgeInsets.all(28),
child: Column( child:
crossAxisAlignment: CrossAxisAlignment.center,
children: [
SignupConfirmationCard( SignupConfirmationCard(
email: email, email: email,
isEmbedded: true, isEmbedded: true,
), ),
const VSpacer(multiplier: 2),
Divider(
height: 1,
color: Theme.of(context).dividerColor.withValues(alpha: 0.6),
),
const VSpacer(multiplier: 2),
const SignupConfirmationLoginPrompt(isEmbedded: true),
],
),
), ),
), ),
], ],
), ),
), ),
), ),
],
);
},
), ),
); );
} }

View File

@@ -95,6 +95,7 @@ class SignUpFormState extends State<SignUpForm> {
Pages.signupConfirm.name, Pages.signupConfirm.name,
extra: SignupConfirmationArgs( extra: SignupConfirmationArgs(
email: controllers.email.text.trim(), email: controllers.email.text.trim(),
password: controllers.password.text,
), ),
); );
}, },

View File

@@ -12,6 +12,7 @@ import 'package:pweb/pages/verification/controller.dart';
import 'package:pweb/pages/verification/resend_dialog.dart'; import 'package:pweb/pages/verification/resend_dialog.dart';
import 'package:pweb/utils/snackbar.dart'; import 'package:pweb/utils/snackbar.dart';
import 'package:pweb/utils/error/snackbar.dart'; import 'package:pweb/utils/error/snackbar.dart';
import 'package:pweb/widgets/resend_link.dart';
import 'package:pweb/generated/i18n/app_localizations.dart'; import 'package:pweb/generated/i18n/app_localizations.dart';
@@ -64,6 +65,12 @@ class AccountVerificationContentState
Widget content; Widget content;
if (controller.isLoading) { if (controller.isLoading) {
content = const Center(child: CircularProgressIndicator()); content = const Center(child: CircularProgressIndicator());
} else if (controller.isAlreadyVerified) {
content = StatusPageSuccess(
successMessage: locs.accountAlreadyVerified,
successDescription: locs.accountAlreadyVerifiedDescription,
action: action,
);
} else if (controller.isSuccess) { } else if (controller.isSuccess) {
content = StatusPageSuccess( content = StatusPageSuccess(
successMessage: locs.accountVerified, successMessage: locs.accountVerified,
@@ -78,18 +85,11 @@ class AccountVerificationContentState
exception: exception:
controller.error ?? Exception(locs.accountVerificationFailed), controller.error ?? Exception(locs.accountVerificationFailed),
action: controller.canResend action: controller.canResend
? OutlinedButton.icon( ? ResendLink(
onPressed: controller.isResending label: locs.signupConfirmationResend,
? null onPressed: _resendVerificationEmail,
: _resendVerificationEmail, isDisabled: !controller.canResend || controller.isResending,
icon: controller.isResending isLoading: controller.isResending,
? const SizedBox(
width: 18,
height: 18,
child: CircularProgressIndicator(strokeWidth: 2),
)
: const Icon(Icons.mark_email_unread_outlined),
label: Text(locs.signupConfirmationResend),
) )
: null, : null,
); );

View File

@@ -26,6 +26,7 @@ class AccountVerificationController extends ChangeNotifier {
Exception? get error => _verificationProvider.error; Exception? get error => _verificationProvider.error;
bool get canResend => _verificationProvider.canResendVerification; bool get canResend => _verificationProvider.canResendVerification;
bool get isResending => _resendStatus == FlowStatus.resending; bool get isResending => _resendStatus == FlowStatus.resending;
bool get isAlreadyVerified => _verificationProvider.isTokenAlreadyUsed;
void startVerification(String token) { void startVerification(String token) {
final trimmed = token.trim(); final trimmed = token.trim();

View File

@@ -22,6 +22,7 @@ class TwoFactorProvider extends ChangeNotifier {
String? _currentPendingToken; String? _currentPendingToken;
Timer? _cooldownTimer; Timer? _cooldownTimer;
int _cooldownRemainingSeconds = 0; int _cooldownRemainingSeconds = 0;
DateTime? _cooldownUntil;
FlowStatus get status => _status; FlowStatus get status => _status;
bool get isSubmitting => _status == FlowStatus.submitting; bool get isSubmitting => _status == FlowStatus.submitting;
@@ -108,48 +109,69 @@ class TwoFactorProvider extends ChangeNotifier {
return; return;
} }
final remaining = pending.cooldownRemainingSeconds; final until = pending.cooldownUntil;
if (remaining <= 0) { if (until == null) {
_stopCooldown(notify: _cooldownRemainingSeconds != 0); _stopCooldown(notify: _cooldownRemainingSeconds != 0);
return; return;
} }
if (_cooldownRemainingSeconds != remaining) { if (!_isCooldownActive(until) && _cooldownRemainingSeconds != 0) {
_startCooldown(remaining); _stopCooldown(notify: true);
return;
}
if (_cooldownUntil == null || _cooldownUntil != until) {
_startCooldownUntil(until);
} }
} }
void _startCooldown(int seconds) { void _startCooldown(int seconds) {
final until = DateTime.now().add(Duration(seconds: seconds));
_startCooldownUntil(until);
}
void _startCooldownUntil(DateTime until) {
_cooldownTimer?.cancel(); _cooldownTimer?.cancel();
_cooldownRemainingSeconds = seconds; _cooldownUntil = until;
_cooldownRemainingSeconds = _cooldownRemaining();
if (_cooldownRemainingSeconds <= 0) { if (_cooldownRemainingSeconds <= 0) {
_cooldownTimer = null; _cooldownTimer = null;
_cooldownUntil = null;
notifyListeners(); notifyListeners();
return; return;
} }
_cooldownTimer = Timer.periodic(const Duration(seconds: 1), (timer) { _cooldownTimer = Timer.periodic(const Duration(seconds: 1), (timer) {
if (_cooldownRemainingSeconds <= 1) { final remaining = _cooldownRemaining();
_cooldownRemainingSeconds = 0; if (remaining <= 0) {
_cooldownTimer?.cancel(); _stopCooldown(notify: true);
_cooldownTimer = null;
notifyListeners();
return; return;
} }
if (remaining != _cooldownRemainingSeconds) {
_cooldownRemainingSeconds -= 1; _cooldownRemainingSeconds = remaining;
notifyListeners(); notifyListeners();
}
}); });
notifyListeners(); notifyListeners();
} }
bool _isCooldownActive(DateTime until) => until.isAfter(DateTime.now());
int _cooldownRemaining() {
final until = _cooldownUntil;
if (until == null) return 0;
final remaining = until.difference(DateTime.now()).inSeconds;
return remaining < 0 ? 0 : remaining;
}
void _stopCooldown({bool notify = false}) { void _stopCooldown({bool notify = false}) {
_cooldownTimer?.cancel(); _cooldownTimer?.cancel();
_cooldownTimer = null; _cooldownTimer = null;
final hadCooldown = _cooldownRemainingSeconds != 0; final hadCooldown = _cooldownRemainingSeconds != 0;
_cooldownRemainingSeconds = 0; _cooldownRemainingSeconds = 0;
_cooldownUntil = null;
if (notify && hadCooldown) { if (notify && hadCooldown) {
notifyListeners(); notifyListeners();

View File

@@ -0,0 +1,8 @@
String formatCooldownSeconds(int seconds) {
final minutes = seconds ~/ 60;
final remainingSeconds = seconds % 60;
if (minutes > 0) {
return '$minutes:${remainingSeconds.toString().padLeft(2, '0')}';
}
return remainingSeconds.toString();
}

View File

@@ -15,6 +15,7 @@ class ErrorHandler {
'account_not_verified': locs.errorAccountNotVerified, 'account_not_verified': locs.errorAccountNotVerified,
'unauthorized': locs.errorLoginUnauthorized, 'unauthorized': locs.errorLoginUnauthorized,
'verification_token_not_found': locs.errorVerificationTokenNotFound, 'verification_token_not_found': locs.errorVerificationTokenNotFound,
'user_already_registered': locs.errorDuplicateEmail,
'internal_error': locs.errorInternalError, 'internal_error': locs.errorInternalError,
'invalid_target': locs.errorInvalidTarget, 'invalid_target': locs.errorInvalidTarget,
'pending_token_required': locs.errorPendingTokenRequired, 'pending_token_required': locs.errorPendingTokenRequired,
@@ -62,6 +63,9 @@ class ErrorHandler {
} }
static String _handleErrorResponseLocs(AppLocalizations locs, ErrorResponse e) { static String _handleErrorResponseLocs(AppLocalizations locs, ErrorResponse e) {
if (e.source == 'user_already_registered') {
return locs.errorDuplicateEmail;
}
final errorMessages = getErrorMessagesLocs(locs); final errorMessages = getErrorMessagesLocs(locs);
// Return the localized message if we recognize the error key, else use the raw details // Return the localized message if we recognize the error key, else use the raw details
return errorMessages[e.error] ?? e.details; return errorMessages[e.error] ?? e.details;

View File

@@ -0,0 +1,48 @@
import 'package:flutter/material.dart';
class ResendLink extends StatelessWidget {
final String label;
final VoidCallback onPressed;
final bool isDisabled;
final bool isLoading;
const ResendLink({
super.key,
required this.label,
required this.onPressed,
this.isDisabled = false,
this.isLoading = false,
});
@override
Widget build(BuildContext context) {
final theme = Theme.of(context);
final color = theme.colorScheme.primary;
final isButtonDisabled = isDisabled || isLoading;
return TextButton(
onPressed: isButtonDisabled ? null : onPressed,
style: TextButton.styleFrom(
padding: EdgeInsets.zero,
minimumSize: const Size(0, 0),
tapTargetSize: MaterialTapTargetSize.shrinkWrap,
alignment: Alignment.centerLeft,
foregroundColor: color,
textStyle: theme.textTheme.bodyMedium?.copyWith(
decoration: TextDecoration.underline,
),
),
child: isLoading
? SizedBox(
width: 16,
height: 16,
child: CircularProgressIndicator(
strokeWidth: 2,
color: color,
),
)
: Text(label),
);
}
}