Skip to content

Various guestbook fixes#612

Merged
XeonMations merged 56 commits intoDarkPack13:masterfrom
XeonMations:guestbook-fixes
Mar 20, 2026
Merged

Various guestbook fixes#612
XeonMations merged 56 commits intoDarkPack13:masterfrom
XeonMations:guestbook-fixes

Conversation

@XeonMations
Copy link
Copy Markdown
Collaborator

@XeonMations XeonMations commented Jan 31, 2026

@FalloutFalcon
Copy link
Copy Markdown
Member

fixes #484 tho fera rework does as well lol.

XeonMations and others added 8 commits January 31, 2026 09:00
## About The Pull Request
## Why It's Good For The Game
## Changelog
:cl:
/:cl:
* Update city_outsides.dm

* Create SanFrancisco.dmm

* Create sanfrancisco.json

* AAA

* Update SanFrancisco.dmm

* Update SanFrancisco.dmm

* grass fixes + apartment door fixes

* sf is outdoors so rain now works on all z levels outside

* changes some npc beacons to prevent blob formations and places some cops around the map in places other than the police station

* fixes underwater fountains, trim issues, window issues, lighting issues, and changes the water in the lake to not be deep instantly, but have some shallow then deep

* fixes depot using all interior areas, adds a tunnel area to block rain from the pacific heights tunnel

* hospital elevator button

* fixes interior rain at under-bridge shop

* more shallow water by coasts

* lighting in the warrens, graveyard gate gap bandaid fix

* more grass fixes and adding placeholder landmarks for supply jobs pending whatever we do with supply/axegang

* Update decor.dm

* Update is_helpers.dm

* Revert "Update is_helpers.dm"

This reverts commit 18ce9a986b6c27fd2f1f398d230fe271847c7db6.

* Update trash.dm

* awaw

* Revert "awaw"

This reverts commit 14df78bb4d0bf7a1d02bf05e5380f1d0be215ac4.

* Update jobs.dm

* Update structure.dm

* never should have come here

* Update decor.dm

* fixes some decal mishaps

* Update SanFrancisco.dmm

* a

* umbra

* fix checks related to umbra

* g

* pending

* puts them back

* awawa

* Update __vtm.dm

---------

Co-authored-by: Marshmellow105 <118340622+Marshmellow105@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 8, 2026

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

@github-actions github-actions Bot added the Stale label Feb 8, 2026
@XeonMations XeonMations removed the Stale label Feb 8, 2026
Copy link
Copy Markdown
Member

@FalloutFalcon FalloutFalcon left a comment

Choose a reason for hiding this comment

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

Why is this pr trying to move guestbook/generic names down to the living level when anything outside of humans cant be expected to have a unique name to start.

Marshmellow105 and others added 3 commits February 9, 2026 00:02
* decal

* dark inner corner

* decal sin fixing, finds the two decals that crash your sdmm and removes them

* the last of the /obj/effect/turf_decal/siding sin fixes I think

* railing wood siding fixes for docks and gio mansion

* cat to stray spawner

* sproingus was crashing my game so I made them all subtypes then added more stray animals to the park

* wood sidings we have

* remove dirt sidings as grass does the job now in most places and add carpet decal colours

* carpet decal subtypening

* rough wood decals

* more wood rail fixes, removes depricated matrixes

* replaces the old method of underground wrapping (matrixes in a 1 deep layer) with rock filling akin to westfieldmall

* railing shit

* forgot these

* whatever
@XeonMations
Copy link
Copy Markdown
Collaborator Author

Why is this pr trying to move guestbook/generic names down to the living level when anything outside of humans cant be expected to have a unique name to start.

real_name is a variable on the base mob level, so anything can have a unique name to start. One use case is fera wolves.

@FalloutFalcon
Copy link
Copy Markdown
Member

Why is this pr trying to move guestbook/generic names down to the living level when anything outside of humans cant be expected to have a unique name to start.

real_name is a variable on the base mob level, so anything can have a unique name to start. One use case is fera wolves.

fera wolves are humans lmao. But what im saying is if multiple mobs have the same real_name (most non-humans) then if you try and guestbook one it guestbooks all of them.

@XeonMations
Copy link
Copy Markdown
Collaborator Author

Why is this pr trying to move guestbook/generic names down to the living level when anything outside of humans cant be expected to have a unique name to start.

real_name is a variable on the base mob level, so anything can have a unique name to start. One use case is fera wolves.

fera wolves are humans lmao. But what im saying is if multiple mobs have the same real_name (most non-humans) then if you try and guestbook one it guestbooks all of them.

the system only works if that mob has a mind.

also whAT DO YOU MEAN FERA WOLVES ARE HUMANS..................

@github-actions
Copy link
Copy Markdown
Contributor

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 both are true, and you have auto-staled anyway, you need to actively ask maintainers (by pinging them in the /tg/station Discord) to (re)review or merge your PR. If no maintainer responds to your request, you may wish to close this PR yourself while you seek maintainer comment, as you will later be able to reopen the PR yourself.

@github-actions
Copy link
Copy Markdown
Contributor

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 both are true, and you have auto-staled anyway, you need to actively ask maintainers (by pinging them in the /tg/station Discord) to (re)review or merge your PR. If no maintainer responds to your request, you may wish to close this PR yourself while you seek maintainer comment, as you will later be able to reopen the PR yourself.

@github-actions github-actions Bot removed the Map Edit label Mar 19, 2026
@XeonMations XeonMations marked this pull request as ready for review March 19, 2026 15:17
@XeonMations XeonMations requested a review from a team as a code owner March 19, 2026 15:17
@buffyuwu buffyuwu added this pull request to the merge queue Mar 20, 2026
@XeonMations XeonMations merged commit f20f41b into DarkPack13:master Mar 20, 2026
32 of 35 checks passed
darkpack13-api Bot added a commit that referenced this pull request Mar 20, 2026
github-actions Bot added a commit that referenced this pull request Mar 20, 2026
@XeonMations XeonMations deleted the guestbook-fixes branch March 21, 2026 16:58
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.

5 participants