Skip to content

Fix #19 - prevent thread content from rendering in html in main view#36

Open
sumaiyamannan wants to merge 1 commit intoopen-lms-open-source:masterfrom
sumaiyamannan:issue19new
Open

Fix #19 - prevent thread content from rendering in html in main view#36
sumaiyamannan wants to merge 1 commit intoopen-lms-open-source:masterfrom
sumaiyamannan:issue19new

Conversation

@sumaiyamannan
Copy link
Copy Markdown

Hi

Fixes Rendering image attachments from discussions in main view causes performance issues. #19

@danmarsden
Copy link
Copy Markdown
Contributor

@fabianbatioja @danielcifuentesopen @TheBryanOpen228 - any chance of getting this PR reviewed and included in the main release? - it's been problematic for a few of our clients and this has been sitting in the PR list for a while - we've just rebased it to make it easier but please let us know if you have any feedback and we can try to adjust the patch further if required.

thanks!

@danmarsden
Copy link
Copy Markdown
Contributor

@jgg9026 any chance of getting someone in your team to take a look at this? - we're happy to make further changes if needed!

@jgg9026
Copy link
Copy Markdown
Contributor

jgg9026 commented May 24, 2023

Hi, I'll take a look

@aspark21
Copy link
Copy Markdown

@Julian-Tovar thanks for merging all of the compatibility fixes for 4.4 and 4.5

Would be great if you could also look at merging this one too, which as described in
#19 addresses a major performance bug where the images of all of the threads get loaded on the forum overview page.

We can then finally come back to using the upstream version of this plugin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants