Skip to content

[Ready] Re-Add Meta Station, Z-Level Sizing + Hole in Map Fix.#450

Open
Scout-Kreed wants to merge 11 commits intoTheSwain:masterfrom
Scout-Kreed:Meta
Open

[Ready] Re-Add Meta Station, Z-Level Sizing + Hole in Map Fix.#450
Scout-Kreed wants to merge 11 commits intoTheSwain:masterfrom
Scout-Kreed:Meta

Conversation

@Scout-Kreed
Copy link

@Scout-Kreed Scout-Kreed commented Aug 7, 2020

About The Pull Request

It was noticed By LemonInTheDark that The Z-level for Meta Station showed up as x-255 and y-526 instead of x-255 and y-255, So i went and moved stuff back to where it should be so that there no longer was a Hole in arrivals and then had to do some adjustments before decreasing the Map Size, the Adjustments i made, was Giving Viro a West Wall again, putting up the Grille defenses Around Port Bow Solars, and Adding the Tiles to Cargo maints.

Why It's Good For The Game

These Changes Allows us to use Meta Station as a Map again and to be able to play on it,
once it is re added to the Map rotation of course.

Changelog

🆑

tweak: tweaked a few things
ix: fixed a few things
fix: fixed metastation issues

/:cl:

Fix Z-Level Size Issues(where the y was 2 times as large as its supposed to be) and put all of the turf and stuff back into where they belong.
@LemonInTheDark
Copy link

closes #448

@Scout-Kreed
Copy link
Author

Edit: Forgot To Move the Areas along with the Turf/Obj, it is now Fixed.

@Helianthus00
Copy link

Mapmerge2 still isn't working it seems. Check that you installed it properly.

@Samuuu3 Samuuu3 added Fix Fixes an in game bug or issue Mapping Map changes labels Aug 16, 2020
@Scout-Kreed Scout-Kreed changed the title Meta Station, Z-Level Size Issues + Hole in Map Fix. [Ready] Meta Station, Z-Level Size Issues + Hole in Map Fix. Aug 18, 2020
@Scout-Kreed Scout-Kreed changed the title [Ready] Meta Station, Z-Level Size Issues + Hole in Map Fix. [Ready] Re-Add Meta Station, Z-Level Sizing + Hole in Map Fix. Aug 18, 2020
@github-actions
Copy link

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

Copy link

@EgorDinamit EgorDinamit left a comment

Choose a reason for hiding this comment

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

Now that areas are fixed it should be okay to merge, but what about "dmm before" file?
I don't think you really need it in the PR.

@Scout-Kreed
Copy link
Author

Adding the single Missing wire in Cargo/Arrivals Maintenance,
Put a Christmas Present in all head lockers including Warden and QM
Renamed and changed description of the Ruppie in Chapel

Comment on lines +13735 to +13739
/obj/item/a_gift/anything,
/obj/item/a_gift/anything,
/obj/item/a_gift/anything,
/obj/item/a_gift/anything,
/obj/item/a_gift/anything,

Choose a reason for hiding this comment

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

You should really, really remove all of the anything in the /obj/item/a_gift/anything as those can spawn anything, from playing cards to an instakill rifle.

Choose a reason for hiding this comment

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

Not just those I highlighted here, but the ones that are all around the station. Removing the anything part turns them in presents that only contain toys in them.

@JohnFulpWillard
Copy link

i like how this is still being TM'ed while it's closed

@Scout-Kreed Scout-Kreed reopened this Oct 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fix Fixes an in game bug or issue Mapping Map changes Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants