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

Added C++/Lua hooks for AActor::Endplay #769

Merged
merged 1 commit into from
Feb 5, 2025
Merged

Added C++/Lua hooks for AActor::Endplay #769

merged 1 commit into from
Feb 5, 2025

Conversation

UE4SS
Copy link
Collaborator

@UE4SS UE4SS commented Jan 31, 2025

Description

This PR adds hooks accessible via both Lua and C++ for AActor::EndPlay, in the same way that we have AActor::BeginPlay.

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Make a Lua mod, register an EndPlay hook that prints the object name.
Start a game, load some save files, and quit.
Open UE4SS.log and make sure the output is correct.

Checklist

  • I have added the necessary description of this PR to the changelog, and I have followed the same format as other entries.
  • Any dependent changes have been merged and published in downstream modules.

Additional context

This may not be merged until #768 is merged.
If #768 doesn't get merged, this PR must be rebased onto main.

@UE4SS
Copy link
Collaborator Author

UE4SS commented Jan 31, 2025

Note that CI fails only because the branch is based on a commit that doesn't have the latest Unreal module commit.
It will pass when #768 is merged, and/or this branch is rebased onto main.

Copy link
Contributor

github-actions bot commented Jan 31, 2025

No description provided.

@CookiePLMonster
Copy link

Confirmed working:
image

@narknon narknon merged commit 7b399d4 into main Feb 5, 2025
1 of 13 checks passed
@narknon narknon deleted the endplay-hook branch February 5, 2025 03:59
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