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

Update block-content.md [Named blocks] #1958

Merged
merged 4 commits into from
Apr 18, 2024

Conversation

Shishouille
Copy link
Contributor

Aaah, I just saw #1894, I hope it's still ok! Some people requested more documentation about named blocks, so here's another proposition 😃

Add documentation for named blocks
@netlify
Copy link

netlify bot commented Aug 24, 2023

Deploy Preview for ember-guides ready!

Name Link
🔨 Latest commit 9f10eff
🔍 Latest deploy log https://app.netlify.com/sites/ember-guides/deploys/6620e30bd9b9f40008bb3154
😎 Deploy Preview https://deploy-preview-1958--ember-guides.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Co-authored-by: Jordan Hawker <hawker.jordan@gmail.com>
Copy link
Member

@IgnaceMaes IgnaceMaes left a comment

Choose a reason for hiding this comment

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

This is GREAT!

I reviewed the other linked PR first but my suggestions there would basically mean writing something like this. The code examples make a lot of sense.

I've put a couple of suggestions. If those are resolved this is good to merge.

Sorry you had to wait so long for a review, that's not ideal.

Copy link
Member

@IgnaceMaes IgnaceMaes left a comment

Choose a reason for hiding this comment

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

As the suggestions were mostly stylistic, I've taken the liberty to just apply them so we can get this through.

Thank you for this great contribution! 🎉

@IgnaceMaes IgnaceMaes merged commit 15fc152 into ember-learn:master Apr 18, 2024
4 checks passed
This was referenced Apr 18, 2024
@Shishouille Shishouille deleted the Shishouille-patch-1 branch April 18, 2024 15:34
@Shishouille
Copy link
Contributor Author

No worries! And you're welcome 😃 I'm glad it got merged 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants