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

Conversation

@adamsong
Copy link
Contributor

Github documenting your Pull Request

Pressing the button on the comms console will trigger a command report, and print a temp ID. After 50 seconds, the ID will start to smolder. 10 seconds later, it will catch fire and turn to ash. Don't be holding it when this happens.

The ID has the same access as the spare, without heads, or id change access.

Changelog

🆑
rscadd: Added a temp spare heads can print from the comms console.
/:cl:

@adamsong adamsong requested a review from a team August 25, 2021 22:00
@Yogbot-13 Yogbot-13 added Feature This adds new content to the game Javascript This PR contains javascript code. labels Aug 25, 2021
@github-actions github-actions bot added Config Config files need to be changed on the host for this to work tgui This PR affects TGUI labels Aug 25, 2021
@adamsong adamsong added the DO NOT MERGE Should not be merged without express approval from a Head Dev label Aug 25, 2021
@Cartlord
Copy link
Contributor

but why

@Anvilman6
Copy link
Contributor

Maybe just make it fizzle out
rather than start a fire

Would be neat if you could examine it to check how much time it has left

@adamsong
Copy link
Contributor Author

rather than start a fire

You get 10 seconds warning, past that, its on you

@adamsong adamsong removed the DO NOT MERGE Should not be merged without express approval from a Head Dev label Aug 26, 2021
Copy link
Contributor

@Bibby0110 Bibby0110 left a comment

Choose a reason for hiding this comment

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

Authentication check

@JamieD1
Copy link
Contributor

JamieD1 commented Aug 30, 2021

@adamsong

Copy link
Contributor

@Bibby0110 Bibby0110 left a comment

Choose a reason for hiding this comment

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

Replace your authentication check with authenticated_as_silicon_or_head
You’ll have to create the proc but that shouldn’t be hard

@adamsong adamsong requested a review from Bibby0110 September 7, 2021 21:58
@JamieD1 JamieD1 merged commit c735e6b into master Sep 9, 2021
@JamieD1 JamieD1 deleted the tmp-spare branch September 9, 2021 23:43
Yogbot-13 added a commit that referenced this pull request Sep 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Config Config files need to be changed on the host for this to work Feature This adds new content to the game Javascript This PR contains javascript code. tgui This PR affects TGUI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants