Skip to content

Update embed schemas for newly required fields#170

Merged
bjester merged 3 commits into
learningequality:mainfrom
bjester:ca-schema-update
Jun 5, 2025
Merged

Update embed schemas for newly required fields#170
bjester merged 3 commits into
learningequality:mainfrom
bjester:ca-schema-update

Conversation

@bjester

@bjester bjester commented Jun 4, 2025

Copy link
Copy Markdown
Member

Summary

  • Adds mandatory channel_id to the schemas, particularly important for the content embedding requests
  • Ensured all IDs are now enforced as UUIDs (with dashes), which will require updates in Studio and CA API to add / remove dashes
  • Enforces language is passed in topic embedding requests, and removes duplicitous ancestor definition to prioritize the one definition on the target topic

References

closes #158

Reviewer guidance

Test updates make sense?

@bjester bjester requested review from akolson and ozer550 June 4, 2025 17:20

@akolson akolson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look correct to me. Thanks @bjester

@bjester bjester merged commit f01c1e1 into learningequality:main Jun 5, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update required fields in CA API schemas

2 participants