Gaulish Gaming Banquet: editor

pegperegogaucho

Warning message is probably bes --- MESSAGE FROM XXL EDITOR --- The checklist entry you are trying to remove is still being referenced by an event sequence (#69)! In order to delete this checklist entry, please delete the event sequence.

adrientd

Yes, I guess we could try like this

adrientd

Also there may be multiple event sequences referencing it.

pegperegogaucho

We could always also give a list: "referenced by event sequences (#69, #420, ...) or (see below) and then include them at the end"

adrientd

Yeah easiest is a bullet point list

pegperegogaucho

Maybe you could also add a button "Fix now" which then deletes those actions

adrientd

Yes, at the same time they should also know what they are doing :Obelul:

pegperegogaucho

What the button do???

deathhound246

Fixing up on my PR for real this time :Obelul:

deathhound246

Have it done in a few mins

deathhound246

That should be it now, and rebased with latest commits too

pegperegogaucho

Excellent, the G-Man will await you in the pull request discussion. Meet him there.

adrientd

@deathhound246 Good thing you rebased, because I realized that the Hex button did not work in Appveyor builds, and I did quick fix :Obelul:

adrientd

So now I can test your feature through the Appveyor build

adrientd

And it looks good!

deathhound246

pegperegogaucho

XXL Editor development has been popping off lately :)

deathhound246

tbh, any more PRs i put it will probably be more specific to Alice. I'm just doing this one cus I like being able to look at a class group and know what it is without having to look up the IDs

adrientd

Just merged!

deathhound246

ty!

#editor