Skip to content

Fix: Added null reference check to ClientInputSender script [MTT-6672]#880

Merged
Elfi0Kuhndorf merged 9 commits intodevelopfrom
fix-unavailable-abilities-null-reference
Mar 12, 2024
Merged

Fix: Added null reference check to ClientInputSender script [MTT-6672]#880
Elfi0Kuhndorf merged 9 commits intodevelopfrom
fix-unavailable-abilities-null-reference

Conversation

@Elfi0Kuhndorf
Copy link
Contributor

@Elfi0Kuhndorf Elfi0Kuhndorf commented Feb 29, 2024

Description

Added null reference checks to ClientInputSender script in order to remove null reference exemption when using an unavailable ability.

Validation:

  1. Open Boss room and press play
  2. Start a game and choose Tank or Mage class in character selection
  3. Press 3 on keyboard (notice in skill UI no ability 3 available for these classes)
  4. Notice no null reference coming up

Issue Number(s)

MTT-6672

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
  • An Index entry has been added in readme.md if applicable

@Elfi0Kuhndorf Elfi0Kuhndorf added the 2-Easy This PR is trivial and can be reviewed quickly label Feb 29, 2024
@Elfi0Kuhndorf Elfi0Kuhndorf requested a review from a team as a code owner February 29, 2024 13:20
@Elfi0Kuhndorf Elfi0Kuhndorf self-assigned this Feb 29, 2024
@Elfi0Kuhndorf Elfi0Kuhndorf requested a review from a team February 29, 2024 14:46
fernando-cortez
fernando-cortez previously approved these changes Mar 1, 2024
View reviewed changes
Copy link
Contributor

@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.

Approving, but marking this as On Hold until this PR lands first.

@fernando-cortez fernando-cortez added 4-On Hold PR can't proceed because it's blocked or is otherwise waiting on something. and removed 4-On Hold PR can't proceed because it's blocked or is otherwise waiting on something. labels Mar 1, 2024
RikuTheFuffs
RikuTheFuffs requested changes Mar 6, 2024
View reviewed changes
Copy link
Contributor

@RikuTheFuffs RikuTheFuffs left a comment

Choose a reason for hiding this comment

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

lots of unneeded formatting changes that reduce readability, can you please remove them?

@unity-cla-assistant
Copy link

unity-cla-assistant commented Mar 7, 2024 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

RikuTheFuffs
RikuTheFuffs previously approved these changes Mar 7, 2024
View reviewed changes
Copy link
Contributor

@RikuTheFuffs RikuTheFuffs left a comment

Choose a reason for hiding this comment

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

looks good now!

@Elfi0Kuhndorf
Copy link
Contributor Author

Elfi0Kuhndorf commented Mar 11, 2024 •
edited
Loading

looks good now!

Thanks! I would need an approving review (due to develop being merged into it I think), then I can merge it :)

RikuTheFuffs
RikuTheFuffs previously approved these changes Mar 11, 2024
View reviewed changes
fernando-cortez
fernando-cortez reviewed Mar 11, 2024
View reviewed changes
@Elfi0Kuhndorf @fernando-cortez
Update CHANGELOG.md
315a172 
Co-authored-by: Fernando Cortez <[email protected]>
fernando-cortez
fernando-cortez approved these changes Mar 12, 2024
View reviewed changes
Copy link
Contributor

@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.

Looks good!

@Elfi0Kuhndorf Elfi0Kuhndorf merged commit 384cfc7 into develop Mar 12, 2024
@Elfi0Kuhndorf Elfi0Kuhndorf deleted the fix-unavailable-abilities-null-reference branch March 12, 2024 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@RikuTheFuffs RikuTheFuffs RikuTheFuffs left review comments

+1 more reviewer

@fernando-cortez fernando-cortez fernando-cortez approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@fernando-cortez fernando-cortez

Labels

2-Easy This PR is trivial and can be reviewed quickly

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Elfi0Kuhndorf @unity-cla-assistant @RikuTheFuffs @fernando-cortez