You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently Records-REST can be configured with using the RECORDS_REST_ENDPOINTS variable with takes a long list of arguments.
Problem:
We are adding more and more options to the list of configuration variables. Each time an option is added, it needs to be propagated down through views. This causes quite a lot of boiler plate code, and possibilities for improper configuration.
Also, it is hard to reuse and repurpose the Records REST API for e.g. the deposit API.
Proposal:
Wrap all configuration options in an object, so that just one object is passed around
Create a endpoint context for each request, so it's easier to override behaviour in e.g. deposit.
Currently Records-REST can be configured with using the
RECORDS_REST_ENDPOINTS
variable with takes a long list of arguments.Problem:
We are adding more and more options to the list of configuration variables. Each time an option is added, it needs to be propagated down through views. This causes quite a lot of boiler plate code, and possibilities for improper configuration.
Also, it is hard to reuse and repurpose the Records REST API for e.g. the deposit API.
Proposal:
Co-authored-by: @jirikuncar
The text was updated successfully, but these errors were encountered: