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

Improve compatibility with all mods that add dungeons to Garden planet #3529

Merged
merged 4 commits into from
Jan 24, 2024

Conversation

edwardspec
Copy link
Contributor

@edwardspec edwardspec commented Jan 16, 2024

  • Improved compatibility with all mods that add dungeons to the starting Garden planet.
  • Updated description of Ship Repairs quest to not mention removed item (key to the Dark Cavern).
  • Fixed quest descriptions where Foraging Table was incorrectly called Foraging Station.
  • Updated flavor text in Glitch race description to match their masteries.

Vinalisj no longer gives a key, he just opens the Dark Cavern.
These mods were incompatible, because their dungeons were competing
with things like "Dark Cavern entrance", causing them not to spawn.

Garden planet was set to have minimum/maximum number of dungeons of 4,
and because FU has exactly 4 dungeons that must spawn there,
all 4 were guaranteed. But some mods were messing this up.

However, as it turns out, we can safely set minumum number of dungeons
to a high number (in this case, 10), because the dungeons are
planet-unique (can't appear more than once),
so if there are only 4 dungeons, and "minumum 10 dungeons" were
requested, then only these 4 will spawn (without duplicates),
which is exactly what we want.

If some mod adds 1 extra dungeon to the Garden planet, then there will
just be 5 dungeons, and FU dungeons won't be prevented to spawn.

The issue can still occur if player installs so many of these mods
that they add 7 or more dungeons, but it's a self-inflicted problem.
After the race revamp, they no longer have bonus to bow/mace,
but the flavor text wasn't updated to reflect this.
@sayterdarkwynd sayterdarkwynd merged commit ecb2443 into sayterdarkwynd:master Jan 24, 2024
4 checks passed
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