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

dynamic forms that load flexible metadata #6844

Closed
wants to merge 43 commits into from

Conversation

orangewolf
Copy link
Member

Summary

Adds dynamic dataloading to form object

@samvera/hyrax-code-reviewers

ShanaLMoore and others added 24 commits June 4, 2024 10:59
This work solely focused on the UI elements needed to build out the ability to import a metadat profile. This heavily relies on backend functionality to complete.
This commit will relocate the Metadata Profiles link to Repository
Content.  We also refactor the modal to make it show up better.
Options were loaded in the modal previously. When this logic was
moved into a controller, it became confusing due to other definitions
of `available_admin_sets`.

Additionally, add a validation to ensure that a work type is selected.
We need a nil guard for @admin_sets_for_select in case other partials
use the @admin_sets_for_select instance variable.
@orangewolf orangewolf added the notes-minor Release Notes: Non-breaking features label Jun 24, 2024
orangewolf and others added 8 commits June 24, 2024 17:16
In this commit we updated code to support dynamic rendering for attribute rows
This commit will add a few different conditions for populating the
label.  If the label is a string we'll simply set that as the label.

ex. `label: 'My Label'`

If the label is a hash we'll look for the key that matches the current
locale.

ex. `label: { en: 'My Label', es: 'Mi Etiqueta' }`

If the locale is not found it will fall back to en as the default, but
if en isn't found it will just use the first value in the hash.

ex. `label: { es: 'Mi Etiqueta', fr: 'Mon Étiquette' }`
@kirkkwang kirkkwang closed this Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notes-minor Release Notes: Non-breaking features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants