Health-Informatics-UoN / Carrot-Mapper

Carrot: Convenient And Reusable Rapid Omop Transformer.
https://carrot4omop.ac.uk
MIT License
12 stars 3 forks source link

💡 Change wording of SR list view filter box #707

Closed spco closed 1 month ago

spco commented 1 month ago

Is there an existing issue for this?

Is your proposal related to a problem or functionality gap?

No response

Describe your proposal

Please consider changing the wording in the filter box (currently "Filter by dataset") to more closely fit the true function, which is to filter on the "name" column, not the "dataset" column. I appreciate that this is somewhat confusing on the BE given the naming of fields in the ScanReport model 😆

Describe alternatives you've considered

No response

I'm part of a Project Team

No response

Anything else?

No response

Are you willing to contribute to developing this feature?

None

AndyRae commented 1 month ago

Yep this is due to the backend model which has to reflected in the API. But the fix is to be able to override this in the data table component with a prop. Should be straightforward to fix.