search

MT-CTF / maps

All the maps used on the CTF server
31 stars 75 forks source link

Snowland map modification #363

Closed Themostrandom closed 1 month ago

Themostrandom commented 2 months ago

I've modified this map to remove several recurring problems: chests spawn outside borders, borders are incorrect (poorly defined map) and I've separated the 2 chest zones so you can add more chests if you wish.

Suggestion : https://discord.com/channels/447819017391046687/448136308108296213/1268549062936760320 and https://discord.com/channels/447819017391046687/448136308108296213/1147071682775891988 from @luis_mi and @alexander_jones6442

HobbitPower commented 2 months ago

@Themostrandom So again: what exactly does your PR change? There was a recent merge that prevents chests from spawning outside the outer barriers. In addition, there are now two chest zones and chests only spawn above ground. All (minor) problems have been fixed in my opinion.

Regarding your PR: I received an error message regarding potential barrier issues. I couldn't merge it in that form and since it doesn't seem to fix any bugs, a merge is out of the question anyway. Unless you give me another reason, as I said, the issues mentioned have already been resolved.

Btw, just in case: Should you have manually edited the conf file and changed the “2” to a “3”: This will not fix it. You will have to re-save the map and re-upload the files. If you have already done this: ignore the previous part.

SoulS33ker commented 2 months ago

@Themostrandom Perhaps you interpreted the message wrong https://discord.com/channels/447819017391046687/1268549062936760320/1268757642512302090 it's saying this bug was already fixed.https://github.com/MT-CTF/maps/pull/357 - merged

HobbitPower commented 1 month ago

As these mentioned issues have already been fixed in a previous PR (#357), I am closing this one.