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

Mordenkainen's Tome of Foes Bestiary #317

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TheXanathar
Copy link

All the monsters from MToF

All the monsters from MToF
@shepster
Copy link

Some errors:

  • Line 4486 ending <text> needs to be </text>
  • Lines 5369, 5373, 5377 legendary> needs to be <legendary>
  • Line 6160 <dex19> needs to be <dex>19
  • Lines 7462, 7619 needs to be <reaction>

Fixing these and it imports fine. Thank you for this data!

@TheXanathar
Copy link
Author

Some errors:

  • Line 4486 ending needs to be
  • Lines 5369, 5373, 5377 legendary> needs to be
  • Line 6160 needs to be 19
  • Lines 7462, 7619 needs to be

Fixing these and it imports fine. Thank you for this data!

thanks so much!

@Pedgleye
Copy link

Pedgleye commented Jan 16, 2020

Getting Import Failed: MONSTER Allip: expected: END_TAG {null descrption
(position:START_TAG
@45:10 in java.io
.InputStreamREader@e5ca9bf

Had the same problem, mucked around a bit
line 6160 needs to be 19 instead of

I had no idea how to fix the Allip code, it seemed fine so I just deleted it and am going to input it manually into the app.

EDIT: the description at the end of the allip block is what is throwing the app off. no other monster has one. In the volo compendium, descriptions are just listed as traits.
EDIT 2: I added the descriptions as traits under incorporeal movement and it works.

Thanks for the awesome code, this is my first time doing any kind of coding, I might try to code the stats from Decent into Avernus now. Its good fun.

garbledhamster pushed a commit to garbledhamster/DnDAppFiles that referenced this pull request Feb 5, 2020
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.

4 participants