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

Conversation

@adamsong
Copy link
Contributor

@adamsong adamsong commented Feb 20, 2022

Document the changes in your pull request

Refactors VV to have two new procs, vv_do_topic and vv_get_header. vv_do_topic is called by the dropdowns, and vv_get_header allows adding to the vv header.

Also added a hiddenprints viewer in VV, because I liked it while stealing the rest of the code.

Based off tg code, mainly tgstation/tgstation#45217

Changelog

🆑
rscadd: Added show hiddenprints to VV
/:cl:

@Yogbot-13 Yogbot-13 added DME Edit This PR affects the yogstation.DME file Feature This adds new content to the game labels Feb 20, 2022
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.

theres a bug, fix it for merge

@github-actions github-actions bot added the Admin This PR affects administrators label Feb 20, 2022
@JamieD1
Copy link
Contributor

JamieD1 commented Feb 25, 2022

If this is a port please credit where from.

Yogbot-13 added a commit that referenced this pull request Mar 6, 2022
Yogbot-13 added a commit that referenced this pull request Mar 6, 2022
@adamsong
Copy link
Contributor Author

adamsong commented Mar 6, 2022

Experienced issues during test merge, will update if fixed

@adamsong adamsong added the DO NOT MERGE Should not be merged without express approval from a Head Dev label Mar 6, 2022
Yogbot-13 added a commit that referenced this pull request Mar 6, 2022
Yogbot-13 added a commit that referenced this pull request Mar 10, 2022
@ToasterBiome ToasterBiome added the TESTMERGED This PR is currently live in a test merge. label Mar 10, 2022
Yogbot-13 added a commit that referenced this pull request Mar 10, 2022
Yogbot-13 added a commit that referenced this pull request Mar 10, 2022
Yogbot-13 added a commit that referenced this pull request Mar 10, 2022
@JamieD1 JamieD1 merged commit e5a1515 into master Mar 13, 2022
@JamieD1 JamieD1 deleted the better-vv branch March 13, 2022 11:22
Yogbot-13 added a commit that referenced this pull request Mar 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Admin This PR affects administrators DME Edit This PR affects the yogstation.DME file DO NOT MERGE Should not be merged without express approval from a Head Dev Feature This adds new content to the game TESTMERGED This PR is currently live in a test merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants