[PR #186] feat: dynamic Documentation link in help menu based on current module #180

Closed
opened 2026-02-26 17:37:38 +03:00 by kerem · 0 comments
Owner

Original Pull Request: https://github.com/Corsinvest/cv4pve-admin/pull/186

State: closed
Merged: Yes


Summary

  • Help menu refactored from RadzenProfileMenu to contextMenuService.Open, fixing broken action handlers (shortcuts, release notes, bug report, who-is-using) that relied on a Value-based switch
  • Dynamic Documentation link: when navigating to a module, the help menu shows "Documentation - ModuleName" pointing to the module-specific docs page; falls back to generic Documentation otherwise
  • HelpUrl property added to ModuleBase — each module declares its doc path (e.g. "modules/dashboard")
  • GetBySlugGetByUrlIModuleService now resolves a module from a full URI (extracts slug from path internally)
  • Removed duplicate help button from RenderModule header — global help menu in topbar is sufficient
  • DocumentationUrl constant added to ApplicationHelper
  • ModuleComponentUrl now includes trailing slash ("/module/")
  • All 14 modules updated with HelpUrl

Test plan

  • Open app, navigate to a module → help menu shows "Documentation - ModuleName" with correct link
  • Navigate away from module → help menu shows generic "Documentation"
  • Keyboard shortcuts dialog opens correctly
  • Release notes dialog opens correctly
  • Report a Bug opens GitHub issue with pre-filled environment
  • Who is using opens correctly
  • Settings button still visible in module header when user has permission
**Original Pull Request:** https://github.com/Corsinvest/cv4pve-admin/pull/186 **State:** closed **Merged:** Yes --- ## Summary - **Help menu refactored** from `RadzenProfileMenu` to `contextMenuService.Open`, fixing broken action handlers (shortcuts, release notes, bug report, who-is-using) that relied on a Value-based switch - **Dynamic Documentation link**: when navigating to a module, the help menu shows _"Documentation - ModuleName"_ pointing to the module-specific docs page; falls back to generic Documentation otherwise - **`HelpUrl` property** added to `ModuleBase` — each module declares its doc path (e.g. `"modules/dashboard"`) - **`GetBySlug` → `GetByUrl`** — `IModuleService` now resolves a module from a full URI (extracts slug from path internally) - **Removed duplicate help button** from `RenderModule` header — global help menu in topbar is sufficient - **`DocumentationUrl` constant** added to `ApplicationHelper` - **`ModuleComponentUrl`** now includes trailing slash (`"/module/"`) - All 14 modules updated with `HelpUrl` ## Test plan - [ ] Open app, navigate to a module → help menu shows "Documentation - ModuleName" with correct link - [ ] Navigate away from module → help menu shows generic "Documentation" - [ ] Keyboard shortcuts dialog opens correctly - [ ] Release notes dialog opens correctly - [ ] Report a Bug opens GitHub issue with pre-filled environment - [ ] Who is using opens correctly - [ ] Settings button still visible in module header when user has permission
kerem 2026-02-26 17:37:38 +03:00
kerem changed title from [PR #186] [MERGED] feat: dynamic Documentation link in help menu based on current module to [PR #186] feat: dynamic Documentation link in help menu based on current module 2026-03-07 19:57:21 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/cv4pve-admin#180
No description provided.