Make Start Date Nullable

Author: thelenw

build.gradle

@@ -1,5 +1,6 @@
 plugins {
     id "org.sonarqube" version "5.0.0.4638"
+    id 'org.flywaydb.flyway' version '10.20.0'
 }
 
 tasks.named("sonarqube") {

server/src/main/java/com/objectcomputing/checkins/services/volunteering/VolunteeringRelationship.java

@@ -49,7 +49,7 @@ public class VolunteeringRelationship {
     @Schema(description = "id of the organization with the relationship")
     private UUID organizationId;
 
-    @NotNull
+    @Nullable
     @Column(name = "start_date")
     @TypeDef(type = DataType.DATE, converter = LocalDateConverter.class)
     @Schema(description = "when the relationship started")
@@ -67,11 +67,11 @@ public class VolunteeringRelationship {
     @Schema(description = "whether the Volunteering Relationship is active")
     private boolean active = true;
 
-    public VolunteeringRelationship(UUID memberId, UUID organizationId, LocalDate startDate, @Nullable LocalDate endDate) {
+    public VolunteeringRelationship(UUID memberId, UUID organizationId, @Nullable LocalDate startDate, @Nullable LocalDate endDate) {
         this(null, memberId, organizationId, startDate, endDate, true);
     }
 
-    public VolunteeringRelationship(UUID memberId, UUID organizationId, LocalDate startDate, @Nullable LocalDate endDate, boolean active) {
+    public VolunteeringRelationship(UUID memberId, UUID organizationId, @Nullable LocalDate startDate, @Nullable LocalDate endDate, boolean active) {
         this(null, memberId, organizationId, startDate, endDate, active);
     }
 }

server/src/main/java/com/objectcomputing/checkins/services/volunteering/VolunteeringRelationshipDTO.java

@@ -26,7 +26,7 @@ public class VolunteeringRelationshipDTO {
     @Schema(description = "id of the organization with the relationship")
     private UUID organizationId;
 
-    @NotNull
+    @Nullable
     @Schema(description = "when the relationship started")
     @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd")
     private LocalDate startDate;
@@ -40,7 +40,7 @@ public class VolunteeringRelationshipDTO {
     @Schema(description = "whether the Volunteering Relationship is active")
     private Boolean active;
 
-    public VolunteeringRelationshipDTO(@NotNull UUID memberId, @NotNull UUID organizationId, @NotNull LocalDate startDate, @Nullable LocalDate endDate) {
+    public VolunteeringRelationshipDTO(@NotNull UUID memberId, @NotNull UUID organizationId, @Nullable LocalDate startDate, @Nullable LocalDate endDate) {
         this(memberId, organizationId, startDate, endDate, true);
     }
 }

server/src/main/java/com/objectcomputing/checkins/services/volunteering/VolunteeringRelationshipRepository.java

@@ -21,7 +21,7 @@ JOIN volunteering_organization AS org USING(organization_id)
               AND rel.member_id = :memberId
               AND (rel.is_active = TRUE OR :includeDeactivated = TRUE)
               AND (org.is_active = TRUE OR :includeDeactivated = TRUE)
-            ORDER BY rel.start_date, org.name""", nativeQuery = true)
+            ORDER BY org.name""", nativeQuery = true)
     List<VolunteeringRelationship> findByMemberIdAndOrganizationId(UUID memberId, UUID organizationId, boolean includeDeactivated);
 
     @Query(value = """
@@ -31,7 +31,7 @@ JOIN volunteering_organization AS org USING(organization_id)
             WHERE rel.member_id = :memberId
               AND (rel.is_active = TRUE OR :includeDeactivated = TRUE)
               AND (org.is_active = TRUE OR :includeDeactivated = TRUE)
-            ORDER BY rel.start_date, org.name""", nativeQuery = true)
+            ORDER BY org.name""", nativeQuery = true)
     List<VolunteeringRelationship> findByMemberId(UUID memberId, boolean includeDeactivated);
 
     @Query(value = """
@@ -41,7 +41,7 @@ JOIN volunteering_organization AS org USING(organization_id)
             WHERE rel.organization_id = :organizationId
               AND (rel.is_active = TRUE OR :includeDeactivated = TRUE)
               AND (org.is_active = TRUE OR :includeDeactivated = TRUE)
-            ORDER BY rel.start_date, org.name""", nativeQuery = true)
+            ORDER BY org.name""", nativeQuery = true)
     List<VolunteeringRelationship> findByOrganizationId(UUID organizationId, boolean includeDeactivated);
 
     @Query(value = """
@@ -50,7 +50,7 @@ JOIN volunteering_organization AS org USING(organization_id)
                     JOIN volunteering_organization AS org USING(organization_id)
             WHERE (rel.is_active = TRUE OR :includeDeactivated = TRUE)
               AND (org.is_active = TRUE OR :includeDeactivated = TRUE)
-            ORDER BY rel.start_date, org.name""", nativeQuery = true)
+            ORDER BY org.name""", nativeQuery = true)
     List<VolunteeringRelationship> findAll(boolean includeDeactivated);
 
     @Query(value = """

server/src/main/resources/db/common/V107__create_volunteering_tables.sql

@@ -16,7 +16,7 @@ CREATE TABLE volunteering_relationship
     relationship_id varchar PRIMARY KEY,
     member_id       varchar REFERENCES member_profile (id),
     organization_id varchar REFERENCES volunteering_organization (organization_id),
-    start_date      timestamp NOT NULL,
+    start_date      timestamp,
     end_date        timestamp,
     is_active       boolean   NOT NULL DEFAULT TRUE
 );

web-ui/src/components/volunteer/VolunteerRelationships.jsx

@@ -112,8 +112,8 @@ const VolunteerRelationships = ({ forceUpdate = () => {}, onlyMe = false }) => {
     setSelectedRelationship({
       memberId: onlyMe ? currentUser.id : '',
       organizationId: '',
-      startDate: null, // Ensure this is a valid date object
-      endDate: null // Ensure this is a valid date object
+      startDate: null,
+      endDate: null
     });
     setRelationshipDialogOpen(true);
   }, [currentUser.id, onlyMe]);
@@ -175,7 +175,7 @@ const VolunteerRelationships = ({ forceUpdate = () => {}, onlyMe = false }) => {
       return;
     }
 
-    const formattedStartDate = formatDate(new Date(startDate));
+    const formattedStartDate = startDate ? formatDate(new Date(startDate)) : null;
     const formattedEndDate = endDate ? formatDate(new Date(endDate)) : null;
 
     const res = await resolve({
@@ -362,7 +362,7 @@ const VolunteerRelationships = ({ forceUpdate = () => {}, onlyMe = false }) => {
           value={selectedRelationship?.organizationId || ''}
         />
           <DatePickerField
-            date={selectedRelationship?.startDate}
+            date={selectedRelationship?.startDate || null}
             label="Start Date"
             setDate={date => setSelectedRelationship({ ...selectedRelationship, startDate: date })}
           />