Dat Vu
Dat Vu
> **Actionable comments posted: 0** > > 🧹 Nitpick comments (3) > > lib/features/thread_detail/domain/extensions/list_email_in_thread_detail_info_extension.dart (2)> `24-36`: **Efficient multi‑ID toggle; consider semantics of returning `this` for empty targets** > > >...
> * [x] remove `NotFoundAuthenticateAccountException` `AutoRedirec....` Done
> hi @dab246 , please try to config helm-chart for sentry configuration: https://ci.linagora.com/linagora/lrs/saas/tools/helm-charts/tmail-frontend/-/tree/main/tmail-frontend?ref_type=heads > I created a merge request for it: https://ci.linagora.com/linagora/lrs/saas/tools/helm-charts/tmail-frontend/-/merge_requests/12
> Also I’m a bit concerned about dio_sentry. Is there any overhead/slowdown for http requests performance? # Does `sentry_dio` Slow Down Your Requests? => No, the slowdown is negligible. `sentry_dio`...
> Dio: backend errors already are captured by the backend logging system on Loki. Is this useful? ## Why do we need `sentry_dio` if the backend already logs errors? Because...
This PR acts as a base from which related changes are split into smaller PRs for easier review and merging: - https://github.com/linagora/tmail-flutter/pull/4205
@coderabbitai please review
> * [x] mailbox view crashed > Fixed Web: https://github.com/user-attachments/assets/c0466171-4b61-429f-8138-2af9cc1235ef Mobile: [Screen_recording_20260102_132949.webm](https://github.com/user-attachments/assets/885e6968-9173-472f-a30a-ea3620fb7ba3)
> * [x] still have some place, we can remove logError > > * core/lib/utils/string_convert.dart: L249 > * lib/features/base/mixin/ai_scribe_mixin.dart: L30 > * lib/features/composer/presentation/extensions/ai_scribe/handle_ai_scribe_in_composer_extension.dart: L33, L53, L71, L83, > * lib/features/composer/presentation/widgets/mobile/mobile_editor_widget.dart:...