mistercrunch commented on PR #28522: URL: https://github.com/apache/superset/pull/28522#issuecomment-2113658031
Starting to feel like we need a refactor of some kind here as this isn't DRY, maybe moving some things to a new `superset/utils/json.py`, taking all the json-related stuff in `superset/utils/core.py`, and moving all this recent utf-16 handling in there too. I think I started this refactor before in a branch but never carried though. Let me push the branch for reference. Are you interested in taking this on? -- 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: notifications-unsubscr...@superset.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
