v4.2 + v4.3: block api v3, latest hero component, latest news component, wchm style, static HTML hero, legacy react hero fallback #327
Merged
Conversation
Bumps [fast-xml-parser](https://github.com/NaturalIntelligence/fast-xml-parser) from 5.5.7 to 5.7.0. - [Release notes](https://github.com/NaturalIntelligence/fast-xml-parser/releases) - [Changelog](https://github.com/NaturalIntelligence/fast-xml-parser/blob/master/CHANGELOG.md) - [Commits](NaturalIntelligence/fast-xml-parser@v5.5.7...v5.7.0) --- updated-dependencies: - dependency-name: fast-xml-parser dependency-version: 5.7.0 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>
…parser-5.7.0 Bump fast-xml-parser from 5.5.7 to 5.7.0
v4.3 - static HTML hero, legacy react hero fallback
v4.2 - block api v3, latest hero component, latest news component, wchm style
…s-10.2.0 Bump ip-address from 10.1.0 to 10.2.0
Bump axios from 1.15.0 to 1.16.0
…builder-1.2.0 Bump fast-xml-builder from 1.1.5 to 1.2.0
Bumps [fast-uri](https://github.com/fastify/fast-uri) from 3.1.0 to 3.1.2. - [Release notes](https://github.com/fastify/fast-uri/releases) - [Commits](fastify/fast-uri@v3.1.0...v3.1.2) --- updated-dependencies: - dependency-name: fast-uri dependency-version: 3.1.2 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
…3.1.2 Bump fast-uri from 3.1.0 to 3.1.2
Bumps and [ws](https://github.com/websockets/ws). These dependencies needed to be updated together. Updates `ws` from 8.18.0 to 8.21.0 - [Release notes](https://github.com/websockets/ws/releases) - [Commits](websockets/ws@8.18.0...8.21.0) Updates `ws` from 8.20.0 to 8.21.0 - [Release notes](https://github.com/websockets/ws/releases) - [Commits](websockets/ws@8.18.0...8.21.0) --- updated-dependencies: - dependency-name: ws dependency-version: 8.20.1 dependency-type: indirect - dependency-name: ws dependency-version: 8.20.1 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
…gin-transform-modules-systemjs-7.29.4 Bump @babel/plugin-transform-modules-systemjs from 7.29.0 to 7.29.4
…870827d Bump ws
Bumps [qs](https://github.com/ljharb/qs) and [express](https://github.com/expressjs/express). These dependencies needed to be updated together. Updates `qs` from 6.13.0 to 6.15.2 - [Changelog](https://github.com/ljharb/qs/blob/main/CHANGELOG.md) - [Commits](ljharb/qs@v6.13.0...v6.15.2) Updates `express` from 4.22.1 to 4.22.2 - [Release notes](https://github.com/expressjs/express/releases) - [Changelog](https://github.com/expressjs/express/blob/v4.22.2/History.md) - [Commits](expressjs/express@v4.22.1...v4.22.2) --- updated-dependencies: - dependency-name: express dependency-version: 4.22.2 dependency-type: indirect - dependency-name: qs dependency-version: 6.15.2 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
…2d6d6d9 Bump qs and express
There was a problem hiding this comment.
Pull request overview
This PR bumps UnityBlocks to v4.3.0 and modernizes several blocks (notably Hero) to improve SEO by rendering static HTML on the frontend while preserving legacy React hydration for previously-saved Hero blocks. It also continues the v4.2 cleanup work (removing deprecated/unfinished blocks and resources) and updates build artifacts/dependencies accordingly.
Changes:
- Hero: switch to static HTML save output, add deprecated old markup + dynamic-import legacy hydration fallback.
- Blocks: move remaining blocks toward Block API v3 and adjust assets (e.g., WCHM CSS).
- Cleanup: remove asu-careers, remove unfinished image-gallery, update manifests/build outputs and npm dependencies.
Reviewed changes
Copilot reviewed 66 out of 79 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| webpack.config.js | Adjust webpack chunk splitting to keep legacy async chunks self-contained. |
| unityblocks.php | Bump plugin version header; remove unused enqueue function. |
| src/wchm/style.scss | Remove SCSS stylesheet (replaced with plain CSS). |
| src/wchm/style.css | Add CSS equivalent of prior SCSS styles. |
| src/wchm/index.js | Update style import from SCSS to CSS. |
| src/wchm/block.json | Update version and switch style to file-based CSS. |
| src/testimonial/inspector.js | Improve image handling in inspector; add alt text field and preview. |
| src/testimonial/block.json | Update block apiVersion to 3. |
| src/news-grid/block.json | Bump news-grid block version. |
| src/image-gallery/save.js | Remove unfinished image-gallery block implementation. |
| src/image-gallery/inspector.js | Remove unfinished image-gallery block implementation. |
| src/image-gallery/index.js | Remove unfinished image-gallery block implementation. |
| src/image-gallery/frontend.js | Remove unfinished image-gallery block implementation. |
| src/image-gallery/edit.js | Remove unfinished image-gallery block implementation. |
| src/image-gallery/controls.js | Remove unfinished image-gallery block implementation. |
| src/image-gallery/block.json | Remove unfinished image-gallery block registration/metadata. |
| src/hero/save.js | Change Hero to serialize static HTML instead of data-* attributes. |
| src/hero/legacy-hydration.js | Add module for dynamic-import legacy hydration dependencies. |
| src/hero/index.js | Register deprecated versions for legacy Hero markup. |
| src/hero/frontend.js | Hydrate only legacy Hero markup via dynamic import. |
| src/hero/deprecated.js | Add deprecated v1 save implementation using data-* attributes. |
| src/hero/block.json | Bump Hero block version; simplify viewScript to file-only. |
| src/events-grid/block.json | Update block apiVersion to 3. |
| src/asu-careers/transformers/brassring-jobs.transformer.js | Remove asu-careers implementation. |
| src/asu-careers/style.scss | Remove asu-careers implementation. |
| src/asu-careers/save.js | Remove asu-careers implementation. |
| src/asu-careers/inspector.js | Remove asu-careers implementation. |
| src/asu-careers/index.js | Remove asu-careers implementation. |
| src/asu-careers/hooks/useBrassring.js | Remove asu-careers implementation. |
| src/asu-careers/frontend.js | Remove asu-careers implementation. |
| src/asu-careers/editor.scss | Remove asu-careers implementation. |
| src/asu-careers/edit.js | Remove asu-careers implementation. |
| src/asu-careers/components/AsuCareers.js | Remove asu-careers implementation. |
| src/asu-careers/block.json | Remove asu-careers block metadata/registration. |
| resources/utils/transformers/brassring-jobs.transformer.js | Remove shared brassring transformer (careers cleanup). |
| resources/utils/hooks/useBrassring.js | Remove shared brassring hook (careers cleanup). |
| resources/unity/component-careers/AsuCareers.js | Remove shared careers component. |
| README.md | Update repository layout docs to remove careers references. |
| package.json | Bump package version and dependency versions. |
| package-lock.json | Update lockfile for dependency changes. |
| build/wchm/block.json | Update built WCHM metadata to match source changes. |
| build/testimonial/index.js | Update built testimonial editor script. |
| build/testimonial/index.asset.php | Update built testimonial asset dependencies/version hash. |
| build/testimonial/frontend.js | Update built testimonial frontend script. |
| build/testimonial/frontend.asset.php | Update built testimonial frontend asset version hash. |
| build/testimonial/block.json | Update built testimonial metadata (apiVersion 3). |
| build/news-grid/index.asset.php | Update built news-grid asset version hash. |
| build/news-grid/frontend.asset.php | Update built news-grid frontend asset version hash. |
| build/news-grid/block.json | Update built news-grid metadata version. |
| build/image-gallery/index.asset.php | Remove built image-gallery assets (block removed). |
| build/image-gallery/frontend.asset.php | Remove built image-gallery assets (block removed). |
| build/image-gallery/block.json | Remove built image-gallery metadata (block removed). |
| build/hero/index.asset.php | Update built hero asset dependencies/version hash. |
| build/hero/hero-legacy.js | Add built legacy Hero chunk for dynamic hydration. |
| build/hero/frontend.js | Update built hero frontend to dynamic import legacy chunk. |
| build/hero/frontend.asset.php | Update built hero frontend asset dependencies/version hash. |
| build/hero/block.json | Update built hero metadata version/viewScript. |
| build/events-grid/index.asset.php | Update built events-grid asset version hash. |
| build/events-grid/frontend.asset.php | Update built events-grid frontend asset version hash. |
| build/events-grid/block.json | Update built events-grid metadata (apiVersion 3). |
| build/blocks-manifest.php | Update manifest to remove removed blocks and bump versions/apiVersion. |
| build/asu-careers/style-index.css | Remove built asu-careers assets (block removed). |
| build/asu-careers/index.css | Remove built asu-careers assets (block removed). |
| build/asu-careers/index.asset.php | Remove built asu-careers assets (block removed). |
| build/asu-careers/frontend.asset.php | Remove built asu-careers assets (block removed). |
| build/asu-careers/block.json | Remove built asu-careers metadata (block removed). |
| build/anchor-menu/index.asset.php | Update built anchor-menu asset dependencies/version hash. |
| .github/copilot-instructions.md | Update docs to remove careers references. |
Files not reviewed (3)
- build/hero/frontend.js: Language not supported
- build/hero/hero-legacy.js: Language not supported
- build/hero/index.js: Language not supported
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
v4.2
unity-blocks.php- remove unneededunity_blocks_scriptsfunctionasu-careersasu-careersresourcesevents-grid- api v3hero- import latest componentimage-galley- remove unused unfinished blocknews-grid- import latest componenttestimonial- api v3, display img in editor, img alt textwchm- import css file, rm scss fileblock-manifest.phpasu-unity-stackpackage.json-npm ilatest resourcesv4.3
Description
Users should notice no changes.
Existing heroes only change to static HTML if page is re-saved.
New heroes are created as static HTML.
Motivation and Context
#321 - requested for SEO
How has this been tested?
Locally
Screenshots (if appropriate):
Types of changes
Checklist: