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

Languages fix #129

Merged
merged 2 commits into from
Mar 14, 2023
Merged

Languages fix #129

merged 2 commits into from
Mar 14, 2023

Conversation

BramvdnHeuvel
Copy link

Elven -> Elvish
Orcish -> Orc
none -> empty string
Elemental -> Primordial

Also, the name of the Phoenixborn Sorcerer was kinda messed up, so I fixed that by looking the creature up in the original document.

Bram and others added 2 commits March 14, 2023 10:58
Elven -> Elvish
Orcish -> Orc
none -> empty string
Elemental -> Primordial
@eepMoody
Copy link
Collaborator

eepMoody commented Mar 14, 2023

Hey @BramvdnHeuvel sorry for the messy additions to this PR. I wanted to get the fixes merged in without throwing them back your way. Super appreciate the contribution!

We had some simultaneous cleanup efforts run into each other, so I did my best to play all your changes (and a few more in the same spirit) back on top of the larger changes that were in the queue as well.

@eepMoody eepMoody merged commit 202f70a into open5e:master Mar 14, 2023
@BramvdnHeuvel BramvdnHeuvel deleted the elven-to-elvish branch March 14, 2023 15:08
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