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

Store Odin resource configs in application database #521

Open
helrond opened this issue Aug 5, 2022 · 2 comments · May be fixed by #526
Open

Store Odin resource configs in application database #521

helrond opened this issue Aug 5, 2022 · 2 comments · May be fixed by #526
Assignees
Labels

Comments

@helrond
Copy link
Member

helrond commented Aug 5, 2022

Is your feature request related to a problem? Please describe.

There are a number of enumeration lists (in /transformer/resources/configs.py) which control values in source data. These lists are likely to be institution-specific. We want to limit editing of application code, while also providing an application that is highly configurable and can meet the needs of multiple institutions.

Describe the solution you'd like

Allow enumeration lists to be stored in the application database and managed via Django admin.

Describe alternatives you've considered

An alternative is to move these configs to the main application config file. However, that is less desirable because it adds a lot of weight to the application config, and is less transparent.

Additional context

We'll need to add some helpers to populate these lists with default values, and will need to update tests with fixtures.

@Tyl13
Copy link

Tyl13 commented Jan 26, 2024

Using the api provided by ASpace, could we automatically populate the database. For example, we could use a csv provided by ASpace which supplies us with the Enumeration code, Enumeration, Value code, and Value. More detail on idea for implementation with your specific changes are talked about within #526.

@helrond
Copy link
Member Author

helrond commented Jan 31, 2024

Hi @Tyl13 see my comments on #526

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

Successfully merging a pull request may close this issue.

2 participants