Skip to content

Conversation

@odlbot
Copy link
Contributor

@odlbot odlbot commented Nov 6, 2025

Carey P Gumaer

Shankar Ambady

Jon Kafton

Chris Chudzicki

ChristopherChudzicki and others added 7 commits November 4, 2025 14:38
* use custom django filter extension to get explode: true

* simplify an assertion

* clarify docstring

* simplify filtering

* regenerate spec
* fix CTA text logic

* add a test that checks for view course cta text when end date is in the past and course is not completed
* Headings on cards for screen reader navigation

* Add tests

* Stray character

* Styles not in use

* Remove console.log

* Style lint

* Refactor to LinkableTitle component

* Move heading level selection into ResourceCard for reuse in SearchDisplay

* Set heading level
* adding topic sync task

* fixing task name
* update mitxonline api package

* add display of welcome message

* fix existing tests broken by API package update and add some new ones

* fix more type checking issues

* simplify null contract check and fix spacing issue when the contract welcome message is null or missing
@github-actions
Copy link

github-actions bot commented Nov 6, 2025

OpenAPI Changes

Show/hide 3 changes: 3 error, 0 warning, 0 info
3 changes: 3 error, 0 warning, 0 info
error	[request-parameter-property-type-specialized] at head/openapi/specs/v1.yaml	
	in API GET /api/v1/contentfiles/
		for the 'query' request parameter 'resource_id', the type/format of property '/items/' was specialized from 'number'/'' to 'integer'/''

error	[request-parameter-property-type-specialized] at head/openapi/specs/v1.yaml	
	in API GET /api/v1/courses/{learning_resource_id}/contentfiles/
		for the 'query' request parameter 'resource_id', the type/format of property '/items/' was specialized from 'number'/'' to 'integer'/''

error	[request-parameter-property-type-specialized] at head/openapi/specs/v1.yaml	
	in API GET /api/v1/learning_resources/{learning_resource_id}/contentfiles/
		for the 'query' request parameter 'resource_id', the type/format of property '/items/' was specialized from 'number'/'' to 'integer'/''


Unexpected changes? Ensure your branch is up-to-date with main (consider rebasing).

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.

6 participants