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

Changes to animated emotes #184

Merged
merged 9 commits into from
Nov 14, 2024
Merged

Changes to animated emotes #184

merged 9 commits into from
Nov 14, 2024

Conversation

FluffMe
Copy link
Collaborator

@FluffMe FluffMe commented Nov 14, 2024

About the PR

Fiixed folders/namespaces
Made attribution point to a particular commit
Moved animation creation to initialization.
Stopped passing event around from a prototype.
Restored default behavior in raising an emote event for unmodified emotes.
Made emotes generic.
And other minor changes.

Why / Balance

I was bored and needed to code a bit.

Technical details

Commits are granular.

Media

N/A

Requirements

Breaking changes

None

Changelog
Not player facing

@FluffMe FluffMe requested a review from KeldWolf as a code owner November 14, 2024 07:14
@github-actions github-actions bot added the S: Needs Review Review is requested label Nov 14, 2024
Copy link

@KeldWolf KeldWolf left a comment

Choose a reason for hiding this comment

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

Thanks for this.

@KeldWolf KeldWolf merged commit 00b4eca into master Nov 14, 2024
12 checks passed
@FluffMe FluffMe deleted the animated-emotes-refactor branch November 14, 2024 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S: Needs Review Review is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants