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

Conversation2Measurements, global variables, generic variables, measurements, etc. #196

Merged
merged 6 commits into from
Apr 30, 2024

Conversation

mikepsinn
Copy link
Contributor

No description provided.

mikepsinn and others added 6 commits April 29, 2024 20:07
Added a new validation for global variables and improved the way user IDs are retrieved by creating a new dedicated utility function. The updated function, getUserId, replaces the prior use of getServerSession across multiple files. The new approach is expected to streamline the user authentication process and reduce code redundancy.
…ecording measurements, displaying charts, amongst others. Error handlers are also included if the CRUD operations fail. For better UX, loader spinners are added during fetch operations.
…lusively tied to 'UserVariable' to also catering 'GlobalVariable'. This refactoring aids in managing code organization and improves components' scalability by incorporating more generic variable types.
Components related to user variables have been refactored from 'userVariable' to 'genericVariables' or 'userVariables' to maintain consistency. The loading state for 'user-variable-overview' and 'generic-variable-list' has been modified to show a spinning icon for better user experience. Optional parameters with default values have been added to 'GenericVariableSearch'.
Copy link

vercel bot commented Apr 30, 2024

@mikepsinn is attempting to deploy a commit to the Crowdsourcing Cures Team on Vercel.

A member of the Team first needs to authorize it.

@mikepsinn mikepsinn merged commit b217262 into curedao:develop Apr 30, 2024
1 check failed
Copy link
Contributor

sweep-ai bot commented Apr 30, 2024

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.
  • Apply: Add docstrings to all functions and file headers.

This is an automated message generated by Sweep AI.

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

Successfully merging this pull request may close these issues.

1 participant