godot-tiled-importer icon indicating copy to clipboard operation
godot-tiled-importer copied to clipboard

Will this get an update to work with Tiled 1.2 Beta?

Open chaosdrop opened this issue 6 years ago • 14 comments

Plugin version

2.x Issue description

The JSON format has changed. Steps to reproduce

Sample map or tileset

chaosdrop avatar Sep 02 '18 23:09 chaosdrop

Eventually, yes, but only after a stable version is released. Also, I'm not really fond of maintaining two versions of the JSON loader, I'll probably update to the new format and ask people to update their Tiled editor (and that can only be done after the stable release).

vnen avatar Sep 03 '18 02:09 vnen

The stable version is out. Just giving you a heads up.

https://www.mapeditor.org/2018/09/18/tiled-1-2-0-released.html

HeadClot avatar Sep 19 '18 22:09 HeadClot

does Tiled 1.2 supported? When 2.3 version release?

zhengying avatar Dec 05 '18 05:12 zhengying

TBH I don't know. I work on this plugin on my free time and I have almost no free time anymore. I will still review PRs if anyone is willing to do it.

vnen avatar Dec 05 '18 12:12 vnen

@zhengying I have been using Tiled 1.2 since it's official release in September. 1.2.1, released in November, works as well. Are you asking because you found an issue or asking before you try?

As someone who has been programming for over 16 years I very highly recommend just jumping into things and getting your feet wet. If it doesn't work, you can file an issue or even figure out what's wrong and fix it yourself; if it does work then all is well and you didn't need to post anything at all. :D

rosshadden avatar Dec 06 '18 01:12 rosshadden

@rosshadden I tried yesterday, I found tmx file parse error, so I found here has an issue about tiled 1.2.

@rosshadden you said you have been using Tiled 1.2 , why your tmx file work fine? did you fix something make it works ?

zhengying avatar Dec 06 '18 02:12 zhengying

Oh I'm sorry, I should have mentioned that. Yeah I had to do this: https://github.com/godotengine/godot/issues/23631#issuecomment-442227273

rosshadden avatar Dec 06 '18 02:12 rosshadden

Thanks @rosshadden !!
You save me!

zhengying avatar Dec 06 '18 02:12 zhengying

AFAIK only the JSON format has changed. So if you're using TMX it shouldn't be a problem. But the plugin still needs to be updated for the new JSON format.

vnen avatar Dec 06 '18 13:12 vnen

I have only ever used the TMX format, so maybe something changed in the past few weeks that wasn't called out.

rosshadden avatar Dec 06 '18 14:12 rosshadden

AFAIK only the JSON format has changed. So if you're using TMX it shouldn't be a problem. But the plugin still needs to be updated for the new JSON format.

but even use tmx format the tiles show not correct, I use jfmaror workaround method , works well now. can you merge this to master? thanks!

Added "tilemap.format = 1" around line 222 of tiled_map_reader.gd after the TileMap.new() link here: jfmajor 's a workround

zhengying avatar Dec 07 '18 01:12 zhengying

but even use tmx format the tiles show not correct, I use jfmaror workaround method , works well now.

This is not related to the new Tiled format, but happens because of a recent change in Godot. This is already tracked in #103.

can you merge this to master? thanks!

If someone make a PR, yes.

vnen avatar Dec 07 '18 01:12 vnen

thanks @vnen I just see the #103 , I agree you that handle it after release 3.1 stable version.

zhengying avatar Dec 07 '18 01:12 zhengying

Hi, what version of tiled map should we use then ?

slevy85 avatar Apr 08 '19 18:04 slevy85