search

mattermost / mattermost-mobile

Next generation iOS and Android apps for Mattermost in React Native
https://about.mattermost.com/
Apache License 2.0
2.22k stars 1.35k forks source link

Convert image size on fly #7818

Open jmmarcus opened 8 months ago

jmmarcus commented 8 months ago

Per Mattermost guidelines, GitHub issues are for bug reports: http://www.mattermost.org/filing-issues/.

For troubleshooting see: http://forum.mattermost.org/. For feature proposals see: http://www.mattermost.org/feature-requests/

If you've found a bug--something appears unintentional--please follow these steps:

Summary

Convert image size on fly

Not find other ticket on this subject

Environment Information

Steps to reproduce

Limit size to 1 MO and error message

Expected behavior

In fact this not a problem on Android device But with Iphone we haven't the avalability to reduce the image size on fly when we post it

Observed behavior (that appears unintentional)

Not really an error just a necessity to implement this fonction because the use of external application must introducing a security fail

amyblais commented 8 months ago

Hi @jmmarcus, I may not be fully understanding the issue.

  1. Can you help share more details about the error you're seeing, and what image size you're attaching?
  2. Can you help share more details about the use cases for being able to convert image sizes?
jmmarcus commented 8 months ago

Hi tank you, 1 When the image size is Up to 1mo. Then the iphone philosophy is the better for us. In this case, can't reduce image size on the fly like in Mastodon. I think it's the server that implements this function on Mastodon. I'd like the Mattermost server to do the same. 2 For now, I'm using a third-party app "ImageSmaller" to reduce the size. She doesn't seem to collect any personal information but it could potentially be a Trojan horse without me knowing. For this reason, it would be better for the server to handle this function.

I don't know if my proposal is wise but I took care of this question asked by French users who would like this feature. According to the feedback, it would be easy with Android but not with iphone

amyblais commented 8 months ago

Thank you for the clarification!

Would you like to contribute this in the feature idea forum so it can be discussed, upvoted and considered for a help wanted ticket?

Please include a link back to this GitHub issue. If you're interested in implementing, please say so and we'll prioritize the review.

You get 10 votes in the feature idea forum, and each one influences the future of the project.