ElfFriend-DnD / foundryvtt-5eOGLCharacterSheet

Aiming to reproduce the layout of the print 5e Sheet within foundry
MIT License
10 stars 14 forks source link

Does not work at all in V10 #38

Open SLinxTheFox opened 2 years ago

SLinxTheFox commented 2 years ago

Just letting you know. I'm super grateful you keep this module working. It's the only character sheet i can stand. If i had the skills to take it over for you i would

AndreCaseyScott commented 2 years ago

I can confirm. Does not show up as a Character Sheet option in v10. Hope someone picks up this project, my players loved using it.

Shepardnick commented 2 years ago

Yes, my players and I have been using this after transferring from roll20 early this year. I find all the other available character sheets to be too spread out with their categories and information tabs. My players and I appreciate the work you've done on this module, and I hope it'll become available for v10.

Hokage3211 commented 2 years ago

40 in case anyone is needing this, check the releases of the repository for the pull request.

mrmchughes commented 1 year ago

I had this module working on Foundry Version 9, after updating to the latest patch of Version 10, the OGL sheet no longer appears as a sheet option. I have attempted deactivating every module but this sheet, and it still does not appear, so I think it is something with Version 10. Thank you for continuing the work on this, hope it can get working again!

hedgehog8711 commented 1 year ago

I would love to see this working with the new version of foundry it looks like they have landed on a stable build if that helps for development. I hope they come back to this project and update it a lot of my group loves the old school sheets, and hates the new ones lol. Thank you devs in advance for your help.

AndreCaseyScott commented 1 year ago

From what I can tell, the code in master includes a fix for V10. I even saw a message from the author saying it's fixed for v10. Still doesn't show.

The manifest.json still references the most latest release on github, which does not include the fix. If @akrigline will create a new release using the latest master branch, then the manifest.json can be updated and the fix applied. At least that's what I suspect at the moment.

mcabel commented 1 year ago

does anyone know if the proposed changes in #40 fix the issues and allow running this on v10?