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

fix: single backtick rendering as a codeblock in casestudies #2208

Merged
merged 8 commits into from
Dec 7, 2023

Conversation

karthikmurakonda
Copy link
Contributor

@karthikmurakonda karthikmurakonda commented Oct 5, 2023

Description
Because of MDX v2 breaking changes here, there is no difference between single backtick(inlineCode is no more present) and triple backtick in the jsx output. Both give <code> tag as shown here

I followed their migration guide to place codeblock component on seeing the pre tag itself.

Related issue(s)
Fixes #2188

@netlify
Copy link

netlify bot commented Oct 5, 2023

Deploy Preview for asyncapi-website ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 08ed137
🔍 Latest deploy log https://app.netlify.com/sites/asyncapi-website/deploys/65720dec7de26a00080d2777
😎 Deploy Preview https://deploy-preview-2208--asyncapi-website.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.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

@github-actions
Copy link

github-actions bot commented Oct 5, 2023

⚡️ Lighthouse report for the changes in this PR:

Category Score
🔴 Performance 31
🟢 Accessibility 98
🟢 Best practices 100
🟢 SEO 100
🔴 PWA 30

Lighthouse ran on https://deploy-preview-2208--asyncapi-website.netlify.app/

components/MDX.js Show resolved Hide resolved
@akshatnema
Copy link
Member

Looks Good to me. @derberg Can you have a look, if we are missing out some MDX somewhere?

@karthikmurakonda
Copy link
Contributor Author

@akshatnema, can I get the Hacktoberfest accepted label? If everything is alright...

Copy link
Member

@derberg derberg left a comment

Choose a reason for hiding this comment

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

all good from my side
most important the fix is done in https://deploy-preview-2208--asyncapi-website.netlify.app/casestudies/adeogroup

@karthikmurakonda thanks so much for the fix!

@derberg derberg added the hacktoberfest-accepted PRs which are accepted under Hacktoberfest label Oct 31, 2023
@derberg
Copy link
Member

derberg commented Oct 31, 2023

@akshatnema your approval is also necessary as in past you requested changes

@asyncapi-bot
Copy link
Contributor

⚡️ Lighthouse report for the changes in this PR:

Category Score
🔴 Performance 35
🟢 Accessibility 98
🟢 Best practices 92
🟢 SEO 100
🔴 PWA 33

Lighthouse ran on https://deploy-preview-2208--asyncapi-website.netlify.app/

@akshatnema
Copy link
Member

/rtm

@asyncapi-bot asyncapi-bot merged commit 1a78f60 into asyncapi:master Dec 7, 2023
17 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted PRs which are accepted under Hacktoberfest ready-to-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Single backtick is recognised as Codeblock
4 participants