Skip to content

Conversation

@tonygravell
Copy link
Contributor

@tonygravell tonygravell commented Jun 1, 2024

In this PR:

  • Added optional setting to ProjectSettings.ts to allow configurable host baseApiUrl
  • Updated files which require a dynamic host variable when creating JSDocs

Addresses issue:
#42361

@mui-bot
Copy link

mui-bot commented Jun 1, 2024

Netlify deploy preview

https://deploy-preview-42472--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against f619823

Copy link
Member

@alexfauquette alexfauquette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks solid 🚀

tonygravell and others added 3 commits June 3, 2024 16:51
Co-authored-by: Alexandre Fauquette <45398769+alexfauquette@users.noreply.github.com>
Signed-off-by: Anthony Gravell <73857273+tonygravell@users.noreply.github.com>
Co-authored-by: Alexandre Fauquette <45398769+alexfauquette@users.noreply.github.com>
Signed-off-by: Anthony Gravell <73857273+tonygravell@users.noreply.github.com>
@tonygravell tonygravell marked this pull request as ready for review June 5, 2024 19:55
@tonygravell tonygravell requested a review from alexfauquette June 5, 2024 19:55
@alexfauquette alexfauquette requested a review from a team June 5, 2024 20:03
translationPagesDirectory: 'docs/translations/api-docs-base',
generateClassName: generateUtilityClass,
isGlobalClassName: isGlobalState,
// #major-version-switch
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mnajdova is #major-version-switch the correct flag? I don't know if we have another flag for it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#default-branch-switch is much more used, honestly first time I see the #major-version-switch :)

Copy link
Member

@alexfauquette alexfauquette Jun 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Come from here. I will harmonize it 👍

@oliviertassinari oliviertassinari added type: bug It doesn't behave as expected. docs Improvements or additions to the documentation. package: material-ui scope: docs-infra Involves the docs-infra product (https://www.notion.so/mui-org/b9f676062eb94747b6768209f7751305). labels Jun 5, 2024
@oliviertassinari oliviertassinari changed the title [material-ui] Added configurable jsdoc host variable [docs-indra] Add configurable jsdoc host variable Jun 5, 2024
@mnajdova mnajdova changed the title [docs-indra] Add configurable jsdoc host variable [docs-infra] Add configurable jsdoc host variable Jun 6, 2024
Copy link
Member

@mnajdova mnajdova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👌, thanks for the contribution! I will let @alexfauquette do the last review.

Copy link
Member

@alexfauquette alexfauquette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good for me, I just wanted the confirmation from core team, we are not bothering them with those diff ;)

@alexfauquette alexfauquette merged commit 8a741e4 into mui:next Jun 7, 2024
joserodolfofreitas pushed a commit to joserodolfofreitas/material-ui that referenced this pull request Jul 29, 2024
Signed-off-by: Anthony Gravell <73857273+tonygravell@users.noreply.github.com>
Co-authored-by: Tony Gravell <tonygravell@Tonys-Mac-mini.local>
Co-authored-by: Alexandre Fauquette <45398769+alexfauquette@users.noreply.github.com>
Co-authored-by: alexandre <alex.fauquette@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to the documentation. scope: docs-infra Involves the docs-infra product (https://www.notion.so/mui-org/b9f676062eb94747b6768209f7751305). type: bug It doesn't behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants