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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds logging for crafting #63995

Closed
wants to merge 1 commit into from
Closed

Adds logging for crafting #63995

wants to merge 1 commit into from

Conversation

itseasytosee
Copy link
Contributor

@itseasytosee itseasytosee commented Jan 11, 2022

About The Pull Request

Adds logging for crafting
Well technically there was limited logging before this change, but only via the investigate_log command used during an ongoing round. This adds full game logging to crafting in its own separate txt file. This is my first time messing around with logs so if you think I fucked up please tell me (might warrant a testmerge).

Why It's Good For The Game

This was coded by me entirely for the purpose of being able to collect data on how often pipeguns are crafted in order to justify an argument to oranges.

image
image
link to thread: https://tgstation13.org/phpBB/viewtopic.php?f=10&t=30653

This change will help me collect the data I need, denying this change is a denial of investigative journalism. It will also help others gather the data they need about the use of crafting recipes in the future.

Changelog

馃啈 itseasytosee
server: Adds logging for crafting
/:cl:

@tgstation-server tgstation-server added the Config Update Time to bother the headadmins for three months to get your config applied label Jan 11, 2022
@Mothblocks
Copy link
Member

dude i #63178

@Mothblocks
Copy link
Member

Mothblocks commented Jan 11, 2022

they exist in crafting.html, ask MSO (or I think edit the tgstation13.org PHP repo?) to put them on parsed-logs

@Mothblocks Mothblocks closed this Jan 11, 2022
@Mothblocks
Copy link
Member

you wouldnt have been able to easily access this anyway because you made it a separate file--only whitelisted files go through to parsed-logs

@LemonInTheDark
Copy link
Member

LemonInTheDark commented Jan 11, 2022

they exist in crafting.html, ask MSO (or I think edit the tgstation13.org PHP repo?) to put them on parsed-logs

you need to ask mso. He doesn't update that repo because he's lazy a closed source shill

@itseasytosee
Copy link
Contributor Author

Cheers for the tip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Config Update Time to bother the headadmins for three months to get your config applied
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants