Sentiaus commented on code in PR #3731:
URL: https://github.com/apache/texera/pull/3731#discussion_r2347690818


##########
core/config/src/main/resources/gui.conf:
##########
@@ -100,5 +100,9 @@ gui {
     # whether to send email notification when workflow execution is 
completed/failed/paused/killed
     workflow-email-notification-enabled = false
     workflow-email-notification-enabled = 
${?GUI_WORKFLOW_WORKSPACE_WORKFLOW_EMAIL_NOTIFICATION_ENABLED}
+
+    # amount of time to be elapsed in minutes before user is detected as 
inactive
+    active-time-in-minutes = 15

Review Comment:
   Since `active-time-in-minutes` is meant to be a gui only effect, i.e. the 
admin is able to see which user's are active based on their time, would it not 
be more appropriate to leave the variable in `gui.conf`?



-- 
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]

Reply via email to