Skip to content

Auth session type#292

Open
bourgeoa wants to merge 2 commits into
stagingfrom
authSessionType
Open

Auth session type#292
bourgeoa wants to merge 2 commits into
stagingfrom
authSessionType

Conversation

@bourgeoa

@bourgeoa bourgeoa commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Types authSession OIDC client

@bourgeoa bourgeoa self-assigned this Jun 12, 2026
@bourgeoa bourgeoa added enhancement New feature or request javascript Pull requests that update javascript code labels Jun 12, 2026
@bourgeoa bourgeoa moved this to In review in SolidOS NLNet UI Jun 12, 2026
@timea-solid

Copy link
Copy Markdown
Member

it looks ok to me but I don't really know in detail.

@NoelDeMartin NoelDeMartin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm also not sure, but I'll approve because this looks like just a backwards compatible or improvement on the previous code, so I guess it's better than what it was before.

Not sure about the webpack-cli upgrade, though, was that necessary?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request javascript Pull requests that update javascript code

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

3 participants