terriajs icon indicating copy to clipboard operation
terriajs copied to clipboard

Published 20 hours ago • verified publisher icon TerriaJS

Scene Editor works on DTV but not NSW or QLD

Open meh9 opened this issue 2 years ago • 4 comments

For some reason the Scene Editor works on DTV, but it does not work in NSW or QLD. Appears to be some form of loading issue?

Screen Shot 2022-05-27 at 11 02 57

meh9 avatar May 27 '22 01:05 meh9

Sorry about that. It was fixed by https://github.com/TerriaJS/terriajs-place-editor/pull/12 and released to dev environments for nsw & qld by @mwu2018 . Release to other environments are pending.

na9da avatar Jun 01 '22 06:06 na9da

@na9da All envs have been on the same version so their behaviour should be the same. @meh9 Maybe you need to clear cookies/cache?

mwu2018 avatar Jun 01 '22 06:06 mwu2018

Currently the behavior is inconsistent between different environments, so it may indeed be fixed but I am going to argue that there is more to do.

Currently NSW selects autodetect by default, so if you don't pay attention that's what it tries and fails, and you get a nice big error if you leave it at that: Screen Shot 2022-07-14 at 13 34 32

VIC and QLD don't even have autodetect as an option: Screen Shot 2022-07-14 at 13 34 39

Screen Shot 2022-07-14 at 13 35 12

Note different capitalization though, what's up with that?

meh9 avatar Jul 14 '22 03:07 meh9

NSW has latest scene editor - QLD and VIC have slightly older versions

Commit here adds IFC and Auto-detect https://github.com/TerriaJS/terriajs-place-editor/commit/6be7e54747fae38fbbdf50e1f3e4e88536155016

I don't think this ticket should be here - it should be in https://github.com/TerriaJS/terriajs-place-editor or the digital twin repos

I have added a comment to update the place editor in vic-dt release ticket https://github.com/TerriaJS/vic-digital-twin/issues/331

We currently don't have a release ticket for qld

nf-s avatar Jul 14 '22 05:07 nf-s