Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: fix incompatable emphasis #988

Merged
merged 2 commits into from
Oct 7, 2024
Merged

feat: fix incompatable emphasis #988

merged 2 commits into from
Oct 7, 2024

Conversation

kellyjosephprice
Copy link
Collaborator

@kellyjosephprice kellyjosephprice commented Oct 4, 2024

PR App Ref CX-1135

🧰 Changes

Attempts to migrate more buggy emphasis syntax.

🧬 QA & Testing

@kellyjosephprice kellyjosephprice marked this pull request as ready for review October 7, 2024 19:12
@kellyjosephprice kellyjosephprice merged commit 811fad4 into next Oct 7, 2024
12 of 13 checks passed
@kellyjosephprice kellyjosephprice deleted the fix/more-emphatic branch October 7, 2024 20:19
kellyjosephprice added a commit that referenced this pull request Oct 8, 2024
@rafegoldberg
Copy link
Contributor

This PR was released!

🚀 Changes included in v7.6.7

kellyjosephprice added a commit that referenced this pull request Oct 28, 2024
| 📖 [PR App][pr] | 🎫 Resolve CX-1135, RM-11085 |
| :-------------: | :-----------------: |

## 🧰 Changes

Fixes incompatable emphasis during the migration.

I [fixed this](#988) in
`@readme/mdx`, but I think it'll be nicer to keep the migration specific
stuff in the main app.

## 💁 Customer Impact

<!-- PLEASE DESCRIBE IF THIS CHANGE AFFECTS CUSTOMERS AND HOW -->
<!-- This is extremely useful for Tony and Marc writing the changelog!
-->

## 🧬 QA & Testing

- [ ] insert the string `*the recommended initial action is to**initiate
a[reversal operation
(rollback)](https://docs.jupico.com/reference/ccrollback)**. *` into a
page
- [ ] note the bold and italics
- [ ] migrate the project
- [ ] confirm that the bold and italics render the same in the dash and
hub

<!-- -->

- [Broken on next][next].
- [Working in this PR][pr]!

[next]: https://next.readme.ninja
[pr]: https://readme-pr-12963.readme.ninja
[ui]: https://readme-pr-12963.readme.ninja/ui

<!-- Uncomment and unescape this if you don't want a PR app! -->
<!-- \[skip preview\] -->
kellyjosephprice added a commit that referenced this pull request Oct 28, 2024
| 📖 [PR App][pr] | 🎫 Resolve CX-1135, RM-11085 |
| :-------------: | :-----------------: |

## 🧰 Changes

Fixes incompatable emphasis during the migration.

I [fixed this](#988) in
`@readme/mdx`, but I think it'll be nicer to keep the migration specific
stuff in the main app.

## 💁 Customer Impact

<!-- PLEASE DESCRIBE IF THIS CHANGE AFFECTS CUSTOMERS AND HOW -->
<!-- This is extremely useful for Tony and Marc writing the changelog!
-->

## 🧬 QA & Testing

- [ ] insert the string `*the recommended initial action is to**initiate
a[reversal operation
(rollback)](https://docs.jupico.com/reference/ccrollback)**. *` into a
page
- [ ] note the bold and italics
- [ ] migrate the project
- [ ] confirm that the bold and italics render the same in the dash and
hub

<!-- -->

- [Broken on next][next].
- [Working in this PR][pr]!

[next]: https://next.readme.ninja
[pr]: https://readme-pr-12963.readme.ninja
[ui]: https://readme-pr-12963.readme.ninja/ui

<!-- Uncomment and unescape this if you don't want a PR app! -->
<!-- \[skip preview\] -->
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants