Reimport config JSON when apply LDtkProject file#70
Reimport config JSON when apply LDtkProject file#70hookSSi wants to merge 1 commit intoCammin:masterfrom
Conversation
Fix LDtkProject apply issue with config. I need to change LDtkConfigData.WriteJson to return the output path and an out bool isChanged.
|
I approve because of the application delay call, though I wonder why changes aren't being reflected, because I haven't seen this explicit issue before, at least on a consistent basis. Would be interested in seeing a reproduction. I've tended to avoid application delay calls so there's no potential for duplicate reimports |
|
FYI EnvironmentOS: Windows 11 Reproduce steps
Error: Warning: At this time I couldn't find config file but save again config file was shown
|
|
Hi, in general, it's expected to run into a lot of warnings and errors when setting this up. I made sure that all errors guide you towards the next step in the process. Once they are all done, then the errors are cleared up. Also in the docs, there is a setup guide for this. Let me know if you have more questions 🙂 |
|
Uh... yeah I know that too, but even after setting it up and handling all the warnings and errors this reproduce step still valid. I just wanted to show you the steps to reproduce it. Because If I'm not mistaken, I think you're wondering how to reproduce it. |
I modified the code because the changes weren't automatically reflected whenever I modified the LDtkProject file.
I simply reimported the LDtkConfig file. Please let me know if there are any issues.