In the next diff (D11766) I end up removing this in favor of just calling saveCredentials at the end (after keyserver auth), but I decided to leave it in here in case that diff needs to get reverted for some reason

In D11763 we check if step is inactive. Do we need to do that here?

This probably shouldn't be named "registration". And, same as in D11763, I think this shouldn't be named "waiting for", when later it represents the state after registration has succeeded.
Also because we don't have a third step to represent the keyserver auth, I think we actually care about two states - "no login in progress" and "identity login concluded, keyserver login in progress"

Good call! Yes, I'll make that change

This probably shouldn't be named "registration"

Good point, I'll make this change

And, same as in D11763, I think this shouldn't be named "waiting for", when later it represents the state after registration has succeeded.

As described there, we're really waiting for data to appear in Redux, but I thought that name was a bit of a mouthful. Open to other suggestions, but opposed to identity_registration here because we're really doing keyserver auth during this step.