From b0fc4377198cacce836786de7b0f2d8d8699c3c0 Mon Sep 17 00:00:00 2001 From: Salvatore Giordano Date: Thu, 24 Mar 2022 11:09:37 +0100 Subject: [PATCH 01/16] feat(llc): handle event.message in channel.truncate events --- packages/stream_chat/lib/src/client/channel.dart | 3 +++ 1 file changed, 3 insertions(+) diff --git a/packages/stream_chat/lib/src/client/channel.dart b/packages/stream_chat/lib/src/client/channel.dart index f2effe04..25aaa249 100644 --- a/packages/stream_chat/lib/src/client/channel.dart +++ b/packages/stream_chat/lib/src/client/channel.dart @@ -1612,6 +1612,9 @@ class ChannelClientState { await _channel._client.chatPersistenceClient ?.deleteMessageByCid(channel.cid); truncate(); + if (event.message != null) { + updateMessage(event.message!); + } })); } From 058e2b42ccfbfcbe5860ea9aaf38450aab14a93d Mon Sep 17 00:00:00 2001 From: Salvatore Giordano Date: Thu, 24 Mar 2022 11:10:32 +0100 Subject: [PATCH 02/16] chore(llc): update changelog --- packages/stream_chat/CHANGELOG.md | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/packages/stream_chat/CHANGELOG.md b/packages/stream_chat/CHANGELOG.md index a74b920e..e054c420 100644 --- a/packages/stream_chat/CHANGELOG.md +++ b/packages/stream_chat/CHANGELOG.md @@ -1,3 +1,9 @@ +## Upcoming + +✅ Added + +- Handle `event.message` in `channel.truncate` events + ## 3.5.1 🐞 Fixed From b11a812721d6d0e8508afd2ed8a32a3f32c5099b Mon Sep 17 00:00:00 2001 From: Salvatore Giordano Date: Thu, 24 Mar 2022 11:42:46 +0100 Subject: [PATCH 03/16] fix(ui): fix initial alignment in message list view --- packages/stream_chat_flutter/lib/src/message_list_view.dart | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/stream_chat_flutter/lib/src/message_list_view.dart b/packages/stream_chat_flutter/lib/src/message_list_view.dart index c6e269b9..9f670f15 100644 --- a/packages/stream_chat_flutter/lib/src/message_list_view.dart +++ b/packages/stream_chat_flutter/lib/src/message_list_view.dart @@ -369,7 +369,7 @@ class _MessageListViewState extends State { double get _initialAlignment { final initialAlignment = widget.initialAlignment; if (initialAlignment != null) return initialAlignment; - return 0.1; + return streamChannel!.initialMessageId == null ? 0 : 0.1; } bool _isInitialMessage(String id) => streamChannel!.initialMessageId == id; From 338b8422fdcda57194c03c3ddc8ce19bf86e802c Mon Sep 17 00:00:00 2001 From: Salvatore Giordano Date: Thu, 24 Mar 2022 11:44:46 +0100 Subject: [PATCH 04/16] chore(ui): update changelog --- packages/stream_chat_flutter/CHANGELOG.md | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/packages/stream_chat_flutter/CHANGELOG.md b/packages/stream_chat_flutter/CHANGELOG.md index 78a52f64..b8cd20ff 100644 --- a/packages/stream_chat_flutter/CHANGELOG.md +++ b/packages/stream_chat_flutter/CHANGELOG.md @@ -1,3 +1,9 @@ +## Upcoming + +🐞 Fixed + +-[[#892]](https://github.com/GetStream/stream-chat-flutter/issues/892): Fix default `initialAlignment` in `MessageListView`. + ## 3.5.1 🐞 Fixed From e4c6d8ccda3a5de4dcdf0b53ca48771ffa4fd146 Mon Sep 17 00:00:00 2001 From: Sahil Kumar Date: Thu, 24 Mar 2022 20:46:05 +0530 Subject: [PATCH 05/16] fix(llc): also save reactions while saving threads data in persistence Signed-off-by: xsahil03x --- .../stream_chat/lib/src/client/channel.dart | 8 ++-- .../lib/src/db/chat_persistence_client.dart | 38 +++++++++++++------ 2 files changed, 31 insertions(+), 15 deletions(-) diff --git a/packages/stream_chat/lib/src/client/channel.dart b/packages/stream_chat/lib/src/client/channel.dart index 25aaa249..b2887168 100644 --- a/packages/stream_chat/lib/src/client/channel.dart +++ b/packages/stream_chat/lib/src/client/channel.dart @@ -2122,12 +2122,12 @@ class ChannelClientState { final BehaviorSubject>> _threadsController = BehaviorSubject.seeded({}); - set _threads(Map> v) { - _channel.client.chatPersistenceClient?.updateMessages( + set _threads(Map> threads) { + _threadsController.add(threads); + _channel.client.chatPersistenceClient?.updateChannelThreads( _channel.cid!, - v.values.expand((v) => v).toList(), + threads, ); - _threadsController.add(v); } /// Channel related typing users last value. diff --git a/packages/stream_chat/lib/src/db/chat_persistence_client.dart b/packages/stream_chat/lib/src/db/chat_persistence_client.dart index 82ba181a..4e10dae3 100644 --- a/packages/stream_chat/lib/src/db/chat_persistence_client.dart +++ b/packages/stream_chat/lib/src/db/chat_persistence_client.dart @@ -197,6 +197,22 @@ abstract class ChatPersistenceClient { /// Deletes all the members by channel [cids] Future deleteMembersByCids(List cids); + /// Updates the channel [cid] threads data along with reactions and users. + Future updateChannelThreads( + String cid, + Map> threads, + ) async { + final messages = threads.values.expand((it) => it).toList(); + final reactions = messages.expand(_expandReactions).toList(); + final users = messages.map((it) => it.user).withNullifyer.toList(); + + await Future.wait([ + updateMessages(cid, messages), + updateReactions(reactions), + updateUsers(users), + ]); + } + /// Update the channel state data using [channelState] Future updateChannelState(ChannelState channelState) => updateChannelStates([channelState]); @@ -239,17 +255,8 @@ abstract class ChatPersistenceClient { channelWithMessages[cid] = messages; channelWithPinnedMessages[cid] = pinnedMessages; - List expandReactions(Message message) { - final own = message.ownReactions; - final latest = message.latestReactions; - return [ - if (own != null) ...own.where((r) => r.userId != null), - if (latest != null) ...latest.where((r) => r.userId != null), - ]; - } - - reactions.addAll(messages.expand(expandReactions)); - pinnedReactions.addAll(pinnedMessages.expand(expandReactions)); + reactions.addAll(messages.expand(_expandReactions)); + pinnedReactions.addAll(pinnedMessages.expand(_expandReactions)); users.addAll([ channel.createdBy, @@ -292,4 +299,13 @@ abstract class ChatPersistenceClient { ), ]); } + + List _expandReactions(Message message) { + final own = message.ownReactions; + final latest = message.latestReactions; + return [ + if (own != null) ...own.where((r) => r.userId != null), + if (latest != null) ...latest.where((r) => r.userId != null), + ]; + } } From d924c18a69e3367d30fc7c0c187ae11c5dbdc095 Mon Sep 17 00:00:00 2001 From: Sahil Kumar Date: Thu, 24 Mar 2022 20:47:09 +0530 Subject: [PATCH 06/16] chore(llc): update CHANGELOG.md Signed-off-by: xsahil03x --- packages/stream_chat/CHANGELOG.md | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/packages/stream_chat/CHANGELOG.md b/packages/stream_chat/CHANGELOG.md index e054c420..de19b33f 100644 --- a/packages/stream_chat/CHANGELOG.md +++ b/packages/stream_chat/CHANGELOG.md @@ -1,5 +1,9 @@ ## Upcoming +🐞 Fixed + +- Fixed reactions not working for threads in offline mode. + ✅ Added - Handle `event.message` in `channel.truncate` events @@ -7,6 +11,7 @@ ## 3.5.1 🐞 Fixed + - `channel.unreadCount` was being set as using global unread count on a very specific case. - The reconnection logic for the WebSocket connection is now more robust. @@ -22,7 +27,7 @@ - [[#890]](https://github.com/GetStream/stream-chat-flutter/pull/890) Fixed Reactions not updating on thread messages. Thanks [bstolinski](https://github.com/bstolinski). - [[#897]](https://github.com/GetStream/stream-chat-flutter/issues/897) Fixed error type mis-match in `AuthInterceptor`. -- [[#891]](https://github.com/GetStream/stream-chat-flutter/pull/891) Fixed reply counter for parent message not +- [[#891]](https://github.com/GetStream/stream-chat-flutter/pull/891) Fixed reply counter for parent message not updating correctly after deleting thread message. - Fix `channelState.copyWith` with respect to pinnedMessages. From dc007653d498c62864ca1e003ee1133fc3147b30 Mon Sep 17 00:00:00 2001 From: Sahil Kumar Date: Thu, 24 Mar 2022 22:55:38 +0530 Subject: [PATCH 07/16] test(llc): fix queryChannels test Signed-off-by: xsahil03x --- .../stream_chat/test/src/client/client_test.dart | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/packages/stream_chat/test/src/client/client_test.dart b/packages/stream_chat/test/src/client/client_test.dart index b59937bb..5ec2bfeb 100644 --- a/packages/stream_chat/test/src/client/client_test.dart +++ b/packages/stream_chat/test/src/client/client_test.dart @@ -645,8 +645,8 @@ void main() { when(() => persistence.getChannelThreads(any())) .thenAnswer((_) async => {}); - when(() => persistence.updateMessages(any(), any())) - .thenAnswer((_) => Future.value()); + when(() => persistence.updateChannelThreads(any(), any())) + .thenAnswer((_) async => {}); when(() => persistence.getChannelStateByCid(any(), messagePagination: any(named: 'messagePagination'), pinnedMessagePagination: @@ -692,7 +692,7 @@ void main() { verify(() => persistence.getChannelThreads(any())) .called((persistentChannelStates + channelStates).length); - verify(() => persistence.updateMessages(any(), any())) + verify(() => persistence.updateChannelThreads(any(), any())) .called((persistentChannelStates + channelStates).length); verify( () => persistence.getChannelStateByCid(any(), @@ -733,8 +733,8 @@ void main() { when(() => persistence.getChannelThreads(any())) .thenAnswer((_) async => {}); - when(() => persistence.updateMessages(any(), any())) - .thenAnswer((_) => Future.value()); + when(() => persistence.updateChannelThreads(any(), any())) + .thenAnswer((_) async => {}); when(() => persistence.getChannelStateByCid(any(), messagePagination: any(named: 'messagePagination'), pinnedMessagePagination: @@ -775,7 +775,7 @@ void main() { verify(() => persistence.getChannelThreads(any())) .called(persistentChannelStates.length); - verify(() => persistence.updateMessages(any(), any())) + verify(() => persistence.updateChannelThreads(any(), any())) .called(persistentChannelStates.length); verify( () => persistence.getChannelStateByCid(any(), From 181a2b7f637b6db8c06afb6081c068a0381bb397 Mon Sep 17 00:00:00 2001 From: Sahil Kumar Date: Fri, 25 Mar 2022 15:31:40 +0530 Subject: [PATCH 08/16] fix(llc): Remove old thread reactions before saving new Signed-off-by: xsahil03x --- .../stream_chat/lib/src/db/chat_persistence_client.dart | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/packages/stream_chat/lib/src/db/chat_persistence_client.dart b/packages/stream_chat/lib/src/db/chat_persistence_client.dart index 4e10dae3..2dfb7cd7 100644 --- a/packages/stream_chat/lib/src/db/chat_persistence_client.dart +++ b/packages/stream_chat/lib/src/db/chat_persistence_client.dart @@ -203,6 +203,12 @@ abstract class ChatPersistenceClient { Map> threads, ) async { final messages = threads.values.expand((it) => it).toList(); + + // Removing old reactions before saving the new + final oldReactions = messages.map((it) => it.id).toList(); + await deleteReactionsByMessageId(oldReactions); + + // Adding new reactions and users data final reactions = messages.expand(_expandReactions).toList(); final users = messages.map((it) => it.user).withNullifyer.toList(); From 604546db382128f8a4a562523333eb69fb5742b4 Mon Sep 17 00:00:00 2001 From: Sahil Kumar Date: Fri, 25 Mar 2022 15:39:28 +0530 Subject: [PATCH 09/16] test(llc): add persistence update thread test Signed-off-by: xsahil03x --- .../src/db/chat_persistence_client_test.dart | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/packages/stream_chat/test/src/db/chat_persistence_client_test.dart b/packages/stream_chat/test/src/db/chat_persistence_client_test.dart index 897bbcb0..7c49cc7d 100644 --- a/packages/stream_chat/test/src/db/chat_persistence_client_test.dart +++ b/packages/stream_chat/test/src/db/chat_persistence_client_test.dart @@ -162,6 +162,23 @@ void main() { expect(channelState, isNotNull); }); + test('updateChannelThreads', () async { + const cid = 'test:cid'; + final user = User(id: 'test-user-id'); + final threads = { + 'parent-test-message': [ + Message( + id: 'test-message', + text: 'test-message', + user: user, + ownReactions: [Reaction(type: 'test', user: user)], + latestReactions: [Reaction(type: 'test', user: user)], + ) + ] + }; + persistenceClient.updateChannelThreads(cid, threads); + }); + test('updateChannelState', () async { final channelState = ChannelState(); persistenceClient.updateChannelState(channelState); From 679faffbaf219b7e22e357e7ce56bd6ab898783c Mon Sep 17 00:00:00 2001 From: Sahil Kumar Date: Wed, 30 Mar 2022 20:43:21 +0530 Subject: [PATCH 10/16] fix(llc): segregate mute from channel mutes Signed-off-by: xsahil03x --- .../lib/src/core/models/channel_mute.dart | 37 +++++++++++++++++++ .../lib/src/core/models/channel_mute.g.dart | 17 +++++++++ .../stream_chat/lib/src/core/models/mute.dart | 14 +++---- .../lib/src/core/models/mute.g.dart | 5 ++- .../lib/src/core/models/own_user.dart | 5 ++- .../lib/src/core/models/own_user.g.dart | 2 +- .../fixtures/{mute.json => channel_mute.json} | 0 ...{mute_test.dart => channel_mute_test.dart} | 6 +-- .../test/src/core/models/own_user_test.dart | 5 ++- 9 files changed, 75 insertions(+), 16 deletions(-) create mode 100644 packages/stream_chat/lib/src/core/models/channel_mute.dart create mode 100644 packages/stream_chat/lib/src/core/models/channel_mute.g.dart rename packages/stream_chat/test/fixtures/{mute.json => channel_mute.json} (100%) rename packages/stream_chat/test/src/core/models/{mute_test.dart => channel_mute_test.dart} (70%) diff --git a/packages/stream_chat/lib/src/core/models/channel_mute.dart b/packages/stream_chat/lib/src/core/models/channel_mute.dart new file mode 100644 index 00000000..67410afc --- /dev/null +++ b/packages/stream_chat/lib/src/core/models/channel_mute.dart @@ -0,0 +1,37 @@ +import 'package:json_annotation/json_annotation.dart'; +import 'package:stream_chat/src/core/models/channel_model.dart'; +import 'package:stream_chat/src/core/models/user.dart'; + +part 'channel_mute.g.dart'; + +/// The class that contains the information about a muted user +@JsonSerializable(createToJson: false) +class ChannelMute { + /// Constructor used for json serialization + ChannelMute({ + required this.user, + required this.channel, + required this.createdAt, + required this.updatedAt, + this.expires, + }); + + /// Create a new instance from a json + factory ChannelMute.fromJson(Map json) => + _$ChannelMuteFromJson(json); + + /// The user that performed the muting action + final User user; + + /// The target user + final ChannelModel channel; + + /// The date in which the use was muted + final DateTime createdAt; + + /// The date of the last update + final DateTime updatedAt; + + /// The date in which the mute expires + final DateTime? expires; +} diff --git a/packages/stream_chat/lib/src/core/models/channel_mute.g.dart b/packages/stream_chat/lib/src/core/models/channel_mute.g.dart new file mode 100644 index 00000000..9b6d00c1 --- /dev/null +++ b/packages/stream_chat/lib/src/core/models/channel_mute.g.dart @@ -0,0 +1,17 @@ +// GENERATED CODE - DO NOT MODIFY BY HAND + +part of 'channel_mute.dart'; + +// ************************************************************************** +// JsonSerializableGenerator +// ************************************************************************** + +ChannelMute _$ChannelMuteFromJson(Map json) => ChannelMute( + user: User.fromJson(json['user'] as Map), + channel: ChannelModel.fromJson(json['channel'] as Map), + createdAt: DateTime.parse(json['created_at'] as String), + updatedAt: DateTime.parse(json['updated_at'] as String), + expires: json['expires'] == null + ? null + : DateTime.parse(json['expires'] as String), + ); diff --git a/packages/stream_chat/lib/src/core/models/mute.dart b/packages/stream_chat/lib/src/core/models/mute.dart index 857b34a1..02c91784 100644 --- a/packages/stream_chat/lib/src/core/models/mute.dart +++ b/packages/stream_chat/lib/src/core/models/mute.dart @@ -1,7 +1,5 @@ import 'package:json_annotation/json_annotation.dart'; -import 'package:stream_chat/src/core/models/channel_model.dart'; import 'package:stream_chat/src/core/models/user.dart'; -import 'package:stream_chat/src/core/util/serializer.dart'; part 'mute.g.dart'; @@ -11,27 +9,27 @@ class Mute { /// Constructor used for json serialization Mute({ required this.user, - required this.channel, + required this.target, required this.createdAt, required this.updatedAt, + this.expires, }); /// Create a new instance from a json factory Mute.fromJson(Map json) => _$MuteFromJson(json); /// The user that performed the muting action - @JsonKey(includeIfNull: false, toJson: Serializer.readOnly) final User user; /// The target user - @JsonKey(includeIfNull: false, toJson: Serializer.readOnly) - final ChannelModel channel; + final User target; /// The date in which the use was muted - @JsonKey(includeIfNull: false, toJson: Serializer.readOnly) final DateTime createdAt; /// The date of the last update - @JsonKey(includeIfNull: false, toJson: Serializer.readOnly) final DateTime updatedAt; + + /// The date in which the mute expires + final DateTime? expires; } diff --git a/packages/stream_chat/lib/src/core/models/mute.g.dart b/packages/stream_chat/lib/src/core/models/mute.g.dart index f57fd312..a0d908a8 100644 --- a/packages/stream_chat/lib/src/core/models/mute.g.dart +++ b/packages/stream_chat/lib/src/core/models/mute.g.dart @@ -8,7 +8,10 @@ part of 'mute.dart'; Mute _$MuteFromJson(Map json) => Mute( user: User.fromJson(json['user'] as Map), - channel: ChannelModel.fromJson(json['channel'] as Map), + target: User.fromJson(json['target'] as Map), createdAt: DateTime.parse(json['created_at'] as String), updatedAt: DateTime.parse(json['updated_at'] as String), + expires: json['expires'] == null + ? null + : DateTime.parse(json['expires'] as String), ); diff --git a/packages/stream_chat/lib/src/core/models/own_user.dart b/packages/stream_chat/lib/src/core/models/own_user.dart index 5246f48e..6133dd76 100644 --- a/packages/stream_chat/lib/src/core/models/own_user.dart +++ b/packages/stream_chat/lib/src/core/models/own_user.dart @@ -1,4 +1,5 @@ import 'package:json_annotation/json_annotation.dart'; +import 'package:stream_chat/src/core/models/channel_mute.dart'; import 'package:stream_chat/src/core/util/serializer.dart'; import 'package:stream_chat/stream_chat.dart'; @@ -79,7 +80,7 @@ class OwnUser extends User { bool? banned, DateTime? banExpires, List? teams, - List? channelMutes, + List? channelMutes, List? devices, List? mutes, int? totalUnreadCount, @@ -142,7 +143,7 @@ class OwnUser extends User { /// List of channels muted by the user. @JsonKey(includeIfNull: false) - final List channelMutes; + final List channelMutes; /// Total unread messages by the user. @JsonKey(includeIfNull: false) diff --git a/packages/stream_chat/lib/src/core/models/own_user.g.dart b/packages/stream_chat/lib/src/core/models/own_user.g.dart index be8f2d45..6cae18ee 100644 --- a/packages/stream_chat/lib/src/core/models/own_user.g.dart +++ b/packages/stream_chat/lib/src/core/models/own_user.g.dart @@ -18,7 +18,7 @@ OwnUser _$OwnUserFromJson(Map json) => OwnUser( totalUnreadCount: json['total_unread_count'] as int? ?? 0, unreadChannels: json['unread_channels'] as int? ?? 0, channelMutes: (json['channel_mutes'] as List?) - ?.map((e) => Mute.fromJson(e as Map)) + ?.map((e) => ChannelMute.fromJson(e as Map)) .toList() ?? const [], id: json['id'] as String, diff --git a/packages/stream_chat/test/fixtures/mute.json b/packages/stream_chat/test/fixtures/channel_mute.json similarity index 100% rename from packages/stream_chat/test/fixtures/mute.json rename to packages/stream_chat/test/fixtures/channel_mute.json diff --git a/packages/stream_chat/test/src/core/models/mute_test.dart b/packages/stream_chat/test/src/core/models/channel_mute_test.dart similarity index 70% rename from packages/stream_chat/test/src/core/models/mute_test.dart rename to packages/stream_chat/test/src/core/models/channel_mute_test.dart index f1f7b606..4c63db00 100644 --- a/packages/stream_chat/test/src/core/models/mute_test.dart +++ b/packages/stream_chat/test/src/core/models/channel_mute_test.dart @@ -1,14 +1,14 @@ import 'package:stream_chat/src/core/models/channel_model.dart'; -import 'package:stream_chat/src/core/models/mute.dart'; +import 'package:stream_chat/src/core/models/channel_mute.dart'; import 'package:stream_chat/src/core/models/user.dart'; import 'package:test/test.dart'; import '../../utils.dart'; void main() { - group('src/models/mute', () { + group('src/models/channel_mute', () { test('should parse json correctly', () { - final mute = Mute.fromJson(jsonFixture('mute.json')); + final mute = ChannelMute.fromJson(jsonFixture('channel_mute.json')); expect(mute.channel, isA()); expect(mute.user, isA()); expect(mute.createdAt, DateTime.parse('2020-12-04T10:39:06.512021Z')); diff --git a/packages/stream_chat/test/src/core/models/own_user_test.dart b/packages/stream_chat/test/src/core/models/own_user_test.dart index 430e9c36..e0dc7801 100644 --- a/packages/stream_chat/test/src/core/models/own_user_test.dart +++ b/packages/stream_chat/test/src/core/models/own_user_test.dart @@ -1,4 +1,5 @@ import 'package:mocktail/mocktail.dart'; +import 'package:stream_chat/src/core/models/channel_mute.dart'; import 'package:stream_chat/stream_chat.dart'; import 'package:test/test.dart'; @@ -6,12 +7,14 @@ import '../../utils.dart'; class MockMute extends Mock implements Mute {} +class ChannelMockMute extends Mock implements ChannelMute {} + class MockDevice extends Mock implements Device {} void main() { final devices = [MockDevice(), MockDevice()]; final mutes = [MockMute(), MockMute()]; - final channelMutes = [MockMute()]; + final channelMutes = [ChannelMockMute()]; final createdAt = DateTime.parse('2021-05-03 12:39:21.817646'); final updatedAt = DateTime.parse('2021-04-03 12:39:21.817646'); final lastActive = DateTime.parse('2021-03-03 12:39:21.817646'); From f26553dab3a9b3b0e9f3d5c7534be3a97d6cb25d Mon Sep 17 00:00:00 2001 From: Sahil Kumar Date: Thu, 31 Mar 2022 14:54:36 +0530 Subject: [PATCH 11/16] chore(llc): Update CHANGELOG.md Signed-off-by: xsahil03x --- packages/stream_chat/CHANGELOG.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/packages/stream_chat/CHANGELOG.md b/packages/stream_chat/CHANGELOG.md index de19b33f..a1583dee 100644 --- a/packages/stream_chat/CHANGELOG.md +++ b/packages/stream_chat/CHANGELOG.md @@ -3,6 +3,8 @@ 🐞 Fixed - Fixed reactions not working for threads in offline mode. +- [[#1046]](https://github.com/GetStream/stream-chat-flutter/issues/1046) After `/mute` command on reload cannot access + any channel. ✅ Added From 1a64d045eeb62f89a8ec6d94f9342b4801afce88 Mon Sep 17 00:00:00 2001 From: Sahil Kumar Date: Thu, 31 Mar 2022 15:37:29 +0530 Subject: [PATCH 12/16] fix(llc): merge channel in channel.update event Signed-off-by: xsahil03x --- packages/stream_chat/lib/src/client/channel.dart | 2 +- packages/stream_chat/lib/src/core/models/channel_model.dart | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/stream_chat/lib/src/client/channel.dart b/packages/stream_chat/lib/src/client/channel.dart index b2887168..a98e9467 100644 --- a/packages/stream_chat/lib/src/client/channel.dart +++ b/packages/stream_chat/lib/src/client/channel.dart @@ -1598,7 +1598,7 @@ class ChannelClientState { _subscriptions.add(_channel.on(EventType.channelUpdated).listen((Event e) { final channel = e.channel!; updateChannelState(channelState.copyWith( - channel: channel, + channel: channelState.channel?.merge(channel), members: channel.members, )); })); diff --git a/packages/stream_chat/lib/src/core/models/channel_model.dart b/packages/stream_chat/lib/src/core/models/channel_model.dart index ed7c2c2e..037c1431 100644 --- a/packages/stream_chat/lib/src/core/models/channel_model.dart +++ b/packages/stream_chat/lib/src/core/models/channel_model.dart @@ -172,7 +172,7 @@ class ChannelModel { updatedAt: other.updatedAt, deletedAt: other.deletedAt, memberCount: other.memberCount, - extraData: other.extraData, + extraData: {...extraData, ...other.extraData}, team: other.team, cooldown: other.cooldown, ); From 23dd1b002251aefde3dc309705e3bc6c1b10b8f8 Mon Sep 17 00:00:00 2001 From: Sahil Kumar Date: Thu, 31 Mar 2022 15:39:49 +0530 Subject: [PATCH 13/16] chore(llc): update CHANGELOG.md Signed-off-by: xsahil03x --- packages/stream_chat/CHANGELOG.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/packages/stream_chat/CHANGELOG.md b/packages/stream_chat/CHANGELOG.md index de19b33f..e69501ba 100644 --- a/packages/stream_chat/CHANGELOG.md +++ b/packages/stream_chat/CHANGELOG.md @@ -3,6 +3,8 @@ 🐞 Fixed - Fixed reactions not working for threads in offline mode. +- [[#1047]](https://github.com/GetStream/stream-chat-flutter/issues/1047) `own_capabilities` extraData missing after + channel update. ✅ Added From 4cd8d6b00f41d1d4d9d281655e1d0fa2445de854 Mon Sep 17 00:00:00 2001 From: Sahil Kumar Date: Fri, 1 Apr 2022 13:19:22 +0530 Subject: [PATCH 14/16] Apply suggestions from code review --- packages/stream_chat/lib/src/core/models/channel_mute.dart | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/stream_chat/lib/src/core/models/channel_mute.dart b/packages/stream_chat/lib/src/core/models/channel_mute.dart index 67410afc..31c07f0e 100644 --- a/packages/stream_chat/lib/src/core/models/channel_mute.dart +++ b/packages/stream_chat/lib/src/core/models/channel_mute.dart @@ -4,7 +4,7 @@ import 'package:stream_chat/src/core/models/user.dart'; part 'channel_mute.g.dart'; -/// The class that contains the information about a muted user +/// The class that contains the information about a muted channel @JsonSerializable(createToJson: false) class ChannelMute { /// Constructor used for json serialization @@ -23,7 +23,7 @@ class ChannelMute { /// The user that performed the muting action final User user; - /// The target user + /// The target channel final ChannelModel channel; /// The date in which the use was muted From d938a0759d04a66e886e584832681921a27ffca5 Mon Sep 17 00:00:00 2001 From: Sahil Kumar Date: Fri, 1 Apr 2022 14:10:04 +0530 Subject: [PATCH 15/16] test(llc): add test for mute model Signed-off-by: xsahil03x --- packages/stream_chat/test/fixtures/mute.json | 32 +++++++++++++++++++ .../src/core/models/channel_mute_test.dart | 3 +- .../test/src/core/models/mute_test.dart | 17 ++++++++++ 3 files changed, 51 insertions(+), 1 deletion(-) create mode 100644 packages/stream_chat/test/fixtures/mute.json create mode 100644 packages/stream_chat/test/src/core/models/mute_test.dart diff --git a/packages/stream_chat/test/fixtures/mute.json b/packages/stream_chat/test/fixtures/mute.json new file mode 100644 index 00000000..5e425392 --- /dev/null +++ b/packages/stream_chat/test/fixtures/mute.json @@ -0,0 +1,32 @@ +{ + "user": { + "id": "super-band-9", + "role": "user", + "created_at": "2020-03-03T16:48:28.853674Z", + "updated_at": "2021-05-26T03:22:20.296181Z", + "last_active": "2021-06-16T11:42:29.466165498Z", + "banned": false, + "online": true, + "username": "Rioland", + "image": "https://placehold.jp/150x150.png", + "invisible": false, + "name": "Proud darkness", + "unread_count": 0 + }, + "target": { + "id": "super-band-10", + "role": "user", + "created_at": "2020-03-03T16:48:28.853674Z", + "updated_at": "2021-05-26T03:22:20.296181Z", + "last_active": "2021-06-16T11:42:29.466165498Z", + "banned": false, + "online": true, + "username": "Holland", + "image": "https://placehold.jp/150x150.png", + "invisible": false, + "name": "Proud brightness", + "unread_count": 0 + }, + "created_at": "2020-12-04T10:39:06.512021Z", + "updated_at": "2020-12-04T10:39:06.512021Z" +} \ No newline at end of file diff --git a/packages/stream_chat/test/src/core/models/channel_mute_test.dart b/packages/stream_chat/test/src/core/models/channel_mute_test.dart index 4c63db00..9a84f263 100644 --- a/packages/stream_chat/test/src/core/models/channel_mute_test.dart +++ b/packages/stream_chat/test/src/core/models/channel_mute_test.dart @@ -9,9 +9,10 @@ void main() { group('src/models/channel_mute', () { test('should parse json correctly', () { final mute = ChannelMute.fromJson(jsonFixture('channel_mute.json')); - expect(mute.channel, isA()); expect(mute.user, isA()); + expect(mute.channel, isA()); expect(mute.createdAt, DateTime.parse('2020-12-04T10:39:06.512021Z')); + expect(mute.updatedAt, DateTime.parse('2020-12-04T10:39:06.512021Z')); }); }); } diff --git a/packages/stream_chat/test/src/core/models/mute_test.dart b/packages/stream_chat/test/src/core/models/mute_test.dart new file mode 100644 index 00000000..f5361506 --- /dev/null +++ b/packages/stream_chat/test/src/core/models/mute_test.dart @@ -0,0 +1,17 @@ +import 'package:stream_chat/src/core/models/mute.dart'; +import 'package:stream_chat/src/core/models/user.dart'; +import 'package:test/test.dart'; + +import '../../utils.dart'; + +void main() { + group('src/models/channel_mute', () { + test('should parse json correctly', () { + final mute = Mute.fromJson(jsonFixture('mute.json')); + expect(mute.user, isA()); + expect(mute.target, isA()); + expect(mute.createdAt, DateTime.parse('2020-12-04T10:39:06.512021Z')); + expect(mute.updatedAt, DateTime.parse('2020-12-04T10:39:06.512021Z')); + }); + }); +} From 3ebf05bddf118eaafd31a55c5adfc8da447f3428 Mon Sep 17 00:00:00 2001 From: Sahil Kumar Date: Fri, 1 Apr 2022 14:10:48 +0530 Subject: [PATCH 16/16] Update packages/stream_chat/lib/src/core/models/channel_mute.dart Co-authored-by: Ayush Shekhar --- packages/stream_chat/lib/src/core/models/channel_mute.dart | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/stream_chat/lib/src/core/models/channel_mute.dart b/packages/stream_chat/lib/src/core/models/channel_mute.dart index 31c07f0e..ab17161b 100644 --- a/packages/stream_chat/lib/src/core/models/channel_mute.dart +++ b/packages/stream_chat/lib/src/core/models/channel_mute.dart @@ -26,7 +26,7 @@ class ChannelMute { /// The target channel final ChannelModel channel; - /// The date in which the use was muted + /// The date in which the channel was muted final DateTime createdAt; /// The date of the last update