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

[MIRROR] Fix: AI Hologram Copying Station Members #2174

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1186
Original PR: tgstation/tgstation#81701

About The Pull Request

This PR fixes a bug which disables the AI hologram's ability to copy the appearance of crew members. Fixes #78403

The bug was caused by developer error in PR #72725 when the typepath /datum/record/crew was used to access GLOB.manifest.locked, which is a list of /datum/record/locked. To fix the bug I replaced the erroneous typepath with /datum/record/locked.

Why It's Good For The Game

This PR fixes the broken feature that used to allow the AI to select from the crew records to set its hologram's appearance.

image image

Changelog

🆑 A.C.M.O.
fix: Fixed the AI hologram's ability to copy the appearance of crew members.
/:cl:

* Fix: AI Hologram Copying Station Members (#81701)

## About The Pull Request

This PR fixes a bug which disables the AI hologram's ability to copy the
appearance of crew members. Fixes #78403

The bug was caused by developer error in PR #72725 when the typepath
`/datum/record/crew` was used to access `GLOB.manifest.locked`, which is
a list of `/datum/record/locked`. To fix the bug I replaced the
erroneous typepath with `/datum/record/locked`.

## Why It's Good For The Game

This PR fixes the broken feature that used to allow the AI to select
from the crew records to set its hologram's appearance.


![image](https://github.com/tgstation/tgstation/assets/17753498/f6cdca47-0795-4b41-8c6c-922a38f597ee)
![image](https://github.com/tgstation/tgstation/assets/17753498/bfb49bf6-2842-405c-b88f-70b707f5d109)

## Changelog

:cl: A.C.M.O.
fix: Fixed the AI hologram's ability to copy the appearance of crew
members.
/:cl:

* Fix: AI Hologram Copying Station Members

---------

Co-authored-by: Dani Glore <[email protected]>
@Iajret Iajret merged commit 339c6bd into master Feb 29, 2024
24 checks passed
@Iajret Iajret deleted the upstream-mirror-1186 branch February 29, 2024 18:30
AnywayFarus added a commit that referenced this pull request Feb 29, 2024
Iajret pushed a commit that referenced this pull request Apr 24, 2024
* brimbeams are a hidden do after (#82856)

* brimbeams are a hidden do after

---------

Co-authored-by: Fikou <[email protected]>
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.

3 participants