Skip to content

Add support for user-defined JS Hooks#717

Merged
leandrocp merged 45 commits intomainfrom
apb/js-hooks
Jan 31, 2025
Merged

Add support for user-defined JS Hooks#717
leandrocp merged 45 commits intomainfrom
apb/js-hooks

Conversation

@APB9785
Copy link
Contributor

@APB9785 APB9785 commented Jan 3, 2025

Resolves #284

@APB9785 APB9785 self-assigned this Jan 3, 2025
@APB9785 APB9785 marked this pull request as ready for review January 7, 2025 17:59
@APB9785 APB9785 requested a review from leandrocp January 7, 2025 17:59
Copy link
Contributor

@leandrocp leandrocp left a comment

Choose a reason for hiding this comment

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

This is looking good! Just left some minor comments.

Copy link
Contributor

@leandrocp leandrocp left a comment

Choose a reason for hiding this comment

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

One more

end

@doc false
def validate_code(changeset) do
Copy link
Contributor

Choose a reason for hiding this comment

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

nice

@leandrocp leandrocp merged commit 86c9ffd into main Jan 31, 2025
3 checks passed
@leandrocp leandrocp deleted the apb/js-hooks branch January 31, 2025 15:34
@leandrocp
Copy link
Contributor

🎉

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.

Add support for user-defined client JS hooks

2 participants