Skip to content

Use spectate target vision mode in first person only #1104

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.

Sign up for GitHub

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

Jump to bottom
Merged
brysondev merged 1 commit into from
May 15, 2025
Merged

Use spectate target vision mode in first person only #1104

brysondev merged 1 commit into from
May 15, 2025

Conversation

@AdamTadeusz
Copy link
Contributor

@AdamTadeusz AdamTadeusz commented Apr 27, 2025 •
edited
Loading

Description

Use spectate target vision mode in first person only. Should also fix seeing other vision modes prematurely.

Toolchain

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

Linked Issues

@AdamTadeusz AdamTadeusz marked this pull request as draft April 27, 2025 19:03
@AdamTadeusz AdamTadeusz marked this pull request as ready for review April 27, 2025 19:08
@AdamTadeusz AdamTadeusz requested a review from a team May 13, 2025 10:23
brysondev
brysondev approved these changes May 15, 2025
View reviewed changes
Copy link
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.

finally

@brysondev brysondev merged commit 8146876 into NeotokyoRebuild:master May 15, 2025
7 checks passed
@AdamTadeusz AdamTadeusz deleted the 212_visionTargetNeoPlayer branch May 15, 2025 09:05
@DESTROYGIRL DESTROYGIRL mentioned this pull request May 30, 2025
Merged
This was referenced Sep 8, 2025
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DESTROYGIRL DESTROYGIRL DESTROYGIRL approved these changes

+1 more reviewer

@brysondev brysondev brysondev approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@AdamTadeusz @brysondev @DESTROYGIRL