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

Precedence of config over the metadata #2282

Open
1 task done
ychoquet opened this issue Jun 19, 2024 · 0 comments
Open
1 task done

Precedence of config over the metadata #2282

ychoquet opened this issue Jun 19, 2024 · 0 comments

Comments

@ychoquet
Copy link
Member

Is there an existing issue for this?

  • I have searched the existing issues

Who request this Feature

Team

User Story

The current implementation uses the defaultsDeep function of loadash to implement the precedence principles of the viewer. It starts from the user data and replaces undefined fields with the metadata and then the defaults..

The new implementation must reverse the strategy. We want to replace the lodash defaultsDeep function with merge and start with the default config and use the metadata config to overwrite the fields in the default config to replace them with the metadata values and then do the same to replace the default+metadata fields using the user config.

Implementation UI

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant