-
Notifications
You must be signed in to change notification settings - Fork 68
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
✨ NEW: make savedSession behavior configurable #280
Conversation
537b963
to
20c074d
Compare
20c074d
to
9ad9bdf
Compare
@rkevin-arch Could you review this? |
The code looks great, gonna test it locally in a bit |
Other than these, it looks pretty good to me |
- fix units for age comparison (milliseconds->seconds) - consistent lastUsed naming
44716b3
to
8f761bf
Compare
@rkevin-arch thanks for your review! I think it's all set now. |
This looks great to me - thanks @rkevin-arch for the review. I'm +1 on this once you are. |
OK given that review comments are addressed and tests are happy, I'm just gonna merge this one and we can iterate in another PR if there are follow-ups needed! |
adds binderOptions.savedSession configuration to give users a little control over #266
savedSession.enabled
savedSession.maxAge
savedSession.storagePrefix
and reorganize the code a little bit to make the flow a little easier to manage with all the different stages to abort, including JSON parse errors, etc.