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

Update entity weight #221

Merged
merged 1 commit into from
Sep 30, 2024
Merged

Update entity weight #221

merged 1 commit into from
Sep 30, 2024

Conversation

jt-traub
Copy link
Contributor

@jt-traub jt-traub commented Sep 29, 2024

The intent was that the entities are not transportable via magic, via transport or via the underworld, so that they cannot be moved around except via overland/ships. However, I had mistakenly set their weight such that high level portal lore and teleport could be used to move them. The easiest fix was to up their weight so that this becomes impossible, though the side effect is they now require more ships to transport. This shouldn't be a problem however in the scheme of things as the larger ships can handle them in a reasonable number and even the smaller ships can do it, with enough of them in the fleet.

The intent was that the entities are not transportable via magic, via
transport or via the underworld, so that they cannot be moved around
except via overland/ships. However, I had mistakenly set their weight
such that high level portal lore and teleport could be used to move
them. The easiest fix was to up their weight so that this becomes
impossible, though the side effect is they now require more ships to
transport. This shouldn't be a problem however in the scheme of things
as they larger ships can handle them in a reasonable number and even
the smaller ships can do it, with enough of them in the fleet.
@valdisz
Copy link
Contributor

valdisz commented Sep 30, 2024

just for the master branch?

@jt-traub
Copy link
Contributor Author

just for the master branch?

Yeah.. I don't need PR approvals to merge into the NO7 branch, so I did that already :)

@jt-traub jt-traub merged commit a5c249a into master Sep 30, 2024
10 checks passed
@jt-traub jt-traub deleted the jt-master-fix-entity-weight branch September 30, 2024 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants