Skip to content

Conversation

@shanbady
Copy link
Contributor

@shanbady shanbady commented Nov 4, 2025

What are the relevant tickets?

Related to item missed in previous PR and https://github.com/mitodl/hq/issues/9056

Description (What does it do?)

This PR adds the scheduled task to update the topics collection which was missed in a previous pr.

How can this be tested?

  1. checkout this branch
  2. run the task manually and see that it succeeds:
from vector_search.tasks import sync_topics
sync_topics()

@shanbady shanbady added the Needs Review An open Pull Request that is ready for review label Nov 4, 2025
@shanbady shanbady marked this pull request as ready for review November 4, 2025 15:24
@shanbady shanbady changed the title adding topic sync task Add periodic topic collection sync task Nov 4, 2025
@mbertrand mbertrand self-assigned this Nov 6, 2025
@github-actions
Copy link

github-actions bot commented Nov 6, 2025

OpenAPI Changes

Show/hide 3 changes: 0 error, 0 warning, 3 info
3 changes: 0 error, 0 warning, 3 info
info	[request-parameter-property-type-generalized] 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 generalized from 'integer'/'' to 'number'/''

info	[request-parameter-property-type-generalized] 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 generalized from 'integer'/'' to 'number'/''

info	[request-parameter-property-type-generalized] 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 generalized from 'integer'/'' to 'number'/''


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

Copy link
Member

@mbertrand mbertrand left a comment

Choose a reason for hiding this comment

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

👍

@mbertrand mbertrand added Waiting on author and removed Needs Review An open Pull Request that is ready for review labels Nov 6, 2025
@shanbady shanbady merged commit 7807fe5 into main Nov 6, 2025
12 checks passed
@shanbady shanbady deleted the shanbady/scheduled-task-for-topic-update branch November 6, 2025 15:44
This was referenced Nov 6, 2025
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.

3 participants