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

fix some small bugs for gamemode fw #897

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

LightBlueCube
Copy link
Contributor

@LightBlueCube LightBlueCube commented Oct 19, 2024

see #868 for more infomation
i deleted my repo cuz i though its all finished, really sorry for that
my english doesnt very well, lmk if u need more infomation

@github-actions github-actions bot added needs testing Changes from the PR still need to be tested needs code review Changes from PR still need to be reviewed in code labels Oct 19, 2024
@@ -124,6 +124,10 @@ void function GamemodeFW_Init()
// _battery_port.gnut needs this
RegisterSignal( "BatteryActivate" )

// mp_thaw's intro spawnpoint have some problem, player can be stucked on the sky, so override it
if ( GetMapName() == "mp_thaw" )
SetSpawnpointGamemodeOverride( TEAM_DEATHMATCH )
Copy link
Contributor

Choose a reason for hiding this comment

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

Does FW have custom spawn logic usually? I'm not sure how i feel about getting rid of that spawn logic for a given map, if its just the intro sequence could we set it back after?

Copy link
Contributor

@Zanieon Zanieon left a comment

Choose a reason for hiding this comment

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

I haven't played Frontier War that much, but this code change is simple enough to understand and i don't see anything breaking if this gets merged. Good to go for me.

@Zanieon Zanieon removed the needs code review Changes from PR still need to be reviewed in code label Nov 25, 2024
@Zanieon Zanieon added READY TO MERGE This mergeable right now and removed needs testing Changes from the PR still need to be tested labels Nov 27, 2024
@Zanieon
Copy link
Contributor

Zanieon commented Nov 27, 2024

Tested and so far no compile errors or problems ingame.

@GeckoEidechse
Copy link
Member

Uh, what bugs exactly does this fix? Would be nice to have unrelated changes separated out ^^"

@Zanieon
Copy link
Contributor

Zanieon commented Nov 27, 2024

Uh, what bugs exactly does this fix? Would be nice to have unrelated changes separated out ^^"

  • Exoplanet spawnpoints being odd
  • Invalid entities on arrays
  • Harvesters couldn't be meleed
  • Turrets shield cap wasn't using their real max shield health property

@GeckoEidechse
Copy link
Member

Whoops, never saw your reply.... 🫠

Would it be possible to split up the changes? ^^"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
READY TO MERGE This mergeable right now
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

4 participants