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

Prescription HUD Construction Crash Fix #449

Merged

Conversation

Cprn-Sprkl
Copy link
Contributor

Added the tag "GlassesNearsight" to the tag list in tags.yml and tagged the glasses prototype with it. The crafting graph for prescription HUDs expects this tag.

Description

The construction graph for the prescription versions of the sec and med HUDs requires an item with the tag "GlassesNearsight". This tag was not on the glasses prototype nor in the list of tags. When attempting to build a prescription HUD with the appropriate HUD, cables, and glasses, but not the glue, the local server would encounter a hard error that closed it. Other combinations were not tested.


TODO

  • Check if this crash can be replicated on others' builds.

Changelog

🆑 sprkl

  • fix: Fixed a crash that occurred while trying to build prescription huds.

Added the tag "GlassesNearsight" to the tag list in tags.yml and tagged the glasses prototype with it. The crafting graph for prescription huds expects this tag.
@github-actions github-actions bot added Status: Needs Review Someone please review this Changes: YML Changes any yml files labels Jan 2, 2025
@Cprn-Sprkl
Copy link
Contributor Author

I'll be unavailable for the next few hours so I figured I'd just submit the fix I have in case this a problem for the main server and not just an issue with my local build.

@Mnemotechnician
Copy link
Collaborator

Test fail should be unrelated, but I'm not sure.

@Fansana Fansana merged commit 5c4e4c3 into Fansana:master Jan 3, 2025
16 checks passed
Floof-Station-Bot added a commit that referenced this pull request Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: YML Changes any yml files Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants