Skip to content
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

Updated Hooks to v12 #2934

Merged
merged 1 commit into from
Nov 21, 2024
Merged

Updated Hooks to v12 #2934

merged 1 commit into from
Nov 21, 2024

Conversation

JPMeehan
Copy link
Collaborator

@JPMeehan JPMeehan commented Nov 21, 2024

Closes #2573

@JPMeehan
Copy link
Collaborator Author

Foundry hasn't officially documented the appv2 hooks, we should probably look into those and write our own? Maybe steal from v13? Can be a separate update

@JPMeehan JPMeehan marked this pull request as ready for review November 21, 2024 17:17
@JPMeehan JPMeehan requested a review from a team as a code owner November 21, 2024 17:17
@JPMeehan JPMeehan self-assigned this Nov 21, 2024
@LukeAbby
Copy link
Collaborator

Are the AppV2 hooks ticketed? If not could you make sure they are?

* @param group - The group being deconstructed
*/
type TearDownGroup<
G extends ReturnType<typeof CanvasGroupMixin> = ReturnType<typeof CanvasGroupMixin<CanvasGroupMixin.BaseClass>>,
Copy link
Collaborator

Choose a reason for hiding this comment

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

It'd also be useful here.

* @param group - The group being drawn
*/
type DrawGroup<
G extends ReturnType<typeof CanvasGroupMixin> = ReturnType<typeof CanvasGroupMixin<CanvasGroupMixin.BaseClass>>,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe this suggests we should have an interface and possible namespace named CanvasGroup available in the global scope.

I kind of like the idea of being able to write CanvasGroup.Any.

On that note I'm pretty sure ReturnType<typeof CanvasGroupMixin> is the same thing as ReturnType<typeof CanvasGroupMixin<CanvasGroupMixin.BaseClass>> because when generics are left off they seem to default to their generic constraint.

So I think this could be CanvasGroup.Any = CanvasGroup.Any or possibly CanvasGroupMixin.Any.

Copy link
Collaborator Author

@JPMeehan JPMeehan Nov 21, 2024

Choose a reason for hiding this comment

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

I'll write that up as a ticket?

EDIT: Done #2937

@LukeAbby LukeAbby merged commit 2444851 into main Nov 21, 2024
2 of 6 checks passed
@LukeAbby LukeAbby deleted the 2573-hooks branch November 21, 2024 18:36
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.

Update hooks.mjs
2 participants