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

Update a few things about "Zombie Bypass" + some other changes #169

Merged
merged 6 commits into from
Feb 24, 2025

Conversation

woisalreadytaken
Copy link
Contributor

@woisalreadytaken woisalreadytaken commented Feb 23, 2025

Addresses #4.

The goal of this is to make it clearer to people what made them be marked as "skipping playing as Infected."

Changes include:

  • Adding the name of the map the player was marked on, and how long ago to the message.
  • Making the feature toggleable through a convar, sm_szf_punish_avoiding_players, functionality is unchanged by default.
  • Adding missing + 1s to global client arrays.
  • Updating cookies to be applied when clients fully connect, and adding proper descriptions to them.

Non-English translations will need to be updated.
Translation formatting for singular time periods are commented out on purpose. They're not necessary in English, but other languages might need them.

Copy link
Member

@FortyTwoFortyTwo FortyTwoFortyTwo left a comment

Choose a reason for hiding this comment

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

For the whole singular time, I think it'd be better to instead say "80 minutes ago" instead of "an hour ago", to be a bit more accurate on a small number and not needing to manage each singular wordings

Copy link
Member

@FortyTwoFortyTwo FortyTwoFortyTwo left a comment

Choose a reason for hiding this comment

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

Need to use either %T or SetGlobalTransTarget in translations format to ensure client is getting the correct language for it.

Also Infected_ForceStart in other translations needs to be removed now that its format is different.

@FortyTwoFortyTwo FortyTwoFortyTwo merged commit 6154fea into redsunservers:master Feb 24, 2025
2 checks passed
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.

2 participants