AAfghahi commented on code in PR #21241: URL: https://github.com/apache/superset/pull/21241#discussion_r971187614
########## superset-frontend/src/logger/LogUtils.ts: ########## @@ -56,6 +65,14 @@ export const LOG_EVENT_TYPE_USER = new Set([ LOG_ACTIONS_MOUNT_EXPLORER, ]); +export const LOG_EVENT_DATASET_TYPE_DATASET_CREATION = new Set([ Review Comment: Yeah I was trying to figure that out as well, because you end up just importing them straight into the component either way. It was the pattern used by the other logging actions above so I copied it, but those sets are also not invoked anywhere. Happy to delete. -- 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]
