Fixes for PostHog #72
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: tech/sendico#72
Reference in New Issue
Block a user
Delete Branch "SEND005"
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?
@@ -69,1 +71,4 @@bool get hasUnhandledError => error != null && !_errorHandled;void markErrorHandled() {Арсений, ты снова выставил внутреннее содержание класса наружу. А зачем? Почему состояние обработки ошибки (которое к провайдеру точно не относится) оказалось внутри провайдера? Перенеси это состояние тому, кто им пользуется. Если пользователей несколько, то это повод создать отдельный провайдер-надстройку.
@@ -61,0 +59,4 @@create: (_) => AccountProvider(onAccountChanged: (account) {if (account == null) return Future<void>.value();return PosthogService.identify(account);лучше все-таки идентификацию положить там, где делается установка внутреннего ресурса провайдера. Отправка identify должна быть прямо в том самом месте, где ставится ресурс. НЕ снаружи. Иначе у тебя identify будет вызываться по поводу и без.
@@ -30,0 +27,4 @@final provider = Provider.of<AccountProvider>(context, listen: false);if (provider.account == null) {provider.restoreIfPossible().catchError((error, stack) {Logger('Account restore failed: $error');именной логгер вместо сообщения?
@@ -59,3 +59,3 @@if (locale != null) {localeProvider.setLocale(locale);// AmplitudeService.localeChanged(locale);PosthogService.localeChanged(locale);unawaited?
@@ -11,3 +12,4 @@void logoutUtil(BuildContext context) {context.read<AccountProvider>().logout();context.read<PermissionsProvider>().reset();PosthogService.reset();должно быть ровно в том месте, где сбрасывается ресурс AccountProvider'а.
@@ -50,3 +50,3 @@onTap: () {onSelected(item);// AmplitudeService.pageOpened(item, uiSource: 'sidebar');PosthogService.pageOpened(item, uiSource: 'sidebar');unawaited?
Pull request closed