PeteAUK / warscroll-designer

A Warscroll creation tool for the Age of Sigmar system (issue tracker)
0 stars 0 forks source link

Warscrolls not actually saving to account #14

Closed TheOccultNerd closed 1 year ago

TheOccultNerd commented 1 year ago

Describe the bug After finishing a scroll and clicking the save icon, it first pops up in the list as undefined, then once save is pressed again the name is correct. however the scroll is not present in the list of created warscolls when the page is re-opened and sign in is complete

To Reproduce create a warscroll, name it, save, save again and return to the page in a new tab.

Expected behaviour being able to access scrolls previously made and saved.

Desktop (please complete the following information):

PeteAUK commented 1 year ago

I suspect this is a duplicate of #13. Could you confirm the following:

I'm struggling to replicate this happening on any machine even when running in incognito mode, so I'm either missing a step or doing something automatically that means the issue doesn't appear 😊

TheOccultNerd commented 1 year ago

Oh sorry if I didnt explain well, I was logged in with google and I do have warscrolls that were saved in the previous version that I can edit and it does properly save those, I just cant save new scrolls.

exonian commented 1 year ago

I get this on all devices, across two google accounts (neither with anything saved, but one set up specially for the occasion to test it wasn't just my account 😅)

PeteAUK commented 1 year ago

I'll try and have another punt this weekend at working out what's going on. Have tried on multiple machines and not encountered it yet, so I'm not quite sure what's happening. Initial feeling was browser caching, but I think that's a red herring in truth.

PeteAUK commented 1 year ago

Blimey that was a bit awkward! However I believe that I've now solved this. Basically it was two problems rather than one which also didn't exist on my local set up. Please test this out and let me know how it goes.

exonian commented 1 year ago

@PeteAUK It works!

exonian commented 1 year ago

I may have spoken too soon... 😬 Having trouble getting a second scroll to save. Trying to characterise it now.

PeteAUK commented 1 year ago

I have just noticed that you can't select a newly saved warscroll - event handlers not attaching!

exonian commented 1 year ago

I think I'm having trouble saving at any point after hitting "new", even with refreshes after save events. I'm not sure if there's a little more to it than that, but "save, refresh, enter name, tab, save, refresh, enter name, tab, save" seems to work fine, but I think it's hitting "new" where my problems start. Maybe.

PeteAUK commented 1 year ago

Ah, worked it out - was the same issue you were having. When clicking "new" the loaded variable wasn't enumerating to empty. I've just modified this and seems to be OK now.

exonian commented 1 year ago

Ah perfect. That'll presumably solve another issue I saw a few minutes ago 🎉 (new scrolls saving as local if you'd loaded up a local scroll before hitting "new")

PeteAUK commented 1 year ago

It should do with luck! But happy to reinvestigate if you're still having problems :)

exonian commented 1 year ago

Nah I'm happy to test that case and report back if it's still there. And also the one I mentioned the other day about being able to save with no warscroll name: I think potentially this stops that too? Cos it was directly after hitting new, and I imagine it not unloading properly may have been what was making the diff false-positive

But yes, as for this ticket I was just about to write here "oh wait scratch that I don't think it's the 'new' thing after all!" but it turned out you'd just fixed it 😂

PeteAUK commented 1 year ago

Cheers. The saving with no name I've got on the short list to sort out - there's a "have I changed" flag that isn't resetting for some reason, however it that should really be a mandatory field too.