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

[blog] New posts #22607

Merged
merged 9 commits into from
Sep 16, 2020
Merged

[blog] New posts #22607

merged 9 commits into from
Sep 16, 2020

Conversation

oliviertassinari
Copy link
Member

No description provided.

@mui-pr-bot
Copy link

mui-pr-bot commented Sep 15, 2020

Details of bundle changes

Generated by 🚫 dangerJS against ff6602a

oliviertassinari and others added 2 commits September 15, 2020 19:08
Co-authored-by: damien tassone <damien.tassone@gmail.com>
Co-authored-by: damien tassone <damien.tassone@gmail.com>
docs/pages/blog/marija-najdova-joining.md Outdated Show resolved Hide resolved
},
},
});
```
Copy link
Member

Choose a reason for hiding this comment

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

Personally I'd leave the code for the documentation.

Copy link
Member

Choose a reason for hiding this comment

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

+1 we already have the link in the text, if people are interested they can take a look :)

Copy link
Member Author

@oliviertassinari oliviertassinari Sep 16, 2020

Choose a reason for hiding this comment

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

My intention was to make the post more "vivid" by introducing some content that isn't text. Code in this case.

docs/pages/blog/marija-najdova-joining.md Outdated Show resolved Hide resolved
docs/pages/blog/marija-najdova-joining.md Outdated Show resolved Hide resolved
docs/pages/blog/marija-najdova-joining.md Outdated Show resolved Hide resolved
docs/pages/blog/spotlight-damien-tassone.md Outdated Show resolved Hide resolved
docs/pages/blog/spotlight-damien-tassone.md Outdated Show resolved Hide resolved
docs/pages/blog/spotlight-damien-tassone.md Outdated Show resolved Hide resolved
docs/pages/blog/spotlight-damien-tassone.md Outdated Show resolved Hide resolved
docs/pages/blog/spotlight-damien-tassone.md Outdated Show resolved Hide resolved
oliviertassinari and others added 3 commits September 15, 2020 21:52
Co-authored-by: Matt <github@nospam.33m.co>
Co-authored-by: Matt <github@nospam.33m.co>
docs/pages/blog/marija-najdova-joining.md Outdated Show resolved Hide resolved
docs/pages/blog/marija-najdova-joining.md Outdated Show resolved Hide resolved
Co-authored-by: Marija Najdova <mnajdova@gmail.com>
@oliviertassinari oliviertassinari merged commit 85615a2 into mui:next Sep 16, 2020
@oliviertassinari oliviertassinari deleted the blog-new-members branch September 16, 2020 11:52
oliviertassinari added a commit that referenced this pull request Sep 16, 2020
@zannager zannager added the blog label Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants