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

Added Shield-Bearer - PY118, Mystic Swordman - PY36, Demonologist - DF9 #167

Merged
merged 8 commits into from
Aug 30, 2022

Conversation

jcklemme
Copy link
Contributor

No description provided.

@jcklemme jcklemme changed the title Added Shield-Bearer template from Pyramid 118 Added Shield-Bearer template from Pyramid 118, Aug 20, 2022
@jcklemme jcklemme changed the title Added Shield-Bearer template from Pyramid 118, Added Shield-Bearer - PY118, Mystic Swordman - PY36 Aug 20, 2022
@jcklemme jcklemme changed the title Added Shield-Bearer - PY118, Mystic Swordman - PY36 Added Shield-Bearer - PY118, Mystic Swordman - PY36, Demonologist - DF9 Aug 20, 2022
@richardwilkes
Copy link
Owner

Let me know when you have the final version of these changes in place, so I can review it all at once.

@jcklemme
Copy link
Contributor Author

jcklemme commented Aug 20, 2022

This is all I'm planning on working on for the week.
To clarify, those templates should be done and I am not planning on adding more in the coming week.

@richardwilkes
Copy link
Owner

The college for the demonologist spells doesn't seem right. Shouldn't it be something like "Demonology" instead?

@jcklemme
Copy link
Contributor Author

jcklemme commented Aug 30, 2022

The college for the demonologist spells doesn't seem right. Shouldn't it be something like "Demonology" instead?

It doesn't actually flat out say what the college name is in DF9, and I scratched my head about it for a while and decided to go with "Demonological" since the abilities are classified as Demonological. I was kind of "computer braining" it and wanted to match as closely to the nearest thing. If that's not right or I missed something you saw I'd believe you lol

Edit - I'm going back over it and yes, it appears that the Power Modifier is "Demonology", but the headers and text refer to them as "Demonological Abilities" which is probably where I got that.

@richardwilkes richardwilkes merged commit 4692f6d into richardwilkes:master Aug 30, 2022
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