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

feat(FTL-16871): integrate with redirect flow #94

Merged
merged 3 commits into from
Oct 8, 2024
Merged

Conversation

rbrazhnyk
Copy link
Contributor

No description provided.

@rbrazhnyk rbrazhnyk self-assigned this Oct 7, 2024
maindotdev
maindotdev previously approved these changes Oct 7, 2024
samples/affinidi-nextjs-nextauthjs/src/lib/clients/iota.ts Outdated Show resolved Hide resolved
samples/auth0-nextjs-nextauthjs/src/lib/clients/iota.ts Outdated Show resolved Hide resolved
andrew-affinidi
andrew-affinidi previously approved these changes Oct 7, 2024
@rbrazhnyk rbrazhnyk merged commit 2c46a08 into main Oct 8, 2024
7 checks passed
@rbrazhnyk rbrazhnyk deleted the rb-iota-redirect branch October 8, 2024 06:18
Comment on lines +96 to +98
function getShareLink(jwt: string, clientId: string) {
return VaultUtils.buildShareLink(jwt, clientId);
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need for a separate function


const IotaRedirectFlowCallbackPage = dynamic(
() => import("../components/iota/RedirectFlowCallbackPage"),
{ ssr: false },
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be no need to make the redirect pages components and disable SSR.


const IotaRedirectFlowPage = dynamic(
() => import("../components/iota/RedirectFlowPage"),
{ ssr: false },
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be no need to make the redirect pages components and disable SSR.

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

Successfully merging this pull request may close these issues.

4 participants