Email Confirmation and refactor for snackbar #332

Merged
tech merged 2 commits from SEND036 into main 2026-01-27 16:11:59 +00:00
Collaborator
No description provided.
protuberanets added 1 commit 2026-01-27 11:43:58 +00:00
protuberanets requested review from tech 2026-01-27 12:07:15 +00:00
protuberanets self-assigned this 2026-01-27 12:07:22 +00:00
tech requested changes 2026-01-27 15:18:54 +00:00
Dismissed
@@ -92,3 +92,3 @@
} catch (e) {
notifyUserOfErrorX(
scaffoldMessenger: sms,
context: context,
Owner

после await контекст лучше не использовать

после await контекст лучше не использовать
@@ -60,3 +59,1 @@
scaffoldMessenger: sm,
errorSituation: imageUpdateError ?? locs.settingsImageUpdateError,
exception: e,
context: context,
Owner

после await контекст лучше не использовать

после await контекст лучше не использовать
@@ -0,0 +1,122 @@
part of 'card.dart';
class _SignupConfirmationCardState extends State<SignupConfirmationCard> {
Owner

можно и так, но для простоты можно выносить отдельный класс - контроллер (аналог провайдера, который занимается данными UI). Тут можно не править ничего.

можно и так, но для простоты можно выносить отдельный класс - контроллер (аналог провайдера, который занимается данными UI). Тут можно не править ничего.
@@ -81,2 +85,2 @@
Future<ScaffoldFeatureController<SnackBar, SnackBarClosedReason>> postNotifyUserOfError({
required ScaffoldMessengerState scaffoldMessenger,
Future<void> postNotifyUserOfError({
required BuildContext context,
Owner

так лучше не делать: маскируешь проблему истечения контекста во время await

так лучше не делать: маскируешь проблему истечения контекста во время await
@@ -17,0 +10,4 @@
String message, {
int delaySeconds = 3,
}) async {
if (!context.mounted) return;
Owner

допустимо, конечно, но у тебя часть сообщений тогда пропадать будет (из-за истечения контекста)

допустимо, конечно, но у тебя часть сообщений тогда пропадать будет (из-за истечения контекста)
protuberanets added 1 commit 2026-01-27 15:41:22 +00:00
tech approved these changes 2026-01-27 16:11:22 +00:00
tech merged commit d78fe5daa0 into main 2026-01-27 16:11:59 +00:00
tech deleted branch SEND036 2026-01-27 16:11:59 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: tech/sendico#332