Open philippotto opened 2 weeks ago
Some first insights: The onboarding screen was not shown because of the schema compliance check, but because the organization list query failed (since the columns of the organization table were changed in that particular migration). This means that
createOrganizationWithAdmin
, which is the same as the standard register form. So it can indeed not destroy existing data.Do you know in what circumstances the other of the two occurrences happened?
I managed to reproduce the issue and for me the /api/organizationsIsEmpty
request correctly replies with 400 when the database access for the organizations table fails. Looks like the frontend catches this 400, and then displays the onboarding screen.
So I’d exchange the backend for the frontend label for this issue. Let me know if you think the backend should behave differently.
In the past, we had at least two occurrences where a WK production system showed the onboarding system, because it considered the DB to be empty. No data was lost, but nevertheless the screen is scary.
We should do the following two things:
Some notes on when this happened: