dombizita commented on code in PR #3868:
URL: https://github.com/apache/ozone/pull/3868#discussion_r1007890611
##########
hadoop-ozone/recon/src/main/resources/webapps/recon/ozone-recon-web/src/utils/autoReloadHelper.tsx:
##########
@@ -49,6 +49,10 @@ class AutoReloadHelper {
this.stopPolling();
}
};
+
+ toggleChecking = (checked: boolean) => {
+ sessionStorage.setItem('toggleCheck', JSON.stringify(checked));
+ };
Review Comment:
Ohh I think I misunderstood you question @DaveTeng0, I thought you are
asking about the change I suggested, sorry :) `toggleCheck` will be the
property we set in the
[sessionStorage](https://developer.mozilla.org/en-US/docs/Web/API/Window/sessionStorage),
so in the browser we can ask for that property until the page session ends (if
we would set it in the
[localStorage](https://developer.mozilla.org/en-US/docs/Web/API/Window/localStorage)
that would have no expiration time). The `checked` is a variable in the code,
which stores the current state of the toggle in the Recon UI. That is why in
the new method we set the `toggleCheck` to `checked`, which is passed as a
parameter to the method (same parameter is used in the already existing
`handleAutoReloadToggle` method). I hope this answers your question :) correct
me if I am wrong @smitajoshi12
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
