Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing redirect_browser_to action on ContinueWith in typescript client #384

Open
4 of 5 tasks
JeremyMoeglich opened this issue Nov 7, 2024 · 1 comment
Open
4 of 5 tasks
Labels
bug Something is not working.

Comments

@JeremyMoeglich
Copy link

Preflight checklist

Ory Network Project

No response

Describe the bug

This is a snippet from the generated Typescript SDK. There should be a redirect_browser_to action, but it's not present.

/**
 * @type ContinueWith
 * @export
 */
export type ContinueWith = {
    action: 'set_ory_session_token';
} & ContinueWithSetOrySessionToken | {
    action: 'show_recovery_ui';
} & ContinueWithRecoveryUi | {
    action: 'show_settings_ui';
} & ContinueWithSettingsUi | {
    action: 'show_verification_ui';
} & ContinueWithVerificationUi;

It is mentioned in the docs https://www.ory.sh/docs/kratos/reference/api#tag/frontend/operation/updateRegistrationFlow
image

The ContinueWithActionRedirectBrowserToString is also nowhere within the sdk.

Reproducing the bug

Install @ory/kratos-client 1.2.1

View it's code in node_modules, see it's missing

Relevant log output

No response

Relevant configuration

No response

Version

Kratos 1.3.0, "@ory/kratos-client": "^1.2.1",

On which operating system are you observing this issue?

Windows

In which environment are you deploying?

Docker

Additional Context

No response

@JeremyMoeglich JeremyMoeglich added the bug Something is not working. label Nov 7, 2024
@ChaserZ98
Copy link

I am also facing this when trying to update a login flow. From my current experience, the DX of using the SDK is awful. The doc is inconsistent and there isn't any well-explained doc about the logic of the SDK. It turns out to me that directly using fetch to call each api is more straightforward except that you have to handle with the type check yourself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working.
Projects
None yet
Development

No branches or pull requests

2 participants