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

Add application assessment #52

Merged
merged 10 commits into from
Jan 3, 2025
Merged

Add application assessment #52

merged 10 commits into from
Jan 3, 2025

Conversation

niclasheun
Copy link
Contributor

This pull request introduces several enhancements and refactoring to the application assessment feature in the core client. The changes include adding new components, updating existing ones, and modifying the network layer to support new functionalities.

Enhancements to Application Assessment:

  • Added AssessmentCard component to handle score and comment submissions for application assessments. (clients/core/src/ApplicationAdministration/ApplicationAssessment/components/AssessmentCard.tsx)
  • Updated ApplicationDetailsView to include new methods for handling score, comment, and acceptance status changes, and integrated the AssessmentCard component. (clients/core/src/ApplicationAdministration/ApplicationAssessment/ApplicationDetailsView.tsx) [1] [2] [3] [4]

Network Layer Updates:

  • Created postApplicationAssessment function to handle assessment submissions. (clients/core/src/network/mutations/postApplicationAssessment.ts)
  • Added getApplicationParticipations function to fetch application participations. (clients/core/src/network/queries/applicationParticipations.ts)

Refactoring and Code Simplification:

  • Replaced course_phase_participations with application_participations in various components to align with new data structure. (clients/core/src/ApplicationAdministration/ApplicationAssessment/ApplicationsOverview.tsx, clients/core/src/ApplicationAdministration/ApplicationAssessment/components/MissingUniversityData.tsx) [1] [2] [3] [4] [5] [6]

UI Enhancements:

  • Added Toaster component for displaying toast notifications. (clients/core/src/App.tsx, clients/shared_library/components/ui/toast.tsx) [1] [2] [3]

These changes collectively improve the functionality and user experience of the application assessment feature, making it more robust and user-friendly.

@niclasheun niclasheun self-assigned this Jan 3, 2025
@niclasheun niclasheun merged commit 23fbdbe into main Jan 3, 2025
6 checks passed
@niclasheun niclasheun deleted the add-application-assessment branch January 3, 2025 16:54
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