devmadhuu commented on PR #10420: URL: https://github.com/apache/ozone/pull/10420#issuecomment-4619470967
> @devmadhuu Thanks for reviewing the changes. Looks PR description seems confusing. > > The backend polling mechanism will still remain in place. This PR only removes the auto-refresh toggle button from the UI. If we keep the toggle enabled now, it will not perform any action, which may create confusion and lead users to believe that the feature is not working. Therefore, it is better to remove the toggle option from the UI. > > I also updated the PR description. @priyeshkaratha So as discussed offline, keep the behavior of page consistent in terms of refresh of page because Pending Deletion card also there on page. So better when UI already polling and continue to poll till refresh of page by calling respective APIs once they finish their execution, we should also call Pending Deletion APIs and refresh their data on the page. -- 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]
