Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@maxion12345
Copy link
Contributor

If this is your first PR, or not, take the time to read our CONTRIBUTING.md file! You can see it here: https://github.com/yogstation13/Yogstation/blob/master/.github/CONTRIBUTING.md

Github documenting your Pull Request

i change something

Wiki Documentation

no

Changelog

🆑
bugfix: Spare cabinet uses normal captain access now
/:cl:

@maxion12345 maxion12345 requested a review from a team as a code owner August 9, 2021 02:30
@Yogbot-13 Yogbot-13 added Fix This fixes an issue. Please link issues in fix PRs Mapping Will result in a change to a map. labels Aug 9, 2021
@adamsong
Copy link
Contributor

adamsong commented Aug 9, 2021

req_access = list(20)

Are you sure this works in map files? req_access_text exists presumably because that doesn't work

@maxion12345
Copy link
Contributor Author

req_access = list(20)

Are you sure this works in map files? req_access_text exists presumably because that doesn't work

Its the same access as the captain's door (20)

@adamsong
Copy link
Contributor

adamsong commented Aug 9, 2021

But cap's door uses the req_access_text variable, I'm not sure this can initialize a list

@patpol4
Copy link
Contributor

patpol4 commented Aug 9, 2021

you need to use the text one

@patpol4
Copy link
Contributor

patpol4 commented Aug 9, 2021

i only skimmed this but #11984 seems to do the same thing but with fireaxe too

@JamieD1
Copy link
Contributor

JamieD1 commented Aug 9, 2021

Bad, do it in code initialize instead.

@JamieD1
Copy link
Contributor

JamieD1 commented Aug 9, 2021

Code change better

@JamieD1 JamieD1 closed this Aug 9, 2021
@maxion12345 maxion12345 deleted the bruh branch October 16, 2021 17:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Fix This fixes an issue. Please link issues in fix PRs Mapping Will result in a change to a map.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants