Within this issue we will face different FE code smells and bad practices:
Use the same naming convention over all components
Remove unused interfaces
Add Utils functions to avoid code repetition
Avoid nested conditions: error handling should be on top of functions
Remove ref/reactive where we do not need reactivity
Move all study status / permission / access checks at the top of a component to be consistent over all components
Remove unused css classes
Remove/Replace style tag, when there is a corresponding tailwind class
Rewrite dynamic class conditions to avoid empty class attribute on html tags
Replace variable and string concatenation (+) and use template literals instead
Reduce complexity in functions and conditions
Changing the collaborator dialog componente if you want to add a member to an existing study. Since it is only possible to assign one role (admin, operator or viewer), I changed the MultiSelect to RadioButtons to be semantically correct.
Within this issue we will face different FE code smells and bad practices:
Changing the collaborator dialog componente if you want to add a member to an existing study. Since it is only possible to assign one role (admin, operator or viewer), I changed the MultiSelect to RadioButtons to be semantically correct.