Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Manifest metadata labels generation using AllinsonFlex is not context-aware #378

Open
aploshay opened this issue Nov 13, 2024 · 0 comments

Comments

@aploshay
Copy link

Story

The logical flow for building manifest metadata using AllinsonFlex currently does not allow for a context-aware reading of the dynamic schema, which leads to the default labels always being used when context-specific ones may be available and applicable:

There's a missed opportunity here to use a dynamic schema lookup to get context-aware label values, either by allowing to pass in fields: at the start of the stack, or having .allinson_flex_fields optionally receive a work identifier to allow looking up the dynamic schema applicable to the work instead of directly querying AllinsonFlex::ProfileProperty records.

See this PR for a workaround applied to an impacted project:
IU-Libraries-Joint-Development/essi#636

Acceptance Criteria

  • Metadata labels in the manifest should be consistent with context-aware flexible metadata labels on the same work's Show page
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

No branches or pull requests

1 participant