fix: Application history log interface loading status display error #2151
Conversation
| @@ -562,7 +564,7 @@ function saveCleanTime() { | |||
| .then(() => { | |||
| MsgSuccess(t('common.saveSuccess')) | |||
| dialogVisible.value = false | |||
| getDetail() | |||
| getDetail(true) | |||
| }) | |||
| .catch(() => { | |||
| dialogVisible.value = false | |||
There was a problem hiding this comment.
In this code snippet, there are several minor improvements and optimizations:
-
Type Annotations: Replace
anywithRef<any>to improve type safety while utilizing Vue's reactivity system. -
Use of Parameters: The parameter definition for the
getDetailfunction allows passing an optional boolean (isLoading) which sets the loading status dynamically. This is beneficial when you want to control whether the loading should occur or not based on certain conditions. -
Function Parameter Naming: Changed
id as stringto justidsince it can be reliably converted to a string during the function call without needing explicit casting.
These changes enhance the readability, maintainability, and performance of the code by improving typesafety and reducing unnecessary conversions.
fix: Application history log interface loading status display error