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

/datum/map_adjustment for map change, job change for each station #11210

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

EvilDragonfiend
Copy link
Member

@EvilDragonfiend EvilDragonfiend commented Jul 18, 2024

About The Pull Request

  • /datum/map_adjustment for map change, job change for each station

map change code is miserable, also our code isn't really good to make a detailed adjustment for each station
this system can adjust station details like job slot, job access, having-lavaland-or-not, and so on.

NOTE

ideally, moving json features to this datum would be nice, but I am not interested to make a lot of change right now.

Why It's Good For The Game

each station needs a different set up, and this datum supports it.

Testing Photographs and Procedure

image

Station is orbitting Phobos, and we have no lavaland

image

job change works well

Changelog

🆑
code: /datum/map_adjustment is newly added - used to adjust map details.
tweak: Echo station no longer spawns Lavaland, and it will be the centre of the universe instead of Lavaland planet
/:cl:

@EvilDragonfiend
Copy link
Member Author

image

station now can be the centre of the universe

Copy link
Member

Choose a reason for hiding this comment

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

Why does radstation have no lavaland?

Copy link
Member Author

@EvilDragonfiend EvilDragonfiend Jul 18, 2024

Choose a reason for hiding this comment

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

rad and echo stations have ores to mine from itselves. That's it. but it won't be that necessary. I am honestly not sure if it would make any meaningful change.

Copy link
Member Author

@EvilDragonfiend EvilDragonfiend Jul 18, 2024

Choose a reason for hiding this comment

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

ahhhhhhhh, it was glow station, not rad station lmao

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants