WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@liamgold
Copy link
Owner

@liamgold liamgold commented Dec 3, 2025

Summary

This PR includes breaking changes, build process improvements, and bug fixes for v3.0.0.

Breaking Changes

Minimum Xperience Version: v30.12.0

  • All Kentico packages upgraded from 30.4.2 to 30.12.0
  • DancingGoat example app upgraded to v30.12.0
  • Users must upgrade to Xperience by Kentico v30.12.0 or higher

SustainabilityTabProperties Base Class Change

  • Changed SustainabilityTabProperties to inherit from WebPageBaseClientProperties instead of TemplateClientProperties
  • Fixes compilation error: "The type 'SustainabilityTabProperties' cannot be used as type parameter 'TClientProperties' in the generic type or method 'WebPageBase'"
  • Required for custom pages within the Websites application

Build Process Improvements

Client Build Artifacts (.gitignore)

  • Added src/Client/dist/* to .gitignore
  • Added src/wwwroot/scripts/resource-checker.js to .gitignore
  • Added .gitkeep to preserve src/wwwroot/scripts/ directory structure
  • Removed <Folder Include="wwwroot\scripts\" /> from .csproj (no longer needed)

GitHub Actions Workflows

  • Updated CI and Publish workflows to build client assets before .NET build
  • Added Node.js 22 setup step
  • Added npm ci and npm run build steps for client assets

Node Version Standardization

  • Added .nvmrc file specifying Node 22
  • Added engines field to package.json requiring Node >=22

Package Updates

Backend (NuGet)

  • Kentico.Xperience.WebApp: 30.4.2 → 30.12.0
  • Kentico.Xperience.Admin: 30.4.2 → 30.12.0
  • Kentico.Xperience.AzureStorage: 30.4.2 → 30.12.0
  • Kentico.Xperience.ImageProcessing: 30.4.2 → 30.12.0

Frontend (npm)

  • @kentico/xperience-admin-base: 30.4.2 → 30.12.0
  • @kentico/xperience-admin-components: 30.4.2 → 30.12.0
  • @kentico/xperience-webpack-config: 30.4.2 → 30.12.0

Documentation Updates

  • Updated README Library Version Matrix:
    • >= 30.12.0 → Library v3.x
    • >= 30.4.2 → Library v1.x - 2.x

Version Bump

  • 2.7.1 → 3.0.0 (major version bump due to breaking changes)

Migration Guide

Users upgrading from v2.x to v3.0.0 must:

  1. Upgrade their Xperience by Kentico installation to v30.12.0 or higher
  2. Update the package: dotnet add package XperienceCommunity.Sustainability --version 3.0.0
  3. No code changes required - the breaking changes are internal/dependency-related

Related PRs

This builds on #67 (initial build artifact improvements)

🤖 Generated with Claude Code

liamgold and others added 2 commits December 3, 2025 22:23
- Add src/wwwroot/scripts/resource-checker.js to .gitignore
- Remove previously committed resource-checker.js from git tracking
- Update package-lock.json (npm install with node >=22 engine requirement)

The resource-checker.js in wwwroot/scripts is a webpack build artifact
from src/Client/src/resource-checker.js and should not be committed.
It's already built during CI/CD by the npm run build command.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
BREAKING CHANGES:
- Minimum required Xperience version is now 30.12.0
- SustainabilityTabProperties now inherits from WebPageBaseClientProperties
  instead of TemplateClientProperties (fixes compilation error)

Updates:
- Upgraded all Kentico packages from 30.4.2 to 30.12.0
- Updated @Kentico npm packages to 30.12.0 in client
- Upgraded DancingGoat example app to v30.12.0
- Updated README Library Version Matrix

Version bumped from 2.7.1 to 3.0.0 due to breaking changes.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Copilot AI review requested due to automatic review settings December 3, 2025 22:55
Copy link
Contributor

Copilot AI left a comment

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 upgrades the XperienceCommunity.Sustainability library to version 3.0.0 with breaking changes requiring Xperience by Kentico v30.12.0 or higher, along with build process improvements and bug fixes.

Key Changes:

  • Upgraded all Kentico packages from v30.4.2 to v30.12.0 (breaking change)
  • Fixed SustainabilityTabProperties inheritance to use WebPageBaseClientProperties instead of TemplateClientProperties
  • Improved build process by adding client build artifacts to .gitignore and standardizing Node.js version requirements
  • Updated GitHub Actions workflows to build client assets before .NET build

Reviewed changes

Copilot reviewed 184 out of 189 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/wwwroot/scripts/resource-checker.js Removed generated client build artifact
src/packages.lock.json Updated Kentico package dependencies from 30.4.2 to 30.12.0
src/XperienceCommunity.Sustainability.csproj Bumped version to 3.0.0 and removed unnecessary folder include
src/UIPages/SustainabilityTab/SustainabilityTab.cs Changed base class to WebPageBaseClientProperties
src/Client/package.json Updated Kentico npm packages to 30.12.0
examples/DancingGoat/* Updated DancingGoat example app to v30.12.0 with commerce and email builder improvements
README.md Updated version matrix to reflect breaking changes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@liamgold liamgold merged commit cd3a3ce into main Dec 3, 2025
5 checks passed
@liamgold liamgold deleted the fix/ignore-resource-checker-build-artifact branch December 3, 2025 22:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants