Closed yiwen101 closed 7 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
a265c73
) 48.87% compared to head (ac5e734
) 48.87%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Hi @yiwen101 thank you for the changes! Some more formatting errors and some minor nits and we should be good to go :)
Thank you so much for the meticulous review, I feel so bad forcing you pointing out so many formatting issues, especially during the CNY holiday. I have made the requested adjustment (and some other minor changes). Thank you for your reviews and help again.
What is the purpose of this pull request?
Resolves #2371
**Overview of changes: Add the rule of using markdown in presentation component in "trouble shooting" and "presentation components" section of the user doc. Update the code example of the four presentation component
Anything you'd like to highlight/discuss: NA
Testing instructions: NA
Proposed commit message: (wrap lines at 72 characters) Update user doc on markdown syntax for presentation component
Checklist: :ballot_box_with_check:
Reviewer checklist:
Indicate the SEMVER impact of the PR:
At the end of the review, please label the PR with the appropriate label:
r.Major
,r.Minor
,r.Patch
.Breaking change release note preparation (if applicable):