[AND-11] Expose flagUser/unflagUser on MessageListController and MessageListViewModels. #5478
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🎯 Goal
Exposes the
flagUser()
andunflagUser()
methods on theMessageListController
andMessageListViewModel (xml and compose)
so that they can be invoked from the UI components, without accessing theChatClient
directly.Linear: https://linear.app/stream/issue/AND-11/expose-flaguserunflaguser-on-messagelistcontrollermessagelistviewmodel
🛠 Implementation details
flagUser()
/unflagUser()
on theMessageListController
(delegates the call to theChatClient
)ErrorEvent.FlagUserError
andErrorEvent.UnflagUserError
for handling errors which occurred while performing the corresponding operationflagUser()
/unflagUser()
on theMessageListViewModel(compose)
Event.FlagUser
andEvent.UnflagUser
for triggering the corresponding operations from thexml
SDK.🎨 UI Changes
NA
🧪 Testing
flagUser
operations.☑️Contributor Checklist
General
develop
branchCode & documentation
☑️Reviewer Checklist
🎉 GIF
Please provide a suitable gif that describes your work on this pull request