feat: add login flow & session management #10
Conversation
I didn't like that the form wasn't centered in the window, so it looks like this now. The macOS screenshot tool tells me the padding at the top and bottom is the same (50px). I don't love the progress view (spinner), so I might get rid of it. I think the buttons disabling/dimming is enough feedback. |
| } | ||
|
|
||
| enum ClientError: Error { | ||
| case unexpectedStatusCode |
There was a problem hiding this comment.
| case unexpectedStatusCode | |
| case unexpectedStatusCode(Int) |
This lets you store the status code
| func user(_ ident: String) async throws -> User { | ||
| let resp = await request("/api/v2/users/\(ident)", method: .get) | ||
| guard let response = resp.response, response.statusCode == 200 else { | ||
| throw ClientError.unexpectedStatusCode |
There was a problem hiding this comment.
fine for now, but we should be parsing the generic error responses for debugging/troubleshooting
| } | ||
| } | ||
| }.onReceive(inspection.notice) { self.inspection.visit(self, $0) } // ViewInspector |
There was a problem hiding this comment.
I absolutely hate that you need to modify your UI code to get the tests to work, but you do. If nalexn/ViewInspector#231 is ever resolved you'd be able to just wrap the view with the inspector at test-time instead.
Closes #3.
Relates to #4.
Aboutpopup.SecureSessionthat writes the session token to the keychain, and the server URL toUserDefaults.Clientfor the SwiftcoderdSDK. Currently only supports theusersendpoint for validating a session token.Screen.Recording.2024-12-11.at.5.40.42.pm.mov
The 'generate a token' hyperlink takes the user to
/cli-auth.