Skip to content

Added AppLoaderRegistry#2633

Merged
Corepex merged 4 commits into
1.xfrom
add-app-loader-registry
Nov 28, 2025
Merged

Added AppLoaderRegistry#2633
Corepex merged 4 commits into
1.xfrom
add-app-loader-registry

Conversation

@Corepex

@Corepex Corepex commented Nov 28, 2025

Copy link
Copy Markdown
Contributor

Additional info

This pull request introduces a new centralized registry for application loaders, enabling modular and extensible loading logic for the app. It also adds support for hiding navigation items dynamically and exposes the new registry for external use. The main changes are grouped below:

App Loader Registry Integration

  • Added a new AppLoaderRegistry class in app-loader/services/app-loader-registry.ts to manage and execute multiple loader modules, supporting registration, retrieval, overriding, and batch loading with error handling.
  • Registered AppLoaderRegistry as a singleton service in the DI container, updated serviceIds to include AppLoader/Registry, and imported it in the main app loader logic. [1] [2] [3]
  • Updated AppLoader component to retrieve the registry from the container and call loadAll() during initialization, ensuring all registered loaders are executed at startup. [1] [2] [3]
  • Exported AppLoaderRegistry for external use via the SDK module index.

Navigation Visibility Improvements

  • Added a hidden property to the IMainNavItem interface, allowing navigation items to be conditionally hidden based on custom logic.
  • Updated the main navigation rendering logic to respect the new hidden property, skipping items where hidden() returns true.

@Corepex Corepex self-assigned this Nov 28, 2025
@Corepex
Corepex requested a review from markus-moser November 28, 2025 12:00
@sonarqubecloud

Copy link
Copy Markdown

@Corepex
Corepex merged commit f1bc4ba into 1.x Nov 28, 2025
1 check passed
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 28, 2025
@markus-moser markus-moser added this to the next milestone Dec 9, 2025
@Corepex
Corepex deleted the add-app-loader-registry branch April 8, 2026 06:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants