Skip to content
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

Fixes #1970 Increasing Image File Size Limit to 50MB #2112

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mjmatthiesen
Copy link

@mjmatthiesen mjmatthiesen commented Sep 27, 2024

Summary

Increase the file size limit from 5MB to 50MB so more image processing can be done within BC rather than externally

Work Item(s)

Fixes #1970

Fixes AB#550942

@mjmatthiesen mjmatthiesen requested a review from a team as a code owner September 27, 2024 21:11
@github-actions github-actions bot added AL: System Application From Fork Pull request is coming from a fork labels Sep 27, 2024
@mjmatthiesen
Copy link
Author

@JesperSchulz Hi Jesper, how do I get the AB work item number it's currently failing on? I didn't see that mentioned in the contributing.md doc. Thanks

@JesperSchulz JesperSchulz added the Linked Issue is linked to a Azure Boards work item label Sep 30, 2024
@github-actions github-actions bot added this to the Version 26.0 milestone Sep 30, 2024
@JesperSchulz
Copy link
Contributor

@JesperSchulz Hi Jesper, how do I get the AB work item number it's currently failing on? I didn't see that mentioned in the contributing.md doc. Thanks

It's either automatically added when an idea already exists, or it's added when the "linked" label is added. I've done the latter here 😊 We'll review this PR asap.

@JesperSchulz JesperSchulz added the Integration GitHub request for Integration area label Sep 30, 2024
@JesperSchulz
Copy link
Contributor

We're still discussing what the possible side-effects of this change could be. This change will for sure increase the memory footprint of the service where this is extensively used.

@JesperSchulz JesperSchulz added the Follow Up The issue has an open question and must be followed up on label Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AL: System Application Follow Up The issue has an open question and must be followed up on From Fork Pull request is coming from a fork Integration GitHub request for Integration area Linked Issue is linked to a Azure Boards work item
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Reasoning for Image Codeunit being limited to 5MB?
3 participants