fix(config): replace external-url into studio api#4803
fix(config): replace external-url into studio api#48039bany wants to merge 1 commit intosupabase:developfrom
Conversation
|
Could you please check this PR @sweatybridge ? |
sweatybridge
left a comment
There was a problem hiding this comment.
The TODO is actually meant to keep Api.ExternalUrl and delete Studio.ApiUrl
d6a1803 to
7852108
Compare
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Cache: Disabled due to Reviews > Disable Cache setting Disabled knowledge base sources:
📒 Files selected for processing (5)
💤 Files with no reviewable changes (2)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe changes refactor the API external URL configuration from a studio-specific field to a global API configuration field. The Comment |
|
Sorry for the late response, |
What kind of change does this PR introduce?
Fix TODO noted in code based
What is the current behavior?
This PR remove external-url in config, remove depended of studio.api on the external-url
What is the new behavior?
The studio.api variable is set directly in the Load configuration.
Additional context
None