Skip to content

Release 0.51.2#2858

Closed
odlbot wants to merge 11 commits intoreleasefrom
release-candidate
Closed

Release 0.51.2#2858
odlbot wants to merge 11 commits intoreleasefrom
release-candidate

Conversation

@odlbot
Copy link
Contributor

@odlbot odlbot commented Jan 13, 2026

Jon Kafton

Anastasia Beglova

Chris Chudzicki

Ahtesham Quraish

Matt Bertrand

mbertrand and others added 11 commits January 9, 2026 15:34
* fix: change media url input behaviour

---------

Co-authored-by: Ahtesham Quraish <[email protected]>
* feat: add celery task for creating the news from article

---------

Co-authored-by: Ahtesham Quraish <[email protected]>
* Download digital credential dialog

* Missing icon

* Force long words to wrap

* Body as scroll container

* Prefetch from article API

* Render article detail with static render and provide banner and byline viewers

* Image viewer and placeholder

* Lockfile

* Extract learning resource IDs and prefetch

* Viewers for learning cards and divider

* Export viewer

* Fix/filter iframe props

* Style lint ignores

* Consolidate resource ID extraction and prefetch behavior

* Remove float, not in use

* Remove comments
@github-actions
Copy link

OpenAPI Changes

Show/hide 26 changes: 14 error, 6 warning, 6 info
26 changes: 14 error, 6 warning, 6 info
error	[response-property-became-optional] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/
		the response property 'results/items/video_url' became optional for the status '200'

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/
		removed the required property 'results/items/thumbnail_height' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/
		removed the required property 'results/items/thumbnail_url' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/
		removed the required property 'results/items/thumbnail_width' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/
		removed the required property 'results/items/youtube_id' from the response with the '200' status

error	[response-property-became-optional] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/{youtube_id}/
		the response property 'video_url' became optional for the status '200'

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/{youtube_id}/
		removed the required property 'thumbnail_height' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/{youtube_id}/
		removed the required property 'thumbnail_url' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/{youtube_id}/
		removed the required property 'thumbnail_width' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/{youtube_id}/
		removed the required property 'youtube_id' from the response with the '200' status

error	[new-required-request-parameter] at head/openapi/specs/v1.yaml	
	in API POST /api/v1/webhooks/video_shorts/
		added the new required 'query' request parameter 'video_metadata'

error	[new-required-request-property] at head/openapi/specs/v1.yaml	
	in API POST /api/v1/webhooks/video_shorts/
		added the new required request property 'video_metadata'

error	[new-required-request-property] at head/openapi/specs/v1.yaml	
	in API POST /api/v1/webhooks/video_shorts/
		added the new required request property 'video_metadata'

error	[new-required-request-property] at head/openapi/specs/v1.yaml	
	in API POST /api/v1/webhooks/video_shorts/
		added the new required request property 'video_metadata'

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/
		removed the optional property 'results/items/description' from the response with the '200' status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/{youtube_id}/
		removed the optional property 'description' from the response with the '200' status

warning	[request-parameter-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/v1/webhooks/video_shorts/
		deleted the 'query' request parameter 'youtube_metadata'
		This is a warning because some apps may return an error when receiving a parameter that they do not expect. It is recommended to deprecate the parameter first.

warning	[request-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/v1/webhooks/video_shorts/
		removed the request property 'youtube_metadata'

warning	[request-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/v1/webhooks/video_shorts/
		removed the request property 'youtube_metadata'

warning	[request-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/v1/webhooks/video_shorts/
		removed the request property 'youtube_metadata'

info	[response-optional-property-added] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/
		added the optional property 'results/items/thumbnail_large_url' to the response with the '200' status

info	[response-optional-property-added] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/
		added the optional property 'results/items/thumbnail_small_url' to the response with the '200' status

info	[response-required-property-added] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/
		added the required property 'results/items/video_id' to the response with the '200' status

info	[response-optional-property-added] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/{youtube_id}/
		added the optional property 'thumbnail_large_url' to the response with the '200' status

info	[response-optional-property-added] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/{youtube_id}/
		added the optional property 'thumbnail_small_url' to the response with the '200' status

info	[response-required-property-added] at head/openapi/specs/v0.yaml	
	in API GET /api/v0/video_shorts/{youtube_id}/
		added the required property 'video_id' to the response with the '200' status


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

@ChristopherChudzicki
Copy link
Contributor

The OpenAPI Diff failures are all to APIs for the feature-flagged / unreleased video sorts section, which currently doesn't use the video api anyway, right?

cc @mbertrand @jonkafton

@jonkafton
Copy link
Contributor

Closing to include #2851, which fixes a major editor bug.

@jonkafton jonkafton closed this Jan 15, 2026
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