TF-78 Implement automatically marked as read when open one email
This commit is contained in:
@@ -6,4 +6,6 @@ abstract class EmailDataSource {
|
|||||||
Future<Email> getEmailContent(AccountId accountId, EmailId emailId);
|
Future<Email> getEmailContent(AccountId accountId, EmailId emailId);
|
||||||
|
|
||||||
Future<bool> sendEmail(AccountId accountId, EmailRequest emailRequest);
|
Future<bool> sendEmail(AccountId accountId, EmailRequest emailRequest);
|
||||||
|
|
||||||
|
Future<bool> markAsRead(AccountId accountId, EmailId emailId, bool unread);
|
||||||
}
|
}
|
||||||
@@ -27,4 +27,13 @@ class EmailDataSourceImpl extends EmailDataSource {
|
|||||||
throw error;
|
throw error;
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@override
|
||||||
|
Future<bool> markAsRead(AccountId accountId, EmailId emailId, bool unread) {
|
||||||
|
return Future.sync(() async {
|
||||||
|
return await emailAPI.markAsRead(accountId, emailId, unread);
|
||||||
|
}).catchError((error) {
|
||||||
|
throw error;
|
||||||
|
});
|
||||||
|
}
|
||||||
}
|
}
|
||||||
@@ -10,6 +10,7 @@ import 'package:jmap_dart_client/jmap/jmap_request.dart';
|
|||||||
import 'package:jmap_dart_client/jmap/mail/email/email.dart';
|
import 'package:jmap_dart_client/jmap/mail/email/email.dart';
|
||||||
import 'package:jmap_dart_client/jmap/mail/email/get/get_email_method.dart';
|
import 'package:jmap_dart_client/jmap/mail/email/get/get_email_method.dart';
|
||||||
import 'package:jmap_dart_client/jmap/mail/email/get/get_email_response.dart';
|
import 'package:jmap_dart_client/jmap/mail/email/get/get_email_response.dart';
|
||||||
|
import 'package:jmap_dart_client/jmap/mail/email/keyword_identifier.dart';
|
||||||
import 'package:jmap_dart_client/jmap/mail/email/set/set_email_method.dart';
|
import 'package:jmap_dart_client/jmap/mail/email/set/set_email_method.dart';
|
||||||
import 'package:jmap_dart_client/jmap/mail/email/set/set_email_response.dart';
|
import 'package:jmap_dart_client/jmap/mail/email/set/set_email_response.dart';
|
||||||
import 'package:jmap_dart_client/jmap/mail/email/submission/address.dart';
|
import 'package:jmap_dart_client/jmap/mail/email/submission/address.dart';
|
||||||
@@ -103,4 +104,33 @@ class EmailAPI {
|
|||||||
throw error;
|
throw error;
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Future<bool> markAsRead(AccountId accountId, EmailId emailId, bool unread) async {
|
||||||
|
final setEmailMethod = SetEmailMethod(accountId)
|
||||||
|
..addUpdates({
|
||||||
|
emailId.id: PatchObject({
|
||||||
|
KeyWordIdentifier.emailSeen.generatePath() : unread ? null : true
|
||||||
|
})
|
||||||
|
});
|
||||||
|
|
||||||
|
final requestBuilder = JmapRequestBuilder(httpClient, ProcessingInvocation());
|
||||||
|
|
||||||
|
final setEmailInvocation = requestBuilder.invocation(setEmailMethod);
|
||||||
|
|
||||||
|
final response = await (requestBuilder
|
||||||
|
..usings(setEmailMethod.requiredCapabilities))
|
||||||
|
.build()
|
||||||
|
.execute();
|
||||||
|
|
||||||
|
final setEmailResponse = response.parse<SetEmailResponse>(
|
||||||
|
setEmailInvocation.methodCallId,
|
||||||
|
SetEmailResponse.deserialize);
|
||||||
|
|
||||||
|
return Future.sync(() async {
|
||||||
|
final emailUpdated = setEmailResponse!.updated![emailId.id];
|
||||||
|
return emailUpdated == null;
|
||||||
|
}).catchError((error) {
|
||||||
|
throw error;
|
||||||
|
});
|
||||||
|
}
|
||||||
}
|
}
|
||||||
@@ -20,4 +20,9 @@ class EmailRepositoryImpl extends EmailRepository {
|
|||||||
Future<bool> sendEmail(AccountId accountId, EmailRequest emailRequest) {
|
Future<bool> sendEmail(AccountId accountId, EmailRequest emailRequest) {
|
||||||
return emailDataSource.sendEmail(accountId, emailRequest);
|
return emailDataSource.sendEmail(accountId, emailRequest);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@override
|
||||||
|
Future<bool> markAsRead(AccountId accountId, EmailId emailId, bool unread) {
|
||||||
|
return emailDataSource.markAsRead(accountId, emailId, unread);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
@@ -6,4 +6,6 @@ abstract class EmailRepository {
|
|||||||
Future<Email> getEmailContent(AccountId accountId, EmailId emailId);
|
Future<Email> getEmailContent(AccountId accountId, EmailId emailId);
|
||||||
|
|
||||||
Future<bool> sendEmail(AccountId accountId, EmailRequest emailRequest);
|
Future<bool> sendEmail(AccountId accountId, EmailRequest emailRequest);
|
||||||
|
|
||||||
|
Future<bool> markAsRead(AccountId accountId, EmailId emailId, bool unread);
|
||||||
}
|
}
|
||||||
@@ -0,0 +1,18 @@
|
|||||||
|
import 'package:core/core.dart';
|
||||||
|
|
||||||
|
class MarkAsEmailReadSuccess extends UIState {
|
||||||
|
|
||||||
|
MarkAsEmailReadSuccess();
|
||||||
|
|
||||||
|
@override
|
||||||
|
List<Object?> get props => [];
|
||||||
|
}
|
||||||
|
|
||||||
|
class MarkAsEmailReadFailure extends FeatureFailure {
|
||||||
|
final exception;
|
||||||
|
|
||||||
|
MarkAsEmailReadFailure(this.exception);
|
||||||
|
|
||||||
|
@override
|
||||||
|
List<Object> get props => [exception];
|
||||||
|
}
|
||||||
@@ -0,0 +1,21 @@
|
|||||||
|
import 'package:core/core.dart';
|
||||||
|
import 'package:dartz/dartz.dart';
|
||||||
|
import 'package:jmap_dart_client/jmap/account_id.dart';
|
||||||
|
import 'package:jmap_dart_client/jmap/mail/email/email.dart';
|
||||||
|
import 'package:tmail_ui_user/features/email/domain/repository/email_repository.dart';
|
||||||
|
import 'package:tmail_ui_user/features/email/domain/state/mark_as_email_read_state.dart';
|
||||||
|
|
||||||
|
class MarkAsEmailReadInteractor {
|
||||||
|
final EmailRepository emailRepository;
|
||||||
|
|
||||||
|
MarkAsEmailReadInteractor(this.emailRepository);
|
||||||
|
|
||||||
|
Future<Either<Failure, Success>> execute(AccountId accountId, EmailId emailId, bool unread) async {
|
||||||
|
try {
|
||||||
|
final result = await emailRepository.markAsRead(accountId, emailId, unread);
|
||||||
|
return result ? Right(MarkAsEmailReadSuccess()) : Left(MarkAsEmailReadFailure(null));
|
||||||
|
} catch (e) {
|
||||||
|
return Left(MarkAsEmailReadFailure(e));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -5,6 +5,7 @@ import 'package:tmail_ui_user/features/email/data/network/email_api.dart';
|
|||||||
import 'package:tmail_ui_user/features/email/data/repository/email_repository_impl.dart';
|
import 'package:tmail_ui_user/features/email/data/repository/email_repository_impl.dart';
|
||||||
import 'package:tmail_ui_user/features/email/domain/repository/email_repository.dart';
|
import 'package:tmail_ui_user/features/email/domain/repository/email_repository.dart';
|
||||||
import 'package:tmail_ui_user/features/email/domain/usecases/get_email_content_interactor.dart';
|
import 'package:tmail_ui_user/features/email/domain/usecases/get_email_content_interactor.dart';
|
||||||
|
import 'package:tmail_ui_user/features/email/domain/usecases/mark_as_email_read_interactor.dart';
|
||||||
import 'package:tmail_ui_user/features/email/presentation/email_controller.dart';
|
import 'package:tmail_ui_user/features/email/presentation/email_controller.dart';
|
||||||
|
|
||||||
class EmailBindings extends Bindings {
|
class EmailBindings extends Bindings {
|
||||||
@@ -15,6 +16,9 @@ class EmailBindings extends Bindings {
|
|||||||
Get.lazyPut(() => EmailRepositoryImpl(Get.find<EmailDataSource>()));
|
Get.lazyPut(() => EmailRepositoryImpl(Get.find<EmailDataSource>()));
|
||||||
Get.lazyPut<EmailRepository>(() => Get.find<EmailRepositoryImpl>());
|
Get.lazyPut<EmailRepository>(() => Get.find<EmailRepositoryImpl>());
|
||||||
Get.lazyPut(() => GetEmailContentInteractor(Get.find<EmailRepository>()));
|
Get.lazyPut(() => GetEmailContentInteractor(Get.find<EmailRepository>()));
|
||||||
Get.put(EmailController(Get.find<GetEmailContentInteractor>()));
|
Get.lazyPut(() => MarkAsEmailReadInteractor(Get.find<EmailRepository>()));
|
||||||
|
Get.put(EmailController(
|
||||||
|
Get.find<GetEmailContentInteractor>(),
|
||||||
|
Get.find<MarkAsEmailReadInteractor>()));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -9,6 +9,7 @@ import 'package:model/model.dart';
|
|||||||
import 'package:tmail_ui_user/features/base/base_controller.dart';
|
import 'package:tmail_ui_user/features/base/base_controller.dart';
|
||||||
import 'package:tmail_ui_user/features/email/domain/state/get_email_content_state.dart';
|
import 'package:tmail_ui_user/features/email/domain/state/get_email_content_state.dart';
|
||||||
import 'package:tmail_ui_user/features/email/domain/usecases/get_email_content_interactor.dart';
|
import 'package:tmail_ui_user/features/email/domain/usecases/get_email_content_interactor.dart';
|
||||||
|
import 'package:tmail_ui_user/features/email/domain/usecases/mark_as_email_read_interactor.dart';
|
||||||
import 'package:tmail_ui_user/features/email/presentation/model/composer_arguments.dart';
|
import 'package:tmail_ui_user/features/email/presentation/model/composer_arguments.dart';
|
||||||
import 'package:tmail_ui_user/features/mailbox_dashboard/presentation/mailbox_dashboard_controller.dart';
|
import 'package:tmail_ui_user/features/mailbox_dashboard/presentation/mailbox_dashboard_controller.dart';
|
||||||
import 'package:tmail_ui_user/main/routes/app_routes.dart';
|
import 'package:tmail_ui_user/main/routes/app_routes.dart';
|
||||||
@@ -17,11 +18,12 @@ class EmailController extends BaseController {
|
|||||||
|
|
||||||
final mailboxDashBoardController = Get.find<MailboxDashBoardController>();
|
final mailboxDashBoardController = Get.find<MailboxDashBoardController>();
|
||||||
final GetEmailContentInteractor _getEmailContentInteractor;
|
final GetEmailContentInteractor _getEmailContentInteractor;
|
||||||
|
final MarkAsEmailReadInteractor _markAsEmailReadInteractor;
|
||||||
|
|
||||||
final emailAddressExpandMode = ExpandMode.COLLAPSE.obs;
|
final emailAddressExpandMode = ExpandMode.COLLAPSE.obs;
|
||||||
EmailContent? emailContent;
|
EmailContent? emailContent;
|
||||||
|
|
||||||
EmailController(this._getEmailContentInteractor);
|
EmailController(this._getEmailContentInteractor, this._markAsEmailReadInteractor);
|
||||||
|
|
||||||
@override
|
@override
|
||||||
void onReady() {
|
void onReady() {
|
||||||
@@ -31,6 +33,7 @@ class EmailController extends BaseController {
|
|||||||
final accountId = mailboxDashBoardController.accountId.value;
|
final accountId = mailboxDashBoardController.accountId.value;
|
||||||
if (accountId != null && presentationEmail != null) {
|
if (accountId != null && presentationEmail != null) {
|
||||||
_getEmailContentAction(accountId, presentationEmail.id);
|
_getEmailContentAction(accountId, presentationEmail.id);
|
||||||
|
markAsEmailRead(unread: false);
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
@@ -67,6 +70,14 @@ class EmailController extends BaseController {
|
|||||||
emailAddressExpandMode.value = expandMode;
|
emailAddressExpandMode.value = expandMode;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
void markAsEmailRead({required bool unread}) async {
|
||||||
|
final accountId = mailboxDashBoardController.accountId.value;
|
||||||
|
final presentationEmail = mailboxDashBoardController.selectedEmail.value;
|
||||||
|
if (accountId != null && presentationEmail != null) {
|
||||||
|
_markAsEmailReadInteractor.execute(accountId, presentationEmail.id, unread);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
bool canComposeEmail() => mailboxDashBoardController.sessionCurrent != null
|
bool canComposeEmail() => mailboxDashBoardController.sessionCurrent != null
|
||||||
&& mailboxDashBoardController.userProfile.value != null
|
&& mailboxDashBoardController.userProfile.value != null
|
||||||
&& mailboxDashBoardController.mapMailboxId.containsKey(PresentationMailbox.roleOutbox)
|
&& mailboxDashBoardController.mapMailboxId.containsKey(PresentationMailbox.roleOutbox)
|
||||||
|
|||||||
@@ -0,0 +1,8 @@
|
|||||||
|
import 'package:jmap_dart_client/jmap/core/patch_object.dart';
|
||||||
|
import 'package:jmap_dart_client/jmap/mail/email/keyword_identifier.dart';
|
||||||
|
|
||||||
|
extension KeyWordIdentifierExtension on KeyWordIdentifier {
|
||||||
|
String generatePath() {
|
||||||
|
return '${PatchObject.keywordsProperty}/$value';
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -37,6 +37,7 @@ export 'extensions/email_extension.dart';
|
|||||||
export 'extensions/presentation_account_extension.dart';
|
export 'extensions/presentation_account_extension.dart';
|
||||||
export 'extensions/user_profile_extension.dart';
|
export 'extensions/user_profile_extension.dart';
|
||||||
export 'extensions/presentation_email_extension.dart';
|
export 'extensions/presentation_email_extension.dart';
|
||||||
|
export 'extensions/keyword_identifier_extension.dart';
|
||||||
|
|
||||||
// Converter
|
// Converter
|
||||||
export 'converter/avatar_id_converter.dart';
|
export 'converter/avatar_id_converter.dart';
|
||||||
|
|||||||
Reference in New Issue
Block a user