Skip to content

Feat: remember last identity provider on login - #66

Merged
PreciousOritsedere merged 2 commits into
mainfrom
39-remember-last-idp
Jul 24, 2026
Merged

Feat: remember last identity provider on login#66
PreciousOritsedere merged 2 commits into
mainfrom
39-remember-last-idp

Conversation

@PreciousOritsedere

Copy link
Copy Markdown
Contributor

Summary

Remembers the last Solid identity provider URL in localStorage and prefills the login field with it on the next visit, so returning users don't have to re-enter their provider.

Closes #39

Test plan

  • Log in with a provider, log out → login field is prefilled with that provider
  • Log in with a different provider → prefill updates to the new one
  • Login still works with the preset options and custom URLs

@jeswr jeswr Jul 24, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would keep this file named as just LoginPage the fact that it remembers the IDP is just a feature of the page.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have pushed a commit for this

@jeswr jeswr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should be aiming to get the "remember last identity provider" feature upstreamed to the reactive authentication library; and to migrate to that library here.

@PreciousOritsedere

Copy link
Copy Markdown
Contributor Author

We should be aiming to get the "remember last identity provider" feature upstreamed to the reactive authentication library; and to migrate to that library here.

Yes I plan to do this subsequently

@PreciousOritsedere
PreciousOritsedere merged commit 2917db0 into main Jul 24, 2026
@PreciousOritsedere
PreciousOritsedere deleted the 39-remember-last-idp branch July 24, 2026 13:59
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.

Feature request: remember last used identity provider

2 participants