Replies: 1 comment 1 reply
-
|
Looking at the code, I think that this is something that SHOULD be fixed, as this for sure is a performance "oops" In my opinion. I'm going to submit a PR suggestion |
Beta Was this translation helpful? Give feedback.
1 reply
-
|
@thienvc This is a great find, I did confirm the performance issue and submitted a fix here - #6243 |
Beta Was this translation helpful? Give feedback.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
After logging in, the following code slowed down the web performance
Dnn.Platform/DNN Platform/Website/admin/Menus/ModuleActions/ModuleActions.ascx.cs
Line 116 in 2e7a5eb
We should save the "QS_FirstLoad" setting somewhere other than in "ModuleSettings"
Because this action leads to calling ClearCache(tabid), making the cache useless
Beta Was this translation helpful? Give feedback.
All reactions