-
Notifications
You must be signed in to change notification settings - Fork 0
Development: Improve interview slots management for professors
#1789
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
base: main
Are you sure you want to change the base?
Conversation
|
@Abi107717 Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details. |
|
@Abi107717 Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details. |
|
🤖 OpenAPI spec and client code auto-updated and committed. |
|
@Abi107717 Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details. |
|
🤖 No OpenAPI or client changes needed. |
- Apply disabled styles (greyed out) for past slots in UI - Hide edit/delete/assign actions for past slots
|
@Abi107717 Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details. |
|
🤖 No OpenAPI or client changes needed. |
|
@Abi107717 Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details. |
|
🤖 No OpenAPI or client changes needed. |
|
@Abi107717 Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details. |
|
🤖 No OpenAPI or client changes needed. |
|
@Abi107717 Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details. |
|
🤖 No OpenAPI or client changes needed. |
|
@Abi107717 Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details. |
|
@Abi107717 Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details. |
|
🤖 No OpenAPI or client changes needed. |
Bofan-Zhu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for improving this 😊. I left some comments.
Functionality looks good to me
...ain/webapp/app/interview/interview-process-detail/slots-section/slots-section.component.html
Outdated
Show resolved
Hide resolved
...ain/webapp/app/interview/interview-process-detail/slots-section/slots-section.component.html
Outdated
Show resolved
Hide resolved
...ain/webapp/app/interview/interview-process-detail/slots-section/slots-section.component.html
Outdated
Show resolved
Hide resolved
...bapp/app/interview/interview-process-detail/slots-section/slot-card/slot-card.component.html
Show resolved
Hide resolved
| """ | ||
| SELECT COUNT(s) FROM InterviewSlot s | ||
| WHERE s.interviewProcess.id = :processId | ||
| AND s.startDateTime > :now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think hibernate provides CURRENT_TIMESTAMP or something similar if I'm not mistaken
|
@Abi107717 Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details. |
|
🤖 No OpenAPI or client changes needed. |
Checklist
General
Server
Client
Motivation and Context
Currently, it is possible to send invitations to more applicants than there are available interview slots, which leads to booking issues.
Description
This PR improves the invitation flow by adding a warning dialog if the number of available future slots is insufficient for the selected applicants. It integrates the empty slot state directly into the calendar view for a cleaner UI and refactors the slot creation trigger to use Angular Signals. Past slots are now visually distinguished (greyed out), and the "Add Slots" button visibility is optimized.
Steps for Testing
Prerequisites:
Log in to TumApply as professor
Navigate to an Interview Process with some uncontacted applicants. (
interviews/overview)Scenario 1: Insufficient Slots Warning
Scenario 2: Empty State & adding Slots
Scenario 3: Past Slots
Review Progress
Code Review
Manual Tests
Test Coverage
Warning: Both client and server tests failed. Coverage could not be fully measured. Please check the workflow logs.
Last updated: 2026-01-27 13:10:57 UTC
Screenshots
Scenario 1

Scenario2

Scenario 3

** If it's a completely new interview process and there are no slots at all**
