Skip to content

Use 569 digital collections update - #395

Merged
ehanson8 merged 2 commits into
mainfrom
USE-569-digital-collections-update
Aug 11, 2026
Merged

Use 569 digital collections update#395
ehanson8 merged 2 commits into
mainfrom
USE-569-digital-collections-update

Conversation

@ehanson8

Copy link
Copy Markdown
Contributor

Purpose and background context

Add digitalcollections as a source for indexing aliases.

How can a reviewer manually see the effects of these changes?

Review Digital Collections records in our Dev1 UI. Note: Reload the page if you get a 500 error since the Dev1 goes to sleep if unused.

Sample search for "medicine" in the Digital Collections tab

Includes new or updated dependencies?

YES

Changes expectations for external applications?

NO

What are the relevant tickets?

Code review

  • Code review best practices are documented here and you are encouraged to have a constructive dialogue with your reviewers about their preferences and expectations.

Why these changes are being introduced:
* TIMDEX needs to support Digital Collections as a source

How this addresses that need:
* Add Digital Collections to VALID_SOURCES in config.py

Side effects of this change:
* NA

Relevant ticket(s):
* https://mitlibraries.atlassian.net/browse/USE-569
@ehanson8
ehanson8 requested a review from a team as a code owner August 11, 2026 13:55

@ghukill ghukill left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Again, just mostly config updates, looks good to me.

@ehanson8
ehanson8 merged commit ea740a3 into main Aug 11, 2026
5 checks passed
@ehanson8
ehanson8 deleted the USE-569-digital-collections-update branch August 11, 2026 14:50
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.

2 participants