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

Release 0.28.1 #1986

Open
wants to merge 9 commits into
base: release
Choose a base branch
from
Open

Release 0.28.1 #1986

wants to merge 9 commits into from

Conversation

odlbot
Copy link
Contributor

@odlbot odlbot commented Jan 24, 2025

Carey P Gumaer

Chris Chudzicki

Shankar Ambady

Matt Bertrand

mbertrand and others added 9 commits January 21, 2025 14:39
* add "videos in this series" carousel

* add "other videos in this series" carousel

* set limit for all carousels to 12

* exclude the current resource

* fix tests

* fix issue causing flaky test results

* change variable name
* refactor carousel building

* add "other episodes in this podcast" and "episodes in this podcast" carousels

* title should be "Recent Episodes"
* remove the v1 drawer and rename v2 drawer

* remove commented out code
* adding a setting to be able to override the chunk size when embedding

* adding collection name override param

* updating spec

* adding test

* added test. switched back to tiktokenizer with chunk size param

* fixing test

* fix test

* catching exception

* fixing tests
* set @mitodl/smoot-design versions

* Remove ol-components ThemeProvider from test files

* switch to smoot-design themeprovider

* remove unused props

* replace Button, ActionButton usages

* bump smoot-design

* remove theme workaround, mui issue fixed

* remove button

* add a linting rule redirecting usage

* try installing inside docker

* fix testimonial button background

* update smoot (no unstable variants)

* bump smoot to npm version

* re-delete v1 drawer
* drawer topic carousels should not be filtered on just courses

* rename argument
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants