Skip to content

Add CDV cache buster argument to Persona Bar loads#7074

Merged
donker merged 1 commit intodnnsoftware:developfrom
bdukes:add-cdv-to-persona-bar-css-load
Mar 6, 2026
Merged

Add CDV cache buster argument to Persona Bar loads#7074
donker merged 1 commit intodnnsoftware:developfrom
bdukes:add-cdv-to-persona-bar-css-load

Conversation

@bdukes
Copy link
Copy Markdown
Contributor

@bdukes bdukes commented Mar 6, 2026

Fixes #7066

Summary

In 10.2.3, some of the Persona Bar extensions were migrated from webpack to Rsbuild. One of the consequences of this change is that the CSS for those modules moved out of the JS bundle into a CSS file. Specifically, in 10.2.2, /DesktopModules/Admin/Dnn.PersonaBar/Modules/Dnn.Servers/css/Servers.css was empty, whereas in 10.2.3 it's 20kb (and /DesktopModules/Admin/Dnn.PersonaBar/Modules/Dnn.Servers/scripts/bundles/servers-bundle.js went from 592kb to 506kb).

Previously, the webpack runtime dynamically loaded that CSS, now it is loaded externally by DNN via its Persona Bar infrastructure. For each Persona Bar component it loads, DNN looks for a file sharing the name of the component in a css directory, so css/Servers.css gets loaded automatically in 10.2.3 and everything works as expected.

However, the way that the CSS file is requested causes it to be cached by the browser. So, in the upgrade scenario, the old empty Servers.css is served instead of the new version that has content. Because the Persona Bar is dynamically loaded in an iframe, even doing a hard refresh in the browser does not clear this cache (but using browser dev tools with caching disabled does). Therefore, this PR adds a cache buster parameter to those CSS requests.

@bdukes bdukes added this to the 10.2.4 milestone Mar 6, 2026
Copy link
Copy Markdown
Contributor

@valadas valadas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

Copy link
Copy Markdown
Contributor

@donker donker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@donker donker merged commit 72066d8 into dnnsoftware:develop Mar 6, 2026
3 checks passed
@bdukes bdukes deleted the add-cdv-to-persona-bar-css-load branch March 9, 2026 18:00
@valadas valadas modified the milestones: 10.2.4, 10.3.0 Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Servers tab styling appears broken in v10.02.03

3 participants