Skip to content
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

[docs] Sync static images from core repository #12525

Merged
merged 3 commits into from Mar 22, 2024

Conversation

LukasTy
Copy link
Member

@LukasTy LukasTy commented Mar 21, 2024

Fixes missing resources on docs preview deploys.

@LukasTy LukasTy added the docs Improvements or additions to the documentation label Mar 21, 2024
@LukasTy LukasTy self-assigned this Mar 21, 2024
@mui-bot
Copy link

mui-bot commented Mar 21, 2024

Deploy preview: https://deploy-preview-12525--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against f552f59

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 like those two are unused. At least I don't see them used in the material-ui repo

@danilo-leal could it be WIP images that has not be deleted?

  • marblism-square.png
  • marblism-rectangular.svg

In production we don't need it, because the path mui.com/static/ leads to the content of the core repo. But I've no idea about how we could reproduce this behavior in preview

@danilo-leal
Copy link
Contributor

All good for me. I'd keep the svgs, though, and just take out the png for the Marablism logo.

@LukasTy
Copy link
Member Author

LukasTy commented Mar 21, 2024

Thanks, I've removed the PNGs added in this PR as they did not seem to be used. 🙈 🤷

@LukasTy LukasTy merged commit 2722f3d into mui:master Mar 22, 2024
17 checks passed
@LukasTy LukasTy deleted the sync-static-folder branch March 22, 2024 12:18
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants