Skip to content

Fix initial player tposing#570

Merged
AdamTadeusz merged 1 commit intoNeotokyoRebuild:masterfrom
AdamTadeusz:032_initialPlayerTposing
Sep 13, 2024
Merged

Fix initial player tposing#570
AdamTadeusz merged 1 commit intoNeotokyoRebuild:masterfrom
AdamTadeusz:032_initialPlayerTposing

Conversation

@AdamTadeusz
Copy link
Copy Markdown
Contributor

Description

Replace weapon_switch with ClientCommand

Toolchain

  • Windows MSVC VS2022
  • Linux GCC Distro Native [Specify distro + GCC version]
  • Linux GCC 10 Sniper 3.0

Linked Issues

@AdamTadeusz AdamTadeusz requested a review from a team September 9, 2024 13:52
Copy link
Copy Markdown
Contributor

@brysondev brysondev left a comment

Choose a reason for hiding this comment

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

lgtm. It was stated on discord we might wanna neotodo and mark Weapon_Switch as borked but as long as it(this fix) works. 👍

@AdamTadeusz AdamTadeusz merged commit d209d44 into NeotokyoRebuild:master Sep 13, 2024
Rainyan added a commit to Rainyan/rebuild that referenced this pull request Sep 17, 2024
@AdamTadeusz AdamTadeusz deleted the 032_initialPlayerTposing branch September 18, 2024 07:32
Rainyan added a commit to Rainyan/rebuild that referenced this pull request Sep 18, 2024
Decouple `UpdateMuzzleFlashProperties` from BasePlayer, and move it to
CNEO_Player shared implementation.

For client side, use a `player_spawn` event listener for figuring out
when the player spawns, instead of trying to capture it in-band from a
networked "slot" command to the user. This fixes the bug that NeotokyoRebuild#570
attempted to address, (which will probably be reverted by NeotokyoRebuild#598, assuming
it's accepted), and will remove the dependency to client side weapon
switch for triggering `UpdateMuzzleFlashProperties` on player spawn.
Rainyan added a commit that referenced this pull request Sep 19, 2024
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.

Player TPosing

3 participants