Skip to content

fix(marcframe): ContributionByRoleStep missing required resources#1748

Merged
olovy merged 1 commit intodevelopfrom
bugfix/contribution-by-role-step
Apr 16, 2026
Merged

fix(marcframe): ContributionByRoleStep missing required resources#1748
olovy merged 1 commit intodevelopfrom
bugfix/contribution-by-role-step

Conversation

@olovy
Copy link
Copy Markdown
Contributor

@olovy olovy commented Apr 16, 2026

ContributionByRoleStep missing required resources

relatorResources was not initialized because elasticFind wasn't yet initialized. Because elastic initialization was moved in cf91b20

Guarded because when running only core whelk elastic+elasticFind are not available.

relatorResources = whelk.elasticFind ? new ResourceCache.RelatorResources(

relatorResources was not initialized because elasticFind wasn't yet
initialized. Because elastic initialization was moved in cf91b20
Copy link
Copy Markdown
Contributor

@kwahlin kwahlin left a comment

Choose a reason for hiding this comment

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

LGTM

@olovy olovy merged commit c468254 into develop Apr 16, 2026
1 check passed
@olovy olovy deleted the bugfix/contribution-by-role-step branch April 16, 2026 08:09
Copy link
Copy Markdown
Member

@niklasl niklasl left a comment

Choose a reason for hiding this comment

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

🚢

olovy added a commit that referenced this pull request Apr 16, 2026
)

relatorResources was not initialized because elasticFind wasn't yet
initialized. Because elastic initialization was moved in cf91b20
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.

3 participants