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

Add custom values to names.cfg #719

Merged
merged 1 commit into from Jan 2, 2021

Conversation

Flabort
Copy link
Contributor

@Flabort Flabort commented Jan 2, 2021

Currently the Apotheosis bosses all have default equipment and names, excluding netherite which has been removed from being possible.

This proposed change adds a number of materials to the pool by giving them names, trying not to be overpowered and appreciating the current values for what non-bosses may spawn with. This also adds a bunch of prefixes to boss names to add a bunch of variety (I'm tired of every boss I meet having the prefix Doctor, for example) and new name-parts for some additional random names.

As a side effect, I believe this will allow some of the named materials to appear as affix dungeon loot, but this is in my opinion desirable - as long as the materials are not overpowered compared to existing affix dungeon loot.

Currently the Apotheosis bosses all have default equipment and names, excluding netherite which has been removed from being possible.

This proposed change adds a number of materials to the pool by giving them names, trying not to be overpowered and appreciating the current values for what non-bosses may spawn with. This also adds a bunch of prefixes to boss names to add a bunch of variety (I'm tired of every boss I meet having the prefix Doctor, for example) and new name-parts for some additional random names.

As a side effect, I believe this will allow some of the named materials to appear as affix dungeon loot, but this is in my opinion desirable - as long as the materials are not overpowered compared to existing affix dungeon loot.
@NielsPilgaard
Copy link
Collaborator

Looks like some good changes, thank you! :)

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.

None yet

2 participants