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

Hard delete fixes for predator landmarks and van if their area changes #3883

Merged
merged 1 commit into from
Jul 15, 2023

Conversation

Drulikar
Copy link
Contributor

@Drulikar Drulikar commented Jul 13, 2023

About the pull request

This PR applies the fixes previously suggested for #3671. That PR was originally replacing much of the colony in a nightmare insert and it exposed how predator landmarks do not handle their area being changed and an issue with the van being replaced. The description made for these landmarks is based on some area information and that is used to index them. This PR makes it so the description of the landmark is saved so it can still be found in the list even if the area changes.

Explain why it's good for the game

Less hard deletes and more futureproofing.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑 Drathek, Steelpoint
fix: Fixed possible hardeletes for predator landmarks and vehicles. Predator teleporation descriptions now do not change if the area is altered at runtime so they can still be found correctly.
/:cl:

@Drulikar Drulikar added the Hard Deletes Finally doing something about the performance problem. label Jul 13, 2023
@github-actions github-actions bot added the Fix Fix one bug, make ten more label Jul 13, 2023
@harryob harryob enabled auto-merge July 15, 2023 04:38
@harryob harryob added this pull request to the merge queue Jul 15, 2023
Merged via the queue into cmss13-devs:master with commit 9b3452c Jul 15, 2023
30 checks passed
github-actions bot added a commit that referenced this pull request Jul 15, 2023
cm13-github added a commit that referenced this pull request Jul 15, 2023
@Drulikar Drulikar deleted the Revive_Hard_delete_fixes branch August 6, 2023 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more Hard Deletes Finally doing something about the performance problem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants