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

[MIRROR] Fixes a hard del in character directory #4961

Merged
merged 2 commits into from
Aug 24, 2024
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Original PR: NovaSector/NovaSector#4067

About The Pull Request

Tin, let's reduce the amount of hard dels.

How This Contributes To The Nova Sector Roleplay Experience

Better resource management, could save a few seconds worth of lag on average per round in total.

Proof of Testing

Screenshots/Videos

image

Changelog

Nothing player facing

vinylspiders and others added 2 commits August 24, 2024 20:15
* Fixes hard del in character directory

* Let's debug this other harddel while we are at it

* Some more safeties

* Enable live reftracking

* Update search_object.dm

* Update search_object.dm

* Revert "Update search_object.dm"

This reverts commit c0cb069a7aad817bcd21ca4568c1b15d3ec85138.

* Revert "Enable live reftracking"

This reverts commit 6233eda4d4092833478cd5e1f4e2c94ef3a92cae.

* Revert "Some more safeties"

This reverts commit dade17d4b497d77157aa83b6898b88df5af580bd.

* Update search_object.dm

* Hard del fix?

* Lingering harddel fix?

* Update character_directory.dm

* Update character_directory.dm

* More robust taur code

* Revert "More robust taur code"

This reverts commit 628c847eca751d1f22712fb7f3be79c0552d6304.

* Hopefully clears up this hard del once and for all

* Update character_directory.dm

* Update character_directory.dm
@FeenieRU FeenieRU merged commit 5faa76e into master Aug 24, 2024
29 checks passed
@Iajret Iajret deleted the upstream-mirror-4067 branch September 1, 2024 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants