-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve feature/pass-spec-directly #356
base: feature/pass-spec-directly
Are you sure you want to change the base?
Improve feature/pass-spec-directly #356
Conversation
|
@serut is attempting to deploy a commit to the Rohit Gohri's projects Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry of the (very) late review and thank you for your continued contributions. I will try to go through my draft PR with your changes and release it soon
@@ -20,7 +16,7 @@ function Redoc(props: RedocProps): JSX.Element { | |||
const { className, optionsOverrides, url, themeId } = props; | |||
const { options } = useSpecOptions(themeId, optionsOverrides); | |||
|
|||
if (getIsExternalUrl(url)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is going to completely disable SSR for anyone using url, if that is what you want then we can just add that as an option I think rather than changing behavior for existing users.
We won't need downloadSpecUrl
option, just a ssr
option, by default true
but which can be disabled to always use client side rendering and skip the bundling logic
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Allright I will try to do that change
This MR fixes #347
getIsExternalUrl
functionthemeId
toApiSchema
downloadSpecUrl
property toRedocProps
and co, to separate the behavior fromurl
that was doing 2 different things