Skip to content

update modal to pull cnhsa eligibility guidelines from the site - #14

Merged
kjroelke merged 1 commit into
mainfrom
cnhsa-compat
Jul 24, 2026
Merged

update modal to pull cnhsa eligibility guidelines from the site#14
kjroelke merged 1 commit into
mainfrom
cnhsa-compat

Conversation

@kjroelke

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Modal block’s CNHSA guidelines flow so the modal loads eligibility guidelines content directly from the CNHSA site, and bumps related version metadata for a new release.

Changes:

  • Switch CNHSA guidelines fetch to https://www.cnhsa.com/wp-json/cnhsa/v1/eligibility-guidelines and use data.rendered as the HTML payload.
  • Bump the Modal block version to 1.1.1 and document the change in the block changelog.
  • Bump plugin/package versions (and update lockfiles/dependency versions) for the release.

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/blocks/modal/view.ts Changes the guidelines fetch URL/response field and adds a TS return type annotation.
src/blocks/modal/CHANGELOG.md Adds v1.1.1 entry documenting the guidelines source fix.
src/blocks/modal/block.json Bumps Modal block version to 1.1.1.
package.json Bumps package version to 2.7.0 and updates WP-related devDependencies.
composer.lock Updates locked dev tooling packages (e.g., stubs/WPCS) consistent with dependency updates.
composer.json Bumps Composer project version to 2.7.0.
cno-plugin-cno-blocks.php Bumps plugin header version to 2.7.0 and updates “Tested up to”.

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

Comment thread src/blocks/modal/view.ts
Comment thread src/blocks/modal/view.ts
Comment thread src/blocks/modal/view.ts
@kjroelke
kjroelke merged commit 32a569f into main Jul 24, 2026
5 checks passed
@kjroelke
kjroelke deleted the cnhsa-compat branch July 24, 2026 20:48
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.

3 participants