feat: set autoConnet() expiry time #307
Conversation
|
Hi, sound good. It is set to a defined time before reconnecting with useAutoConnect, the TLL is set to 60 sg in milliseconds @Iwueseiter should check the format of the changes to ensure they comply with the rules, run |
|
Hi @Iwueseiter , remember that today is the last day of the ODHack, you just need toead comments, Only run the tests so that the CI Jobs is ok |
|
Noted |
…into expiry_autoconnect
|
@metalboyrick I’ve ran all the formatting commands, but it’s still failing. I noticed Prettier wasn’t included in the dependencies, so I had to install it. Also, what formatting style is the CI expecting? There’s no .prettierrc configuration file in the codebase. |
|
We use the default NextJS core vitals for linting. You can try running If that does not work I've checked for you what the formatter expects, It expected double quotes instead of single quotes for the strings. You can ping us in the telegram group ASAP should you run into further troubles, I have the same handle. Thank you! |
| const [_, setLastConnector] = useLocalStorage<{ id: string; ix?: number }>( | ||
| "lastUsedConnector", | ||
| { id: "" }, | ||
| { | ||
| initializeWithValue: false, | ||
| }, | ||
| ); | ||
| const [, setLastConnectionTime] = useLocalStorage<number>( | ||
| "lastConnectionTime", |
There was a problem hiding this comment.
Kindly extract this "lastConnectionTime" to a constant, thank you!
|
@metalboyrick please review. |
|
Hi @Iwueseiter, Thanks a lot for your All changes updated, and Approved for me, need review by @metalboyrick |
|
@metalboyrick kindly review. Thank you |
Set autoConnet() expiry time
Fixes #280
Types of change
Comments (optional)