Release 0.71.16#3531
Closed
odlbot wants to merge 10 commits into
Closed
Conversation
* remove semantic chunker * removed deprecated langchain-experimental package * fix tests * remove unused setting
…3512) Program certificates on MIT Learn now display the CMS certificate page's product_name ("Certificate Title") as the title, gated behind the `cms-certificate-title` PostHog flag. When the flag is off (or product_name is empty) it falls back to the program title — today's behaviour. Part of mitodl/hq#11907.
* reove product page feature flag * Use resource.url for MITx Online CTA links instead of client-side rewrite CallToActionSection rewrote MITx Online course/program URLs to Learn product-page paths client-side (coursePageView/programPageView). The backend MITx Online ETL now populates resource.url with those same Learn product URLs (gated on has_product_page), so the rewrite is a duplicate. Drop the MITx Online branch in getResourceUrl and let those resources fall through to resource.url like every other resource. The result is behavior-preserving for published resources (same path, same-origin so still internal/no-UTM and client-navigated), and resources without a product page now correctly get no link rather than a dead product-page path derived from readable_id. ProgramAsCourseCard keeps its programPageView rewrite: its data comes straight from the MITx Online dashboard API and has no resource.url. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* Add card type text * remove chevron from org cards * add end date display and tests for it * fix separator rendering on compact cards * match by test id * gate end date section on hasEnded * Remove old start date countdown and consolidate with end date indicator into one "course date indicator" * consolidate date popover and date indicator into one component with the indicator as a trigger and reorganize utility functions from cardshared * fix mobile alignment of CTA * fix cta alignment in unenrolled course cards * add a threshold to formatCalendarDays that retuns a short date string from Intl if the day is more than 90 days away in either direction * fix courseDateText null check * switch date popover trigger to a button instead of link and add aria attributes * add screen reader readable aria labels to all CTA's in the dashboard * fix tests
OpenAPI ChangesNo changes detected Unexpected changes? Ensure your branch is up-to-date with |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Shankar Ambady
Matt Bertrand
Danielle Frappier
Carey P Gumaer
Chris Chudzicki
Zaman Afzal