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

Forgix changing JSON files inside data directory? #30

Closed
Kitteh6660 opened this issue May 16, 2024 · 5 comments · Fixed by #35
Closed

Forgix changing JSON files inside data directory? #30

Kitteh6660 opened this issue May 16, 2024 · 5 comments · Fixed by #35

Comments

@Kitteh6660
Copy link

Kitteh6660 commented May 16, 2024

I have an issue with Forgix and for some reason, it decided to rename those JSON files and thus break the mod's datapack and crashing the world creation screen.

Is there a way to stop Forgix from doing that?

image

I'm using Forgix 1.2.9 if that helps.

@Kitteh6660 Kitteh6660 changed the title Forgix changing JSON files inside data file? Forgix changing JSON files inside data directory? May 16, 2024
@Ran-Mewo
Copy link
Member

Could I see those files? I have a feeling it's accidentally detecting those files as mixins

@Kitteh6660
Copy link
Author

I can do that. Those are the affected JSON files.

fabric-patch_froglight.json
forge-flower_luzawood.json
forge-patch_froglight.json
fabric-flower_luzawood.json

I will be updating my code repository so you can see what's wrong.

@hypherionmc
Copy link
Collaborator

hypherionmc commented May 16, 2024

@Ran-Mewo could this be a case of the file being renamed because it's located in the forge/fabric folders? Possibly related to this?

@Kitteh6660
Copy link
Author

I've updated the source code: https://github.com/Kitteh6660/Ceilands

The original JSON files are stored in common.

@hypherionmc
Copy link
Collaborator

I think as a safety thing, we shouldn't ever touch files inside assets and data. There shouldn't ever be a case where we need to modify something there

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 a pull request may close this issue.

3 participants