Skip to content

Hotfix: adding backwards compatible exports - #23626

Merged
nielslyngsoe merged 6 commits into
release/17.6.1from
v17/hotfix/uui-backward-compatability-adjustments
Aug 11, 2026
Merged

Hotfix: adding backwards compatible exports#23626
nielslyngsoe merged 6 commits into
release/17.6.1from
v17/hotfix/uui-backward-compatability-adjustments

Conversation

@nielslyngsoe

@nielslyngsoe nielslyngsoe commented Aug 11, 2026

Copy link
Copy Markdown
Member

Adding a few named exports for backwards compatibility with UUI v.1

Copilot AI lite review requested due to automatic review settings August 11, 2026 09:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds backward-compatible export names in the backoffice “external/uui” bridge to ease migration/compatibility with legacy UUI 1.x code while using UUI 2.x.

Changes:

  • Adds deprecated aliases (Timer, ActiveMixinInterface, etc.) mapping legacy names to UUI 2.x exports.
  • Reintroduces a deprecated demandCustomElement helper to warn when required custom elements are not registered.
Suppressed comments (2)

src/Umbraco.Web.UI.Client/src/external/uui/index.ts:32

  • The new JSDoc has a few typos/grammar issues (e.g., “v.17”, “not begin available”, double period) and uses a non-standard @func tag. Since this is a compatibility shim with a planned removal, it would be good to make the deprecation/TODO line consistent with the existing TODO (Vxx) style in this file and tighten the wording.
 * For v.17 we need to keep this function around for backwards compatibility with the legacy UUI 1.x custom elements.
 * TODO: To be removed in v.18.
 * Fire a warning if the custom element with the provided name isn't available.
 * @func demandCustomElement
 * @param {HTMLElement} requester - Reference to the element requiring this custom element..

src/Umbraco.Web.UI.Client/src/external/uui/index.ts:39

  • The default warning message string has a grammar issue (“work appropriate”). Since this is user-facing in the dev console, consider making it read naturally.
	message: string = `This element has to be present for ${requester.nodeName} to work appropriate.`,

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/Umbraco.Web.UI.Client/src/external/uui/index.ts Outdated
nielslyngsoe and others added 2 commits August 11, 2026 11:30
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…ity-adjustments

# Conflicts:
#	src/Umbraco.Web.UI.Client/src/external/uui/index.ts
@nielslyngsoe
nielslyngsoe enabled auto-merge (squash) August 11, 2026 09:32
@madsrasmussen madsrasmussen added release/17.6.1 area/frontend status/regression A previously working feature that has broken or changed behavior unexpectedly type/bug labels Aug 11, 2026
@sonarqubecloud

Copy link
Copy Markdown

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

Labels

area/frontend release/17.6.1 status/regression A previously working feature that has broken or changed behavior unexpectedly type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants