Re: [PR] fix(frontend): double prefixed API url [superset]

["rusackas (via GitHub)" <gi...@apache.org>]

rusackas commented on PR #35641:
URL: https://github.com/apache/superset/pull/35641#issuecomment-4320871955

   @jakubhruby — friendly ping on this one. It's been sitting for a few months and has picked up some blockers:
   
   - Merge conflicts against `master` — needs a rebase.
   - CI is currently failing.
   - A couple of korbit-ai suggestions on `superset-frontend/src/explore/exploreUtils/index.js:262` are still open (regex escaping + avoiding RegExp creation in a hot path) — worth a look while you're in there, though they're optional.
   
   If you're still interested in landing this fix, could you rebase and push? Otherwise we can close it out and revisit later. No rush — just trying to keep the queue tidy.


-- 
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-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org