Rebase onto vector-im/riot-ios #97
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Improvements: * Replace the deprecated MXMediaManager and MXMediaLoader interfaces use (see matrix-org/matrix-ios-sdk/pull/593). * Replace the deprecated MXKAttachment and MXKImageView interfaces use (see matrix-org/matrix-ios-kit/pull/487). * Server Quota Notices: Implement the blue banner (1937). * Lazy loading: Enable it by default (if the homeserver supports it). * i18n: Add Spanish (sp). * Settings: Make advanced info copyable (2023). * Settings: Made cryptography info copyable, thanks to @daverPL (PR 1999). Bug fix: * Notifications: old notifications can reappear (1985). * Fix missing read receipts when lazy-loading room members. * Weird text color when selecting a message (2046). * Fix input toolbar reset in RoomViewController on MXSession state change (2006 and 2008). * User defaults: the preset application language (if any) is ignored. * Recents: Avoid to open a room twice (it crashed on room creation on quick HSes). * Riot-bot: Do not try to create a room with it if the user homeserver is not federated. * Empty app if initial /sync fails (1975). * Fix possible race conditions in direct rooms management.
- Remove deprecated MXKImageView interface use
SBiOSoftWhare
approved these changes
Nov 14, 2018
| @@ -19,12 +19,28 @@ import Foundation | |||
| @objcMembers | |||
| final class AvatarImageViewModel: NSObject { | |||
|
|
|||
| let thumbStringUrl: String? | |||
| let avatarContentURI: String? | |||
| let mediaManager: MXMediaManager? | |||
There was a problem hiding this comment.
We should find a way to not set the MXMediaManager on the view model in the future
.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.