-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Need to split the maximum size for tipline content video files. #2048
Need to split the maximum size for tipline content video files. #2048
Conversation
We need one maximum size for the Check upload and another one that is actually the maximum size supported by WhatsApp. Fixes: CV2-5326.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just have a suggestion to read size values from config not hard coded
def header_file_video_max_size | ||
# Max size that WhatsApp supports | ||
def header_file_video_max_size_whatsapp | ||
16 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's better to read the value from config not hard coded
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seconded!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. thanks!
end | ||
|
||
# Max size for Check (we need to convert it to H.264, so let's be safe) | ||
def header_file_video_max_size_check | ||
10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's better to read the value from config not hard coded
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. thanks!
Thanks guys, I turned those into configuration keys. |
Description
We need one maximum size for the Check upload and another one that is actually the maximum size supported by WhatsApp. This separation is needed because we need to convert the uploaded video to H.264, which is the codec support by WhatsApp. So we need a safe margin for that.
Fixes: CV2-5326.
How has this been tested?
Unit test updated.
Checklist