feat: Better user experience when importing buckets

[johan-bjareholt]

Instead of being redirected to an empty page the error is now shown on
the same page and the buckets list reloaded

[johan-bjareholt]

@ErikBjare Seems to be some error with the screenshot test?

[ErikBjare]

@johan-bjareholt Looks like it works as intended and found an error while trying to load the page.

Did you look for the error in your own browser or try to run the e2e tests locally?

[johan-bjareholt]

@ErikBjare Seems to break when you add the "dom" lib to tsconfig in aw-client-js.
It passes anyway if you just pass --skip-js-errors to testcase, but it's better to just fix the root cause.

[ErikBjare]

Ah, as I mentioned in my late review of your aw-client-js PR then.

And yeah, passing --skip-js-errors would kinda ruin the point of the test :)

[johan-bjareholt]

And yeah, passing --skip-js-errors would kinda ruin the point of the test :)

Well, it still runs and passes the test so I wouldn't say pointless. It just ignores runtime errors in the browser.
But it's still a bad idea.

[johan-bjareholt]

Reverted the aw-client-js change and moved it into aw-webui which will always have a DOM, so it makes more sense to have it here.