Skip to content

beforeSource, afterSource, and afterAll hooks #343

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

jhadobe
Copy link
Contributor

@jhadobe jhadobe commented Jul 21, 2025

This PR adds descriptions of new hooks types and provides example use cases.

@jhadobe jhadobe requested a review from kmaschi July 21, 2025 22:03
@jhadobe jhadobe self-assigned this Jul 21, 2025
@jhadobe jhadobe added the major-update Significant original updates to existing content label Jul 21, 2025
@github-project-automation github-project-automation bot moved this to 📋 Needs Review in Commerce - Pull Requests Jul 21, 2025
@jhadobe jhadobe requested a review from dani97 July 21, 2025 22:07
```

```ts
interface BeforeSourceTransformObject {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Did this interface come from somewhere or was AI generated?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kmaschi these were inferred by AI, let me know if they should be removed.

```

```ts
interface AfterSourceTransformObject {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Similar question about the source of this interface.

@jhadobe jhadobe requested a review from keharper July 23, 2025 20:26
@github-project-automation github-project-automation bot moved this from 📋 Needs Review to 👍 Approved in Commerce - Pull Requests Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major-update Significant original updates to existing content
Projects
Status: 👍 Approved
Development

Successfully merging this pull request may close these issues.

3 participants