-
Notifications
You must be signed in to change notification settings - Fork 87
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: bump frontend-lib-content-components to 2.5.1 #1174
fix: bump frontend-lib-content-components to 2.5.1 #1174
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## open-release/redwood.master #1174 +/- ##
==============================================================
Coverage ? 92.32%
==============================================================
Files ? 708
Lines ? 12540
Branches ? 2690
==============================================================
Hits ? 11577
Misses ? 926
Partials ? 37 ☔ View full report in Codecov by Sentry. |
Sandbox deployment failed 💥 |
Sandbox deployment successful 🚀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @KristinAoki , just a couple notes. I'll test this with the PR sandbox in the meantime.
@@ -48,18 +52,32 @@ const courseUpdatesInitialValues = (requestType) => { | |||
}; | |||
|
|||
const renderComponent = ({ requestType }) => render( | |||
<IntlProvider locale="en"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these course-updates changes necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, because of the changes to the editor props. The editors for the course updates page use the same Tiny MCE editors
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gotcha, ty
Co-authored-by: Kyle McCormick <[email protected]>
Sandbox deployment successful 🚀 |
Sandbox deployment successful 🚀 |
Sandbox deployment successful 🚀 |
Quick tip, when bumping a package several version, you can link to whole set of changes by writing the URL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested all three editors, and this looks good to me. Thanks Kristin!
I also verified the image paste and insert fix fix that this PR will bring in 🚀
I am going to edit your commit message when I merge to explain that we are backporting this bump in order to remove the feedback link.
The primary purpose of this version bump backport is to remove the 2U feedback form link from the editors (openedx/frontend-lib-content-components#476), but several other improvements will also be pulled in: * Fix the Text editor so that when an image is added, it is added at the cursor, instead of the beginning of the component. * Improve editor load time by reducing API calls and switching some calls to be lazy. * Align controls better in the group feedback component. * Add validation for start & stop date fields. * Fix image handling bugs in both the raw & visual text editors. Full changelog: openedx/frontend-lib-content-components@v2.1.11...v2.5.1 Co-authored-by: Kyle McCormick <[email protected]>
Settings
Description
This is a backport to include the resolution of
frontend-lib-content-components
#476This upgrade includes the following version updates:
See
frontend-lib-content-components
tagged releases for more information on individual tags.