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

Some bug fix #41

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Conversation

p4535992
Copy link

@Varriount some strange bug.... made some fast fix.

@Fridan99
Copy link

@p4535992 Do you know if your PR works well in v11? In v10.291 there is no problem at all.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why was this file deleted? It's required in order to push the package release to Foundry's module registry.

@p4535992
Copy link
Author

a sorry I had thought that you had abandoned the module on your side and went with self-development.

I don't know if your old release, supports the new structure I generate with vite (give it a try), anyway I renamed the release.yml file to main.yml (i cannot remember why) it should be the same with the auto publish part commented out.

I, @Forien , and @trioderegion (more the last two) are trying to create a new evolution of the module that has many limitations and bugs, and try to merge FUI with Archon, if you are interested (https://github.com/Forien/foundryvtt-forien-unidentified-items).

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.

3 participants