Skip to content
New issue

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

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

Already on GitHub? # 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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant