Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR restructures the solution to improve maintainability through centralized package management and a more organized directory layout. Projects are moved into dedicated src/, examples/, and tools/ folders, and centralized configuration files (Directory.Build.props, Directory.Packages.props, and NuGet.config) are introduced to manage versioning, dependencies, and build settings across the solution.
- Solution-wide build and package management via
Directory.Build.propsandDirectory.Packages.props - Centralized NuGet source configuration with package source mapping
- Project references updated to reflect new directory structure
Reviewed Changes
Copilot reviewed 11 out of 442 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
Umbraco.Community.BlockPreview.sln |
Reorganized solution with new folder structure, added solution items for config files |
Directory.Build.props |
Centralized build properties including target framework, authorship, and package validation settings |
Directory.Packages.props |
Centralized package version management with version ranges for Umbraco and third-party dependencies |
NuGet.config |
Defined package sources and source mapping for NuGet.org and Umbraco feeds |
Umbraco.Community.BlockPreview.TestSite.csproj |
Updated to use centralized versioning, disabled packing, and adjusted project references |
VERSION |
Removed legacy version file in favor of version.json |
*.config (uSync) |
Updated test site content with serialized Umbraco block data |
oneColumnSectionBlock.cshtml |
Added Razor view for rendering one-column section blocks |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 11 out of 442 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…nity.BlockPreview.TestSite.csproj Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 11 out of 442 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…ickbutterfield/BlockPreview into v4/feature/project-restructure
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 13 out of 465 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Added `launchUrl` property to `launchSettings.json` for IIS Express. - Updated `heroBlock.cshtml` to display `Height` and apply dynamic styles. - Refactored `BlockPreviewService.cs` to: - Add support for `BlockGrid` and `BlockList` property editors. - Ensure mutually exclusive conditions for editor alias handling. - Add fallback handling for `JsonObject` serialization. - Improved code maintainability and extended functionality.
|
This pull request introduces significant improvements to the project structure, centralizes package management, and updates build configuration for better maintainability and compatibility. The main changes include reorganizing the solution layout, introducing central props and package configuration files, updating NuGet sources, and refining project references and build settings.
Project and Solution Structure
Umbraco.Community.BlockPreview.slnhas been reorganized to reflect a more modular structure, moving projects intosrc/,examples/, andtools/directories, and adding solution folders for "Examples" and "Tools". Common files like.gitignore,Directory.Build.props,Directory.Packages.props, andNuGet.configare now tracked as solution items for easier access and management. [1] [2] [3]Centralized Build and Package Management
Directory.Build.propsandDirectory.Packages.propsto enable centralized management of target frameworks, authorship, license, and package versions, including Umbraco and third-party dependencies. These files also configure strictness and validation for package builds and reference version ranges. [1] [2]NuGet Source Configuration
NuGet.configthat defines and maps package sources for NuGet.org and Umbraco prerelease/nightly feeds, ensuring packages are resolved from the correct sources.Test Site Project Updates
Umbraco.Community.BlockPreview.TestSite.csproj) is updated to match the new structure, disables packing and package validation, usesVersionOverridefor package references, and imports the necessary build targets for transitive dependencies. The project reference path is updated to point to the new location.Version File Removal
VERSIONfile has been removed, likely in favor of the new versioning approach managed via props and version.json.