Improve file structure of components in Boilerplate (#10968) #10970
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the WalkthroughThis change reorganizes namespaces and file structures for components in the Boilerplate project template, aligning them with a more logical and maintainable hierarchy. Several class and namespace renamings were performed, SCSS import paths were adjusted, and redundant or obsolete files were removed or updated. No functional logic was altered. Changes
Sequence Diagram(s)Not applicable: The changes are limited to namespace restructuring, file moves, and minor non-functional updates. Assessment against linked issues
Assessment against linked issues: Out-of-scope changesNo out-of-scope functional code changes detected. Poem
✨ Finishing Touches🧪 Generate Unit Tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (3)
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Categories/AddOrEditCategoryModal.razor.cs (1)
1-4: Same namespace update — see earlier verification noteThe change mirrors the dashboard refactor; nothing further.
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Categories/CategoriesPage.razor.cs (1)
1-5: Namespace update mirrors prior commentNo additional issues spotted.
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/TwoFactorSection.razor.cs (1)
1-4: Namespace realignment consistentChange is identical in spirit to previous files; nothing else to flag.
🧹 Nitpick comments (6)
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Categories/CategoriesPage.razor.scss (1)
1-2: Validate and centralize SCSS imports. The reduced../../../Styles/abstractspath aligns with other component stylesheets—please confirm these imports still resolve correctly after the change. Consider using a shared index file or Sass alias to minimize fragile relative paths.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/AppMenu.razor.scss (1)
1-2: Confirm updated SCSS import paths for AppMenu. Paths were adjusted to../../../Styles/abstracts/...to match the new folder structure. Please verify these partials still compile and consider centralizing imports if this pattern repeats.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Management/RolesPage.razor (1)
118-118: Validate header text localization with zero-count case.
Currently the code always invokesLocalizereven whenselectedRoleUsers.Countis 0, passing an empty string which may produce a trailing space or unintended output. Consider moving the count check outside the localization call to only invokeLocalizerwhenCount > 0, e.g.:HeaderText="@(selectedRoleUsers.Count > 0 ? Localizer[nameof(AppStrings.UsersWithCount), $"({selectedRoleUsers.Count})"] : string.Empty)"src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Boilerplate.Client.Core.csproj (1)
47-47: Add<Using>for new common components namespace
IncludingBoilerplate.Client.Core.Components.Commonensures shared components are globally available. For consistency, consider placing it alphabetically before theLayoutentry.Proposed diff:
- <Using Include="Boilerplate.Client.Core.Components.Layout" /> - <Using Include="Boilerplate.Client.Core.Components.Common" /> + <Using Include="Boilerplate.Client.Core.Components.Common" /> + <Using Include="Boilerplate.Client.Core.Components.Layout" />src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/DashboardPage.razor.cs (1)
1-1: Remove unintended BOM/invisible character.
The file begins with a byte order mark or zero-width character before the//+:cnd:noEmitcomment. This can cause encoding inconsistencies and build issues. Please save the file in UTF-8 without BOM.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/OverallStatsWidget.razor.cs (1)
1-1: Remove unintended BOM/invisible character.
An extra BOM or zero-width character appears before the firstusingdirective. Please ensure the file is encoded as UTF-8 without BOM to avoid hidden characters in source control.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Knowledge Base: Disabled due to Reviews > Disable Knowledge Base setting
📒 Files selected for processing (60)
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Boilerplate.Client.Core.csproj(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/AppClientCoordinator.cs(2 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/AppErrorBoundary.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Common/AppPageData.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Common/LoadingComponent.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Common/Prompt.razor.cs(2 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Common/RedirectToSignIn.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/AppErrorBoundary.razor.scss(0 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/AppMenu.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/AppMenu.razor.scss(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/DiagnosticSpacer.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/Header.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/Header.razor.scss(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/IdentityHeader.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/IdentityHeader.razor.scss(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/SignOutConfirmDialog.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/_Imports.razor(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Authorized/Settings/UpgradeAccountSection.razor.scss(0 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Categories/AddOrEditCategoryModal.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Categories/AddOrEditCategoryModal.razor.scss(0 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Categories/CategoriesPage.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Categories/CategoriesPage.razor.scss(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/DashboardPage.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/OverallStatsWidget.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/OverallStatusCard.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/OverallStatusCard.razor.scss(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/ProductsCountPerCategoryWidget.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/ProductsCountPerCategoryWidget.razor.scss(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/ProductsPercentageWidget.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/ProductsPercentageWidget.razor.scss(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Management/RolesPage.razor(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Management/RolesPage.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Management/RolesPage.razor.scss(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Management/UsersPage.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Management/UsersPage.razor.scss(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Products/AddOrEditProductPage.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Products/ProductsPage.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Products/ProductsPage.razor.scss(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/Accordion.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/Account/AccountSection.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/Account/ChangeEmailTab.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/Account/ChangePhoneNumberTab.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/Account/DeleteAccountTab.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/Account/PasswordlessTab.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/ProfileSection.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/SessionsSection.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/SessionsSection.razor.scss(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/SettingsPage.razor(0 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/SettingsPage.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/TwoFactorSection.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/TwoFactorSection.razor.scss(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/UpgradeAccountSection.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/UpgradeAccountSection.razor.scss(0 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/_Imports.razor(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/SystemPromptsPage.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/TodoPage.razor.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/TodoPage.razor.scss(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/_Imports.razor(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Services/PromptService.cs(1 hunks)src/Templates/Boilerplate/Bit.Boilerplate/src/Server/Boilerplate.Server.Web/Components/_Imports.razor(1 hunks)
💤 Files with no reviewable changes (5)
- src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/SettingsPage.razor
- src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/UpgradeAccountSection.razor.scss
- src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/AppErrorBoundary.razor.scss
- src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Authorized/Settings/UpgradeAccountSection.razor.scss
- src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Categories/AddOrEditCategoryModal.razor.scss
🧰 Additional context used
🧬 Code Graph Analysis (1)
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Management/RolesPage.razor.cs (1)
src/Templates/Boilerplate/Bit.Boilerplate/src/Shared/Services/AppFeatures.cs (1)
Management(10-20)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build and test
🔇 Additional comments (48)
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/Account/AccountSection.razor.cs (1)
3-3: Updated file-scoped namespace is correct
The removal of theAuthorizedsegment aligns with the new directory structure and employs C# 10 file-scoped namespace syntax. Ensure the physical folder path matches.../Pages/Settings/Accountto avoid any mismatches during compilation.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Products/ProductsPage.razor.scss (1)
1-2: Corrected SCSS import paths
The adjusted relative paths now correctly resolve tosrc/Client/Boilerplate.Client.Core/Styles/abstracts/..., matching the project’s directory structure and keeping imports consistent across components.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Products/ProductsPage.razor.cs (1)
5-5: Verify namespace-consistency in Razor markup
The code-behind namespace was updated toBoilerplate.Client.Core.Components.Pages.Productsto match the new folder layout. Please confirm that the corresponding.razorfile’s@namespace(and any route directives) have been updated accordingly to avoid namespace mismatches at build/runtime.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/_Imports.razor (1)
26-26: Approve added import for service contracts.Including
@using Boilerplate.Client.Core.Services.Contractsensures components in this namespace can reference your service interfaces without fully qualifying them.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/TwoFactorSection.razor.scss (1)
1-3: Approve new component SCSS.The rule setting
section { width: 100%; }in an isolated.razor.scssfile will scope to this component only under Blazor's CSS isolation. No global side‐effects are introduced.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/ProductsCountPerCategoryWidget.razor.scss (1)
1-1: Approve updated import path.The adjusted import from
../../../Styles/abstracts/_media-queries.scsscorrectly reflects the new folder hierarchy.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/TodoPage.razor.cs (1)
4-4: Approve flattened namespace.Moving
TodoPagetoBoilerplate.Client.Core.Components.Pagesaligns with the updated folder structure and removes the obsoleteAuthorizedsegment.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/_Imports.razor (1)
1-2: Centralize Settings namespace imports. Introducing_Imports.razorwith@using Boilerplate.Client.Core.Components.Pages.Settings.Accountcleans up individual component files. Verify that the custom@*+:cnd:noEmit*@directive is recognized and applied consistently by your build tooling.src/Templates/Boilerplate/Bit.Boilerplate/src/Server/Boilerplate.Server.Web/Components/_Imports.razor (1)
10-10: Verify necessity of new common components import. Adding@using Boilerplate.Client.Core.Components.Commonmakes shared client-core components available server-side. Ensure these components are actually referenced to avoid redundant imports.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Common/AppPageData.razor.cs (1)
1-1: Validate namespace alignment with folder structure. The namespaceBoilerplate.Client.Core.Components.Commonnow mirrors theComponents/Commondirectory. Confirm that your project’s default<Using>entries and any_Imports.razorfiles are updated so all references resolve correctly.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Common/LoadingComponent.razor.cs (1)
2-2: Align file-scoped namespace with folder structure.The namespace has been updated to
Boilerplate.Client.Core.Components.Common, which matches theComponents/Commondirectory.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/AppErrorBoundary.razor.cs (1)
1-1: Align file-scoped namespace with folder.Updated from
.Components.Layoutto.Components, matching the file's location inComponents/.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Management/RolesPage.razor.scss (1)
1-2: Verify updated SCSS import paths.Changed imports to
../../../Styles/...; ensure the build resolves these paths correctly and no import errors occur.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/Header.razor.scss (1)
1-2: Verify SCSS import path adjustment.Updated relative imports to
../../../Styles/...; confirm correct file resolution and that styles compile without regressions.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/AppMenu.razor.cs (1)
5-5: Align file-scoped namespace with folder structure.Updated to
Boilerplate.Client.Core.Components.Layout.Header, matching theComponents/Layout/Headerdirectory.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/_Imports.razor (1)
1-2: Register header namespace for Layout componentsThe new
_Imports.razorcorrectly imports theLayout.Headernamespace, simplifying component usage in this folder.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/Header.razor.cs (1)
3-3: Apply C# 10 file-scoped namespaceThe namespace is updated to
Boilerplate.Client.Core.Components.Layout.Header;, aligning with the directory structure and using the concise file-scoped syntax.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/DiagnosticSpacer.razor.cs (1)
1-1: Apply C# 10 file-scoped namespaceUpdated namespace to
Boilerplate.Client.Core.Components.Layout.Header;to match theHeaderfolder location.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/ProductsPercentageWidget.razor.cs (1)
1-1: Align widget with refactored namespace and dependenciesThe
using Boilerplate.Shared.Controllers.Dashboarddirective and file-scoped namespaceBoilerplate.Client.Core.Components.Pages.Dashboard;have been updated to reflect the new folder layout and correct controller import.Also applies to: 3-3
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/ProductsPercentageWidget.razor.scss (1)
1-1: Correct SCSS import path depthThe import path for
_media-queries.scssis now at three levels up, ensuring the shared style file is resolved correctly.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/SystemPromptsPage.razor.cs (1)
5-5: Namespace updated correctly: The newBoilerplate.Client.Core.Components.Pagesnamespace aligns with the file’s folder structure and removes the obsoleteAuthorizedsegment.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Management/UsersPage.razor.scss (1)
1-2: SCSS import paths updated correctly: Reduced directory traversal to three levels, matching the project’sStyles/abstractsdirectory and aligning with other .scss files.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/SessionsSection.razor.scss (1)
1-1: SCSS import path adjusted correctly: Reduced one level of traversal to reference_media-queries.scssfrom theStyles/abstractsfolder, consistent with the broader restructure.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/SettingsPage.razor.cs (1)
1-1: Namespace updated correctly: ThePages.Settingsnamespace now matches the folder path and removes the oldAuthorizedsegment. Ensure the newly-added_Imports.razorin this directory includes@using Boilerplate.Client.Core.Components.Pages.Settings.Account.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Management/RolesPage.razor.cs (1)
5-5: Namespace updated correctly: Switching toPages.Managementaligns with the directory structure and drops the deprecatedAuthorizedlayer. Verify that the corresponding.razorand styling files reflect this namespace update.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Management/UsersPage.razor.cs (1)
10-10:✅ Verification successful
Namespace alignment
Namespace updated toBoilerplate.Client.Core.Components.Pages.Managementto reflect the new file path. Ensure all references (including razor markup and imports) have been updated from the oldAuthorized.Managementnamespace.Run this script to catch any lingering references:
🏁 Script executed:
#!/bin/bash rg -n "Boilerplate\.Client\.Core\.Components\.Pages\.Authorized\.Management"Length of output: 78
🏁 Script executed:
#!/bin/bash # Search for any lingering references to the old namespace in C# and Razor files rg -n "Authorized\.Management" -g "*.cs" -g "*.razor" rg -n "Boilerplate\.Client\.Core\.Components\.Pages\.Authorized" -g "*.cs" -g "*.razor"Length of output: 145
Namespace alignment confirmed
All references to the oldAuthorized.Managementnamespace have been removed or replaced. The namespacenamespace Boilerplate.Client.Core.Components.Pages.Management;is correctly applied in
UsersPage.razor.cs, and no lingering usages were found in any.csor.razorfiles.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/ProfileSection.razor.cs (1)
6-6:✅ Verification successful
Namespace realignment for Settings
Changed toBoilerplate.Client.Core.Components.Pages.Settingsto match the relocated folder. Verify that no code still imports the oldAuthorized.Settingsnamespace and that_Imports.razoror other usings have been updated accordingly.
🏁 Script executed:
#!/bin/bash rg -n "Boilerplate\.Client\.Core\.Components\.Pages\.Authorized\.Settings"Length of output: 76
To ensure we’ve caught all lingering imports and updated your Razor imports as well, let’s broaden the search:
🏁 Script executed:
#!/bin/bash # 1. Look for any explicit using directives still pointing at the old namespace rg -n "using Boilerplate\.Client\.Core\.Components\.Pages\.Authorized\.Settings" # 2. Scan all Razor import files for references to Authorized.Settings rg -n "Authorized\.Settings" -g "*_Imports.razor" # 3. Catch any other stray mentions across the repo rg -n "Authorized\.Settings"Length of output: 165
Namespace alignment complete
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/ProfileSection.razor.cshas been updated tonamespace Boilerplate.Client.Core.Components.Pages.Settings;- Ran ripgrep across the repo—no uses of
Authorized.Settingsremain in code or_Imports.razorfiles.No further changes needed.
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/SignOutConfirmDialog.razor.cs (1)
3-3:✅ Verification successful
Updated namespace for header component
MovedSignOutConfirmDialogintoLayout.Headerfor clearer separation of header-related components. Confirm that the new_Imports.razorincludes this namespace and that consumers have updated imports.
🏁 Script executed:
#!/bin/bash rg -n "Boilerplate\.Client\.Core\.Components\.Layout\.(?!Header)"Length of output: 375
To verify that only the
Layout.Headernamespace is used and that_Imports.razorfiles include it, let’s run:
🏁 Script executed:
#!/bin/bash # 1) Find any Layout.* imports outside of Header grep -R -n "Boilerplate\.Client\.Core\.Components\.Layout\." . | grep -v "Layout\.Header\." # 2) Ensure all _Imports.razor files include the new Header namespace find . -name "_Imports.razor" -print0 | xargs -0 grep -Hn "Boilerplate\.Client\.Core\.Components\.Layout\.Header"Length of output: 1492
Namespace and Imports Verified
- No stray
Boilerplate.Client.Core.Components.Layout.*namespaces outside ofLayout.Header_Imports.razorinComponents/Layoutincludes@using Boilerplate.Client.Core.Components.Layout.HeaderAll good to go.
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/TodoPage.razor.scss (1)
1-2:✅ Verification successful
SCSS import path update
Adjusted relative traversal to two levels up (../../Styles/abstracts/...) to align with the reorganized folder structure. Ensure these partials are reachable and the styles compile without errors.
🏁 Script executed:
#!/bin/bash # Verify partial locations from this file’s directory fd --base-directory src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core --relative-path "Styles/abstracts/_media-queries.scss" fd --base-directory src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core --relative-path "Styles/abstracts/_bit-css-variables.scss"Length of output: 307
🏁 Script executed:
#!/bin/bash # Search for the SCSS partials across the repository to confirm their locations fd --type f --name '_media-queries.scss' fd --type f --name '_bit-css-variables.scss'Length of output: 587
🏁 Script executed:
#!/bin/bash # Properly search for SCSS partials by filename fd _media-queries.scss -t f fd _bit-css-variables.scss -t fLength of output: 937
SCSS import paths validated
Both_media-queries.scssand_bit-css-variables.scssare located at
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Styles/abstracts,
which matches the../../Styles/abstracts/...imports inTodoPage.razor.scss. No further changes required.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/OverallStatusCard.razor.scss (1)
1-1:✅ Verification successful
SCSS partial import correction
Updated import to../../../Styles/abstracts/_media-queries.scssreflecting the new component hierarchy. Verify that_media-queries.scssis present at this location and that no build warnings are introduced.
🏁 Script executed:
#!/bin/bash fd --base-directory src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core --relative-path "Styles/abstracts/_media-queries.scss"Length of output: 151
🏁 Script executed:
#!/bin/bash # Search for _media-queries.scss across the entire repository fd --type f "_media-queries.scss"Length of output: 507
Import path verified
Confirmed that_media-queries.scssexists at
src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Styles/abstracts/_media-queries.scss
No build warnings are expected from this import.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/Accordion.razor.cs (1)
1-1: Ensure namespace alignment and update related references.
The namespace was updated to remove theAuthorizedsegment. Verify that the corresponding.razormarkup (and any explicit@namespacedirectives) and all component imports/usages reflect this change to avoid mismatches at compile or runtime.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Products/AddOrEditProductPage.razor.cs (1)
6-6: Verify namespace update across markup and code-behind.
TheAuthorizedsegment was removed from the namespace. Ensure the.razorfile’s@namespace(if specified) and any project-level imports or references to this component have been updated accordingly.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/UpgradeAccountSection.razor.cs (1)
3-3: Confirm namespace realignment and razor updates.
The namespace was changed to dropAuthorized. Make sure the.razormarkup’s namespace and any references (e.g., imports in parent components) are in sync to prevent broken component resolution.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/SessionsSection.razor.cs (1)
5-5: Validate namespace harmonization and matching markup.
TheAuthorizedsegment was removed. Verify that the Razor file’s implicit or explicit namespace remains correct and that anyusingor@usingstatements are updated so the component still compiles and renders.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/OverallStatusCard.razor.cs (1)
1-1: Namespace adjusted to match folder hierarchy
TheBoilerplate.Client.Core.Components.Pages.Dashboardnamespace now aligns with the file’s path underComponents/Pages/Dashboard.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/IdentityHeader.razor.scss (1)
1-1: SCSS import path consolidated
Importing only the CSS variables reduces redundancy. The relative path../../../Styles/abstracts/_bit-css-variables.scsscorrectly points to the shared variables.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/Account/DeleteAccountTab.razor.cs (1)
3-3:✅ Verification successful
Namespace updated to remove
Authorizedsegment
The newBoilerplate.Client.Core.Components.Pages.Settings.Accountnamespace matches the directory structure. Ensure no leftover references to the oldAuthorized.Settings.Accountnamespace remain in your_Imports.razorfiles.
🏁 Script executed:
#!/bin/bash # Verify no stale references to the old namespace rg -n "Authorized.Settings.Account"Length of output: 35
Namespace update verified and no stale references remain
The updatedBoilerplate.Client.Core.Components.Pages.Settings.Accountnamespace aligns with the folder structure. I ran a search forAuthorized.Settings.Accountacross the codebase (including_Imports.razorfiles) and found no matches.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/Account/ChangeEmailTab.razor.cs (1)
4-4: Namespace realigned with new folder structure
Switching toBoilerplate.Client.Core.Components.Pages.Settings.Accountkeeps this component consistent with its directory.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/Account/PasswordlessTab.razor.cs (1)
4-4: Namespace updated to match the new folder hierarchy
The namespace now aligns with theSettings/Accountdirectory, reflecting the removal of the obsoleteAuthorizedsegment.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Settings/Account/ChangePhoneNumberTab.razor.cs (1)
4-4: Namespace updated to match the new folder hierarchy
The namespace now aligns with theSettings/Accountdirectory, reflecting the removal of the obsoleteAuthorizedsegment.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Layout/Header/IdentityHeader.razor.cs (1)
1-1: Namespace updated to reflect the newLayout.Headergrouping
The namespace now matches theComponents/Layout/Headerpath, ensuring consistency with other header components.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/AppClientCoordinator.cs (2)
21-21: Class renamed toAppClientCoordinatorfor consistency
The class declaration now matches the filename and new naming conventions across the project.
36-36: Logger injection updated to use the new class type
InjectedILogger<AppClientCoordinator>replaces the oldILogger<ClientAppCoordinator>, keeping logging aligned with the renamed class.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Common/Prompt.razor.cs (1)
1-1: Namespace updated toCommonto categorize shared components
Moved thePromptcomponent into theComponents.Commonnamespace to better reflect its role.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/ProductsCountPerCategoryWidget.razor.cs (1)
1-3:✅ Verification successful
Namespace refactor approved – please sweep for dangling references
The
Authorizedsegment has been cleanly removed from the namespace and import lines. Looks good, but please make sure no other files still reference the old namespace to avoid silent runtime failures.
🏁 Script executed:
#!/bin/bash # Search the entire repo for any remaining usages of the removed “Authorized” namespaces rg -n 'Boilerplate\.Client\.Core\.Components\.Pages\.Authorized' -g '*.cs'Length of output: 76
🏁 Script executed:
#!/bin/bash # Comprehensive search for the removed “Authorized” namespace across all files rg -n 'Boilerplate\.Client\.Core\.Components\.Pages\.Authorized' # Fallback search for any usage of “Pages.Authorized” in the repo rg -n 'Pages\.Authorized'Length of output: 94
No remaining Authorized namespace references found
I performed a comprehensive search across all source files for the old
Boilerplate.Client.Core.Components.Pages.Authorizednamespace and found no occurrences. You can safely merge these changes.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Common/RedirectToSignIn.cs (1)
1-3: Component moved to Common – all goodMove to
Components.Commonmakes sense; ensure all Razor views import the new namespace (the server_Imports.razoralready does).src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/DashboardPage.razor.cs (1)
4-4: Approve namespace alignment.
Updating the namespace from.Authorized.Dashboardto.Dashboardcorrectly mirrors the new folder structure and simplifies the component hierarchy.src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/Pages/Dashboard/OverallStatsWidget.razor.cs (1)
4-4: Approve namespace alignment.
Changing the namespace to.Dashboardaccurately reflects the component’s new location and maintains consistency across related widgets.
...tes/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Services/PromptService.cs
Show resolved
Hide resolved
msynk
deleted the
10968-templates-boilerplate-file-structure-improvements
branch
1 participant
closes #10968
Summary by CodeRabbit
Refactor
Style
Bug Fixes
Chores