increase audio and picture file size limit to 10MB #301
Conversation
Test Results 4 files ±0 119 suites ±0 27m 53s ⏱️ - 1m 52s For more details on these failures, see this check. Results for commit d368d78. ± Comparison against base commit 327aa7e. ♻️ This comment has been updated with latest results. |
This is in response to users who report a need to add audio and pictures to their project over 1MB. The 1MB is reported to be too small. Because we don't know practically how this will impact the majority of users, I am happy to introduce this change and evaluate the impact (plus/minus) on Language Forge and FLEx users who use Language Depot to collaborate.
jasonleenaylor
force-pushed
the
feature/10MB
branch
from
94f5be0 to
d368d78
Compare
papeh
left a comment
papeh
left a comment
There was a problem hiding this comment.
Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @megahirt)
until unit tests can be fixed This reverts commit aa17de8.
|
Hi @papeh, I'm confused about the workflow in this PR. It says that the PR was merged with failing unit tests (why was it merged?) but then the commit was later reverted "until unit tests can be fixed". Does this mean that the change is no longer in master? |
Fixes #277
This is in response to users who report a need to add audio and pictures to their project over 1MB in FLEx and Language Forge. The 1MB is reported to be too small.
Because we don't know practically how this will impact the majority of users, I am happy to introduce this change and evaluate the impact (pluses and minuses) on Language Forge and FLEx users who use Language Depot to collaborate, as well as the increased server storage needed to accommodate the higher limits.
This change is