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

Correct highlighted lines and wrap <comment>s in React fragment in tutorial chapter 6 #5058

Merged
merged 5 commits into from
May 4, 2022

Conversation

Philzen
Copy link
Contributor

@Philzen Philzen commented Apr 5, 2022

PR Name says it all – see the commit messages for explanation and references.

Regarding e597b60 – this root cause this currently doesn't need to be highlighted is #5060, which i believe may be unwanted behaviour. So i may as well drop that commit again.

@netlify
Copy link

netlify bot commented Apr 5, 2022

Deploy Preview for redwoodjs-docs ready!

Name Link
🔨 Latest commit ee349aa
🔍 Latest deploy log https://app.netlify.com/sites/redwoodjs-docs/deploys/62723bfe049bdf0008545de0
😎 Deploy Preview https://deploy-preview-5058--redwoodjs-docs.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 settings.

@Philzen
Copy link
Contributor Author

Philzen commented Apr 5, 2022

@jtoar In relation to #5054 it can also be observed in this tutorial chapter that there is a mix of imports – sometimes it's @redwoodjs/testing, and sometimes it is @redwoodjs/testing/web ... Guess this should be harmonized at some point.

@Philzen Philzen force-pushed the minor-tutorial-chapter-6-fixes branch from 55b3543 to d44c793 Compare April 5, 2022 22:56
@jtoar jtoar self-requested a review April 9, 2022 05:32
@jtoar jtoar self-assigned this Apr 15, 2022
@jtoar jtoar added release:docs This PR only updates docs and removed topic/docs labels May 4, 2022
jtoar added 2 commits May 4, 2022 17:37
Going to add this back for now since it's used in the example
@jtoar jtoar enabled auto-merge (squash) May 4, 2022 08:40
@jtoar jtoar merged commit 174673f into redwoodjs:main May 4, 2022
@jtoar jtoar modified the milestones: next-release, v1.4.0 May 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:docs This PR only updates docs
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants