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

Feature: Art and Sound Pass for Vandal Imp Tossed Attack [MTT-2732] #689

Merged
merged 11 commits into from
Aug 16, 2022

Conversation

jilfranco-unity
Copy link
Contributor

@jilfranco-unity jilfranco-unity commented Jun 28, 2022

Description

An art and sound pass for the Vandal Imp tossed attack! Created a bunch of fancy new VFX and worked with Fernando to implement them for the tossed attack. Also cut up our existing tossed attack sound effect to work with this version.

(cherry picked from commit baef02ba6280f7d48eba6e8305b5af24f7340361): These changes were originally in a branch based off of the Network Rigid Body branch that has since been merged into develop, so to keep things clean, I've cherry picked them into this branch which is based off of develop.

Here's how it looks now :)

TossedAttack.mp4

Issue Number(s)

Jira ticket [MTT-2732]

Contribution checklist

  • Tests have been added for boss room and/or utilities pack
  • Release notes have been added to the project changelog file and/or package changelog file
  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • JIRA ticket ID is in the PR title or at least one commit message
  • Include the ticket ID number within the body message of the PR to create a hyperlink

@jilfranco-unity jilfranco-unity added 1-Needs Review PR needs attention from the assignee and reviewers type:feature New feature, request or improvement labels Jun 28, 2022
@SamuelBellomo
Copy link
Contributor

Contribution check list says there are tests added?

CHANGELOG.md Outdated Show resolved Hide resolved
jilfranco-unity and others added 2 commits June 30, 2022 16:41
Co-authored-by: Sam Bellomo <71790295+SamuelBellomo@users.noreply.github.com>
@jilfranco-unity
Copy link
Contributor Author

Contribution check list says there are tests added?

My b, didn't mean to check this! :P

Copy link
Contributor

@LPLafontaineB LPLafontaineB left a comment

Choose a reason for hiding this comment

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

Found a bug while testing. Late-joining clients see all previous bombs on the ground as they were before exploding
image

@LPLafontaineB LPLafontaineB added the 2-Reviewed with Comments PR requires owner's attention label Aug 1, 2022
… the tossed attack explosion effect that made late join clients see explosion effects everywhere
… all geometry and to instead be soft particles that blend into the ground to fix the look of them clipping into the ground
@jilfranco-unity jilfranco-unity removed the 2-Reviewed with Comments PR requires owner's attention label Aug 5, 2022
@LPLafontaineB LPLafontaineB added 2-One More Review One review in, one to go and removed 1-Needs Review PR needs attention from the assignee and reviewers labels Aug 5, 2022
LPLafontaineB
LPLafontaineB previously approved these changes Aug 5, 2022
fernando-cortez
fernando-cortez previously approved these changes Aug 5, 2022
Copy link
Collaborator

@fernando-cortez fernando-cortez left a comment

Choose a reason for hiding this comment

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

Only nitpick is to place the little rascal somewhere in between the starting area and the boss? Maybe right before the door? He just screams mini-boss to me 😃

…he surrounding imps a bit so that he fit in better
@jilfranco-unity
Copy link
Contributor Author

jilfranco-unity commented Aug 10, 2022

Moved the vandal imp to be closer to the boss door! I also rearranged the surrounding imps a bit so that he fit in better :)

image

@fernando-cortez fernando-cortez added 3-Good to Merge and removed 2-One More Review One review in, one to go labels Aug 15, 2022
@jilfranco-unity jilfranco-unity merged commit 8ef7941 into develop Aug 16, 2022
@jilfranco-unity jilfranco-unity deleted the feature/vandal-imp-tossed-attack-art branch August 16, 2022 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3-Good to Merge type:feature New feature, request or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants