IMediaServiceProvider-Image extensions

[Drjoachim]

Currently looks like this:

 public interface IMediaServiceProvider
    {
        MediaType SupportedType { get; }
        Task<IEnumerable<MediaItem>> GetItems(string tenantId);
        Task PersistMediaFile(UserInfo userInfo, string mediaUrl);
        Task PersistTextMessage(UserInfo userInfo, TextMessage message);
    }

=> But since we are aiming to implement video #125 ; shouldnt it be changed to:

 public interface IMediaServiceProvider
    {
        MediaType SupportedType { get; }
        Task<IEnumerable<MediaItem>> GetItems(string tenantId);
        Task PersistImageFile(UserInfo userInfo, string mediaUrl, string contentType);
        Task PersistTextMessage(UserInfo userInfo, TextMessage message);
        Task PersistVideoFile(UserInfo userInfo, string mediaUrl, string contentType);
    }

with contentype containing the Mimetype.

I was working on #67 and was just wondering how to properly solve this and if you approve

[Drjoachim]

see https://github.com/CoditEU/htc-social-connect/tree/feature/webApp/imageExtension

[tomkerkhove]

I think we need to refactor them into IVideoProvider, ITextMessageProvider & IImageProvider with their dedicated methods aligned to the needs, no?

[Drjoachim]

Makes more sense indeed