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

Simple fix for use of Item5e.createOwned #369

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

Conversation

slowglass
Copy link

Remove use of Item5e.createOwned as this has been dropped from the product.
This was causing issues when BetterRolls5e was interacting with spells that were fired from Magic Items

@slowglass
Copy link
Author

I appreciate you have stopped development on Better Rolls. However I have created the PR incase some one picks up this work

@CarlosFdez
Copy link
Collaborator

The project has been picked up, but its getting quite the update, and apparently this has been fixed as part of it. Just to let you know.

@Th3Rom3
Copy link

Th3Rom3 commented Apr 11, 2022

@CarlosFdez Are there any infos on the progress or is it done as a separate fork?

@Chris-Davies
Copy link

When can we get eyes on the update please. Would be very happy to test

@CarlosFdez
Copy link
Collaborator

I am not sure. I think they're waiting for Foundry V10 maybe.

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