Skip to content

feat: expose JImageProps for easier ts consumption - #798

Merged
GauBen merged 1 commit into
mainfrom
expose-jimageprops
Sep 22, 2026
Merged

GauBen merged 1 commit into
mainfrom
expose-jimageprops

Conversation

@GauBen

@GauBen GauBen commented Sep 21, 2026

Copy link
Copy Markdown
Member

Small quality of life improvement while working on luxe

@GauBen GauBen added the 🗒️ no-changelog No release notes or changelog entry should be added for this PR or Issue label Sep 21, 2026
Copilot AI lite review requested due to automatic review settings September 21, 2026 12:17
@github-actions

Copy link
Copy Markdown

🦜 Chachalog

No changelog entries detected. Learn more about Chachalog.

Create a new entry online or run npx chachalog@0.5.6 prompt to create a new entry locally.

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.

Copilot review overview

🟢 Approval recommended

No unresolved review comments remain.

Review effort: Lite
Findings: None

What changed in this PR

Exposes JImageProps as a public TypeScript type for library consumers.

Changes:

  • Defines the combined JImageProps type.
  • Re-exports it from the library entry point.
File Description
javascript-modules-library/​src/​index.ts Re-exports JImageProps.
javascript-modules-library/​src/​components/​JImage.tsx Defines the JImageProps type.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@pkg-pr-new

pkg-pr-new Bot commented Sep 21, 2026

Copy link
Copy Markdown

Open in StackBlitz

yarn add https://pkg.pr.new/@jahia/create-module@798.tgz
yarn add https://pkg.pr.new/@jahia/javascript-modules-library@798.tgz
yarn add https://pkg.pr.new/@jahia/vite-plugin@798.tgz

commit: 5de7fea

@GauBen
GauBen merged commit 9cd2bb7 into main Sep 22, 2026
51 of 52 checks passed
@GauBen
GauBen deleted the expose-jimageprops branch September 22, 2026 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🗒️ no-changelog No release notes or changelog entry should be added for this PR or Issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants