Merge pull request 'feat: add task points' (#6) from feature/003-task-points into main

Reviewed-on: #6

README.md

@@ -10,7 +10,7 @@ Backend använder en lokal H2-databas i minnet. Databasschemat hanteras med
 Flyway, och lokal utvecklingsdata återställs när backend startas om.
 
 API:t innehåller endpoints under `/api/users` för användare och `/api/tasks` för
-att skapa, lista och tilldela gemensamma hushållsuppgifter.
+att skapa och lista gemensamma hushållsuppgifter.
 
 ## Starta backend
 

backend/src/main/java/se/rubble/hemhub/api/ApiExceptionHandler.java

@@ -2,15 +2,10 @@ package se.rubble.hemhub.api;
 
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
-import org.springframework.web.method.annotation.MethodArgumentTypeMismatchException;
 import org.springframework.web.bind.annotation.ExceptionHandler;
 import org.springframework.web.bind.annotation.RestControllerAdvice;
 
 import se.rubble.hemhub.task.InvalidTaskException;
-import se.rubble.hemhub.task.InvalidTaskAssignmentException;
-import se.rubble.hemhub.task.AssigneeNotFoundException;
-import se.rubble.hemhub.task.TaskAssignmentConflictException;
-import se.rubble.hemhub.task.TaskNotFoundException;
 import se.rubble.hemhub.user.InvalidUserNameException;
 import se.rubble.hemhub.user.UserNameAlreadyExistsException;
 
@@ -38,39 +33,5 @@ public class ApiExceptionHandler {
         return ResponseEntity.badRequest()
                 .body(new ApiError("INVALID_TASK", exception.getMessage()));
     }
-
-    @ExceptionHandler(InvalidTaskAssignmentException.class)
-    public ResponseEntity<ApiError> handleInvalidTaskAssignment(
-            InvalidTaskAssignmentException exception) {
-        return ResponseEntity.badRequest()
-                .body(new ApiError("INVALID_TASK_ASSIGNMENT", exception.getMessage()));
 }
 
-    @ExceptionHandler(MethodArgumentTypeMismatchException.class)
-    public ResponseEntity<ApiError> handleInvalidPathParameter() {
-        return ResponseEntity.badRequest()
-                .body(new ApiError(
-                        "INVALID_TASK_ASSIGNMENT",
-                        "Uppgifts-id måste vara ett giltigt UUID."));
-    }
-
-    @ExceptionHandler(TaskNotFoundException.class)
-    public ResponseEntity<ApiError> handleTaskNotFound() {
-        return ResponseEntity.status(HttpStatus.NOT_FOUND)
-                .body(new ApiError("TASK_NOT_FOUND", "Uppgiften finns inte."));
-    }
-
-    @ExceptionHandler(AssigneeNotFoundException.class)
-    public ResponseEntity<ApiError> handleAssigneeNotFound() {
-        return ResponseEntity.status(HttpStatus.NOT_FOUND)
-                .body(new ApiError("USER_NOT_FOUND", "Användaren finns inte."));
-    }
-
-    @ExceptionHandler(TaskAssignmentConflictException.class)
-    public ResponseEntity<ApiError> handleTaskAssignmentConflict() {
-        return ResponseEntity.status(HttpStatus.CONFLICT)
-                .body(new ApiError(
-                        "TASK_ASSIGNMENT_NOT_ALLOWED",
-                        "Ansvarig kan endast ändras för väntande uppgifter."));
-    }
-}

backend/src/main/java/se/rubble/hemhub/task/AssigneeNotFoundException.java

@@ -1,4 +0,0 @@
-package se.rubble.hemhub.task;
-
-public class AssigneeNotFoundException extends RuntimeException {
-}

backend/src/main/java/se/rubble/hemhub/task/CreateTaskRequest.java

@@ -2,11 +2,7 @@ package se.rubble.hemhub.task;
 
 import tools.jackson.databind.JsonNode;
 
-public record CreateTaskRequest(
+public record CreateTaskRequest(String title, String description, JsonNode points) {
-        String title,
-        String description,
-        JsonNode points,
-        JsonNode assigneeId) {
 
     Integer integerPoints() {
         if (points == null || !points.isIntegralNumber() || !points.canConvertToInt()) {
@@ -15,8 +11,4 @@ public record CreateTaskRequest(
 
         return points.intValue();
     }
-
-    UUIDValue parsedAssigneeId() {
-        return UUIDValue.optional(assigneeId);
-    }
 }

backend/src/main/java/se/rubble/hemhub/task/InvalidTaskAssignmentException.java

@@ -1,8 +0,0 @@
-package se.rubble.hemhub.task;
-
-public class InvalidTaskAssignmentException extends RuntimeException {
-
-    public InvalidTaskAssignmentException(String message) {
-        super(message);
-    }
-}

backend/src/main/java/se/rubble/hemhub/task/Task.java

@@ -8,11 +8,7 @@ import jakarta.persistence.Entity;
 import jakarta.persistence.EnumType;
 import jakarta.persistence.Enumerated;
 import jakarta.persistence.Id;
-import jakarta.persistence.JoinColumn;
-import jakarta.persistence.ManyToOne;
 import jakarta.persistence.Table;
-import jakarta.persistence.FetchType;
-import se.rubble.hemhub.user.User;
 
 @Entity
 @Table(name = "task")
@@ -34,10 +30,6 @@ class Task {
     @Column(nullable = false)
     private int points;
 
-    @ManyToOne(fetch = FetchType.LAZY)
-    @JoinColumn(name = "assignee_id")
-    private User assignee;
-
     @Column(name = "created_at", nullable = false)
     private Instant createdAt;
 
@@ -50,7 +42,6 @@ class Task {
             String description,
             TaskStatus status,
             int points,
-            User assignee,
             Instant createdAt) {
         if (points < 1 || points > 99) {
             throw new InvalidTaskException(
@@ -62,7 +53,6 @@ class Task {
         this.description = description;
         this.status = status;
         this.points = points;
-        this.assignee = assignee;
         this.createdAt = createdAt;
     }
 
@@ -86,21 +76,6 @@ class Task {
         return points;
     }
 
-    User getAssignee() {
-        return assignee;
-    }
-
-    void changeAssignee(User assignee) {
-        ensureAssignmentChangeAllowed();
-        this.assignee = assignee;
-    }
-
-    void ensureAssignmentChangeAllowed() {
-        if (status != TaskStatus.WAITING) {
-            throw new TaskAssignmentConflictException();
-        }
-    }
-
     Instant getCreatedAt() {
         return createdAt;
     }

backend/src/main/java/se/rubble/hemhub/task/TaskAssignmentConflictException.java

@@ -1,4 +0,0 @@
-package se.rubble.hemhub.task;
-
-public class TaskAssignmentConflictException extends RuntimeException {
-}

backend/src/main/java/se/rubble/hemhub/task/TaskController.java

@@ -1,13 +1,10 @@
 package se.rubble.hemhub.task;
 
 import java.util.List;
-import java.util.UUID;
 
 import org.springframework.http.HttpStatus;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PostMapping;
-import org.springframework.web.bind.annotation.PutMapping;
-import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.ResponseStatus;
@@ -31,24 +28,9 @@ public class TaskController {
     @PostMapping
     @ResponseStatus(HttpStatus.CREATED)
     public TaskResponse create(@RequestBody(required = false) CreateTaskRequest request) {
-        UUIDValue assigneeId = request == null
-                ? UUIDValue.optional(null)
-                : request.parsedAssigneeId();
         return taskService.create(
                 request == null ? null : request.title(),
                 request == null ? null : request.description(),
-                request == null ? null : request.integerPoints(),
+                request == null ? null : request.integerPoints());
-                assigneeId.value());
-    }
-
-    @PutMapping("/{taskId}/assignee")
-    public TaskResponse updateAssignee(
-            @PathVariable UUID taskId,
-            @RequestBody(required = false) UpdateTaskAssigneeRequest request) {
-        if (request == null) {
-            throw new InvalidTaskAssignmentException("Fältet assigneeId måste anges.");
-        }
-
-        return taskService.updateAssignee(taskId, request.parsedAssigneeId().value());
     }
 }

backend/src/main/java/se/rubble/hemhub/task/TaskNotFoundException.java

@@ -1,4 +0,0 @@
-package se.rubble.hemhub.task;
-
-public class TaskNotFoundException extends RuntimeException {
-}

backend/src/main/java/se/rubble/hemhub/task/TaskRepository.java

@@ -4,13 +4,9 @@ import java.util.List;
 import java.util.UUID;
 
 import org.springframework.data.jpa.repository.JpaRepository;
-import org.springframework.data.jpa.repository.EntityGraph;
 
 interface TaskRepository extends JpaRepository<Task, UUID> {
 
-    @EntityGraph(attributePaths = "assignee")
     List<Task> findAllByOrderByCreatedAtAscIdAsc();
-
-    @EntityGraph(attributePaths = "assignee")
-    java.util.Optional<Task> findOneById(UUID id);
 }
+

backend/src/main/java/se/rubble/hemhub/task/TaskResponse.java

@@ -9,7 +9,6 @@ public record TaskResponse(
         String description,
         TaskStatus status,
         int points,
-        AssigneeResponse assignee,
         Instant createdAt) {
 
     static TaskResponse from(Task task) {
@@ -19,14 +18,6 @@ public record TaskResponse(
                 task.getDescription(),
                 task.getStatus(),
                 task.getPoints(),
-                AssigneeResponse.from(task.getAssignee()),
                 task.getCreatedAt());
     }
-
-    public record AssigneeResponse(UUID id, String name) {
-
-        static AssigneeResponse from(se.rubble.hemhub.user.User user) {
-            return user == null ? null : new AssigneeResponse(user.getId(), user.getName());
-        }
-    }
 }

backend/src/main/java/se/rubble/hemhub/task/TaskService.java

@@ -9,24 +9,19 @@ import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
-import se.rubble.hemhub.user.User;
-import se.rubble.hemhub.user.UserRepository;
-
 @Service
 class TaskService {
 
     private final TaskRepository taskRepository;
-    private final UserRepository userRepository;
     private final Clock clock;
 
     @Autowired
-    TaskService(TaskRepository taskRepository, UserRepository userRepository) {
+    TaskService(TaskRepository taskRepository) {
-        this(taskRepository, userRepository, Clock.systemUTC());
+        this(taskRepository, Clock.systemUTC());
     }
 
-    TaskService(TaskRepository taskRepository, UserRepository userRepository, Clock clock) {
+    TaskService(TaskRepository taskRepository, Clock clock) {
         this.taskRepository = taskRepository;
-        this.userRepository = userRepository;
         this.clock = clock;
     }
 
@@ -41,8 +36,7 @@ class TaskService {
     TaskResponse create(
             String requestedTitle,
             String requestedDescription,
-            Integer requestedPoints,
+            Integer requestedPoints) {
-            UUID requestedAssigneeId) {
         String title = requestedTitle == null ? "" : requestedTitle.trim();
         String description = normalizeDescription(requestedDescription);
 
@@ -61,39 +55,17 @@ class TaskService {
                     "Poäng måste vara ett heltal mellan 1 och 99.");
         }
 
-        User assignee = findAssignee(requestedAssigneeId);
         Task task = new Task(
                 UUID.randomUUID(),
                 title,
                 description,
                 TaskStatus.WAITING,
                 requestedPoints,
-                assignee,
                 Instant.now(clock));
 
         return TaskResponse.from(taskRepository.save(task));
     }
 
-    @Transactional
-    TaskResponse updateAssignee(UUID taskId, UUID requestedAssigneeId) {
-        Task task = taskRepository.findOneById(taskId)
-                .orElseThrow(TaskNotFoundException::new);
-        task.ensureAssignmentChangeAllowed();
-        User assignee = findAssignee(requestedAssigneeId);
-
-        task.changeAssignee(assignee);
-        return TaskResponse.from(task);
-    }
-
-    private User findAssignee(UUID requestedAssigneeId) {
-        if (requestedAssigneeId == null) {
-            return null;
-        }
-
-        return userRepository.findById(requestedAssigneeId)
-                .orElseThrow(AssigneeNotFoundException::new);
-    }
-
     private static String normalizeDescription(String requestedDescription) {
         if (requestedDescription == null) {
             return null;

backend/src/main/java/se/rubble/hemhub/task/UUIDValue.java

@@ -1,25 +0,0 @@
-package se.rubble.hemhub.task;
-
-import java.util.UUID;
-
-import tools.jackson.databind.JsonNode;
-import tools.jackson.databind.node.JsonNodeType;
-
-record UUIDValue(boolean present, UUID value) {
-
-    static UUIDValue optional(JsonNode node) {
-        if (node == null || node.isNull()) {
-            return new UUIDValue(node != null, null);
-        }
-
-        if (node.getNodeType() != JsonNodeType.STRING) {
-            throw new InvalidTaskAssignmentException("Användar-id måste vara ett giltigt UUID.");
-        }
-
-        try {
-            return new UUIDValue(true, UUID.fromString(node.stringValue()));
-        } catch (IllegalArgumentException exception) {
-            throw new InvalidTaskAssignmentException("Användar-id måste vara ett giltigt UUID.");
-        }
-    }
-}

backend/src/main/java/se/rubble/hemhub/task/UpdateTaskAssigneeRequest.java

@@ -1,16 +0,0 @@
-package se.rubble.hemhub.task;
-
-import tools.jackson.databind.JsonNode;
-
-public record UpdateTaskAssigneeRequest(JsonNode assigneeId) {
-
-    UUIDValue parsedAssigneeId() {
-        UUIDValue parsed = UUIDValue.optional(assigneeId);
-
-        if (!parsed.present()) {
-            throw new InvalidTaskAssignmentException("Fältet assigneeId måste anges.");
-        }
-
-        return parsed;
-    }
-}

backend/src/main/java/se/rubble/hemhub/user/User.java

@@ -10,7 +10,7 @@ import jakarta.persistence.Table;
 
 @Entity
 @Table(name = "app_user")
-public class User {
+class User {
 
     @Id
     private UUID id;
@@ -34,11 +34,11 @@ public class User {
         this.createdAt = createdAt;
     }
 
-    public UUID getId() {
+    UUID getId() {
         return id;
     }
 
-    public String getName() {
+    String getName() {
         return name;
     }
 

backend/src/main/java/se/rubble/hemhub/user/UserRepository.java

@@ -4,7 +4,8 @@ import java.util.UUID;
 
 import org.springframework.data.jpa.repository.JpaRepository;
 
-public interface UserRepository extends JpaRepository<User, UUID> {
+interface UserRepository extends JpaRepository<User, UUID> {
 
     boolean existsByNormalizedName(String normalizedName);
 }
+

backend/src/main/resources/db/migration/V4__add_task_assignee.sql

@@ -1,6 +0,0 @@
-ALTER TABLE task
-    ADD COLUMN assignee_id UUID;
-
-ALTER TABLE task
-    ADD CONSTRAINT fk_task_assignee
-        FOREIGN KEY (assignee_id) REFERENCES app_user (id);

backend/src/test/java/se/rubble/hemhub/task/TaskApiTest.java

@@ -15,7 +15,6 @@ import org.springframework.web.context.WebApplicationContext;
 
 import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get;
 import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post;
-import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.put;
 import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath;
 import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
 
@@ -53,7 +52,6 @@ class TaskApiTest {
                 .andExpect(jsonPath("$.description").value("Bottenvåningen"))
                 .andExpect(jsonPath("$.status").value("WAITING"))
                 .andExpect(jsonPath("$.points").value(7))
-                .andExpect(jsonPath("$.assignee").value((Object) null))
                 .andExpect(jsonPath("$.createdAt").isString());
 
         mockMvc.perform(get("/api/tasks"))
@@ -61,67 +59,6 @@ class TaskApiTest {
                 .andExpect(jsonPath("$[0].points").value(7));
     }
 
-    @Test
-    void createsUnassignedTaskWhenAssigneeIsExplicitlyNull() throws Exception {
-        mockMvc.perform(post("/api/tasks")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content("""
-                                {"title": "Dammsuga", "points": 1, "assigneeId": null}
-                                """))
-                .andExpect(status().isCreated())
-                .andExpect(jsonPath("$.assignee").value((Object) null));
-    }
-
-    @Test
-    void createsAndListsTaskWithAssignee() throws Exception {
-        UUID userId = createUser("Anna");
-
-        mockMvc.perform(post("/api/tasks")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content("""
-                                {
-                                  "title": "Dammsuga",
-                                  "points": 7,
-                                  "assigneeId": "%s"
-                                }
-                                """.formatted(userId)))
-                .andExpect(status().isCreated())
-                .andExpect(jsonPath("$.assignee.id").value(userId.toString()))
-                .andExpect(jsonPath("$.assignee.name").value("Anna"));
-
-        mockMvc.perform(get("/api/tasks"))
-                .andExpect(status().isOk())
-                .andExpect(jsonPath("$[0].assignee.id").value(userId.toString()))
-                .andExpect(jsonPath("$[0].assignee.name").value("Anna"));
-    }
-
-    @Test
-    void rejectsUnknownAssigneeWhenCreatingTask() throws Exception {
-        mockMvc.perform(post("/api/tasks")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content("""
-                                {
-                                  "title": "Dammsuga",
-                                  "points": 7,
-                                  "assigneeId": "00000000-0000-0000-0000-000000000099"
-                                }
-                                """))
-                .andExpect(status().isNotFound())
-                .andExpect(jsonPath("$.code").value("USER_NOT_FOUND"));
-
-        mockMvc.perform(post("/api/tasks")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content("""
-                                {
-                                  "title": "Dammsuga",
-                                  "points": 7,
-                                  "assigneeId": "inte-ett-uuid"
-                                }
-                                """))
-                .andExpect(status().isBadRequest())
-                .andExpect(jsonPath("$.code").value("INVALID_TASK_ASSIGNMENT"));
-    }
-
     @Test
     void storesBlankDescriptionAsNull() throws Exception {
         mockMvc.perform(post("/api/tasks")
@@ -196,12 +133,9 @@ class TaskApiTest {
         UUID secondId = UUID.fromString("00000000-0000-0000-0000-000000000002");
         UUID newestId = UUID.fromString("00000000-0000-0000-0000-000000000003");
 
-        taskRepository.save(new Task(
+        taskRepository.save(new Task(newestId, "Nyast", null, TaskStatus.WAITING, 3, newer));
-                newestId, "Nyast", null, TaskStatus.WAITING, 3, null, newer));
+        taskRepository.save(new Task(secondId, "Andra", null, TaskStatus.IN_PROGRESS, 2, older));
-        taskRepository.save(new Task(
+        taskRepository.save(new Task(firstId, "Första", null, TaskStatus.COMPLETED, 1, older));
-                secondId, "Andra", null, TaskStatus.IN_PROGRESS, 2, null, older));
-        taskRepository.save(new Task(
-                firstId, "Första", null, TaskStatus.COMPLETED, 1, null, older));
 
         mockMvc.perform(get("/api/tasks"))
                 .andExpect(status().isOk())
@@ -211,93 +145,6 @@ class TaskApiTest {
                 .andExpect(jsonPath("$[2].title").value("Nyast"));
     }
 
-    @Test
-    void assignsChangesAndRemovesAssigneeWithoutChangingOtherTaskFields() throws Exception {
-        UUID firstUserId = createUser("Bo");
-        UUID secondUserId = createUser("Cecilia");
-        String taskId = createTask("Dammsuga", "Bottenvåningen", 7, null);
-
-        updateAssignee(taskId, """
-                {"assigneeId": "%s"}
-                """.formatted(firstUserId))
-                .andExpect(status().isOk())
-                .andExpect(jsonPath("$.assignee.id").value(firstUserId.toString()))
-                .andExpect(jsonPath("$.status").value("WAITING"))
-                .andExpect(jsonPath("$.title").value("Dammsuga"))
-                .andExpect(jsonPath("$.description").value("Bottenvåningen"))
-                .andExpect(jsonPath("$.points").value(7));
-
-        updateAssignee(taskId, """
-                {"assigneeId": "%s"}
-                """.formatted(secondUserId))
-                .andExpect(status().isOk())
-                .andExpect(jsonPath("$.assignee.id").value(secondUserId.toString()));
-
-        updateAssignee(taskId, """
-                {"assigneeId": null}
-                """)
-                .andExpect(status().isOk())
-                .andExpect(jsonPath("$.assignee").value((Object) null))
-                .andExpect(jsonPath("$.status").value("WAITING"))
-                .andExpect(jsonPath("$.title").value("Dammsuga"))
-                .andExpect(jsonPath("$.description").value("Bottenvåningen"))
-                .andExpect(jsonPath("$.points").value(7));
-    }
-
-    @Test
-    void rejectsMissingAssigneeFieldAndInvalidUuid() throws Exception {
-        String taskId = createTask("Dammsuga", null, 1, null);
-
-        updateAssignee(taskId, "{}")
-                .andExpect(status().isBadRequest())
-                .andExpect(jsonPath("$.code").value("INVALID_TASK_ASSIGNMENT"));
-        updateAssignee(taskId, """
-                {"assigneeId": "inte-ett-uuid"}
-                """)
-                .andExpect(status().isBadRequest())
-                .andExpect(jsonPath("$.code").value("INVALID_TASK_ASSIGNMENT"));
-        updateAssignee("inte-ett-uuid", """
-                {"assigneeId": null}
-                """)
-                .andExpect(status().isBadRequest())
-                .andExpect(jsonPath("$.code").value("INVALID_TASK_ASSIGNMENT"));
-    }
-
-    @Test
-    void returnsNotFoundForUnknownTaskAndUnknownAssignee() throws Exception {
-        String taskId = createTask("Dammsuga", null, 1, null);
-
-        updateAssignee("00000000-0000-0000-0000-000000000099", """
-                {"assigneeId": null}
-                """)
-                .andExpect(status().isNotFound())
-                .andExpect(jsonPath("$.code").value("TASK_NOT_FOUND"));
-        updateAssignee(taskId, """
-                {"assigneeId": "00000000-0000-0000-0000-000000000099"}
-                """)
-                .andExpect(status().isNotFound())
-                .andExpect(jsonPath("$.code").value("USER_NOT_FOUND"));
-    }
-
-    @Test
-    void rejectsAssigneeChangeForTaskThatIsNotWaiting() throws Exception {
-        UUID taskId = UUID.randomUUID();
-        taskRepository.save(new Task(
-                taskId,
-                "Pågående",
-                null,
-                TaskStatus.IN_PROGRESS,
-                3,
-                null,
-                Instant.parse("2026-07-26T12:00:00Z")));
-
-        updateAssignee(taskId.toString(), """
-                {"assigneeId": null}
-                """)
-                .andExpect(status().isConflict())
-                .andExpect(jsonPath("$.code").value("TASK_ASSIGNMENT_NOT_ALLOWED"));
-    }
-
     private ResultActions createTaskWithPoints(int points) throws Exception {
         return mockMvc.perform(post("/api/tasks")
                 .contentType(MediaType.APPLICATION_JSON)
@@ -312,50 +159,4 @@ class TaskApiTest {
                 .andExpect(status().isBadRequest())
                 .andExpect(jsonPath("$.code").value("INVALID_TASK"));
     }
-
-    private UUID createUser(String name) throws Exception {
-        String response = mockMvc.perform(post("/api/users")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content("""
-                                {"name": "%s"}
-                                """.formatted(name)))
-                .andExpect(status().isCreated())
-                .andReturn()
-                .getResponse()
-                .getContentAsString();
-
-        String id = com.jayway.jsonpath.JsonPath.read(response, "$.id");
-        return UUID.fromString(id);
-    }
-
-    private String createTask(
-            String title,
-            String description,
-            int points,
-            UUID assigneeId) throws Exception {
-        String descriptionJson = description == null ? "null" : "\"%s\"".formatted(description);
-        String assigneeJson = assigneeId == null ? "null" : "\"%s\"".formatted(assigneeId);
-        String response = mockMvc.perform(post("/api/tasks")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content("""
-                                {
-                                  "title": "%s",
-                                  "description": %s,
-                                  "points": %d,
-                                  "assigneeId": %s
-                                }
-                                """.formatted(title, descriptionJson, points, assigneeJson)))
-                .andExpect(status().isCreated())
-                .andReturn()
-                .getResponse()
-                .getContentAsString();
-
-        return com.jayway.jsonpath.JsonPath.read(response, "$.id");
-    }
-
-    private ResultActions updateAssignee(String taskId, String body) throws Exception {
-        return mockMvc.perform(put("/api/tasks/{taskId}/assignee", taskId)
-                .contentType(MediaType.APPLICATION_JSON)
-                .content(body));
-    }
 }

backend/src/test/java/se/rubble/hemhub/task/TaskTest.java

@@ -22,7 +22,6 @@ class TaskTest {
                 null,
                 TaskStatus.WAITING,
                 points,
-                null,
                 Instant.parse("2026-07-26T12:00:00Z"));
     }
 }

docs/architecture.md

@@ -29,7 +29,6 @@ Den ansvarar för:
 - hämtning och presentation av användare och uppgifter;
 - lokalt val av aktiv användare;
 - formulär för att skapa användare och uppgifter;
-- val och visning av ansvarig användare på uppgifter;
 - klientnära validering och begripliga felmeddelanden;
 - uppgiftsbrädan med kolumnerna Väntande, Pågående och Klart.
 
@@ -61,7 +60,6 @@ Aktuella endpoints:
 - `POST /api/users`
 - `GET /api/tasks`
 - `POST /api/tasks`
-- `PUT /api/tasks/{taskId}/assignee`
 
 ### Databas och migreringar
 
@@ -79,7 +77,6 @@ Flyway kör migreringarna:
 - `V1__create_users.sql`
 - `V2__create_tasks.sql`
 - `V3__add_task_points.sql`
-- `V4__add_task_assignee.sql`
 
 Hibernate är konfigurerat med `ddl-auto=validate`; Flyway skapar schemat och
 Hibernate validerar entiteterna mot det.
@@ -107,19 +104,12 @@ En uppgift lagras i tabellen `task` med:
 - `description`: valfri beskrivning, högst 500 tecken;
 - `status`: `WAITING`, `IN_PROGRESS` eller `COMPLETED`;
 - `points`: obligatoriskt heltal mellan 1 och 99;
-- `assignee_id`: nullable främmande nyckel till `app_user`;
 - `created_at`: en `Instant`, lagrad som `TIMESTAMP WITH TIME ZONE`.
 
 Status lagras som enumens textvärde genom `EnumType.STRING`. Nya uppgifter får
 alltid status `WAITING`. Poängintervallet skyddas i backend och med en
-databasconstraint. En uppgift kan vara otilldelad eller referera till exakt en
+databasconstraint. Det finns ingen relation mellan uppgifter och användare;
-ansvarig användare. Relationen hämtas tillsammans med uppgifterna när de listas,
+alla aktiva användare ser samma uppgiftslista.
-så API-responsen kan innehålla ansvarigs `id` och `name` utan separata
-frontend-anrop. Alla aktiva användare ser samma uppgiftslista.
-
-Ansvarig är valfri vid skapande. Endast väntande uppgifter kan få ändrad
-ansvarig genom det särskilda tilldelnings-API:t. Tilldelning ändrar aldrig
-uppgiftens status.
 
 ### Aktiv användare
 
@@ -134,9 +124,8 @@ lokalt per browser och utgör inte autentisering eller behörighetskontroll.
 ### Felhantering
 
 Backend använder ett litet gemensamt JSON-format med `code` och `message`.
-`ApiExceptionHandler` översätter kända valideringsfel till `400 Bad Request`,
+`ApiExceptionHandler` översätter kända valideringsfel till `400 Bad Request`
-saknade uppgifter eller användare till `404 Not Found` och dubbletter eller
+och dubbletter av användarnamn till `409 Conflict`.
-otillåtna tilldelningsändringar till `409 Conflict`.
 
 Frontend skiljer mellan fel vid hämtning och skapande. Hämtfel kan
 återförsökas. Formulärfel visas nära formuläret och inmatningen behålls vid

docs/features/004-task-assignment.md

@@ -1,122 +0,0 @@
-# Feature 4 – Tilldelning av uppgifter
-
-## Status
-
-Pågående.
-
-## Bakgrund
-
-HemHub har centralt lagrade användare och gemensamma uppgifter. För att senare
-kunna införa regler för pågående arbete behöver en uppgift kunna ha en ansvarig
-användare, utan att tilldelning samtidigt ändrar uppgiftens status.
-
-## Mål
-
-- välja en valfri ansvarig när en uppgift skapas;
-- visa ansvarig på uppgiftskortet;
-- tilldela, byta eller ta bort ansvarig på en väntande uppgift;
-- lagra tilldelningen centralt så att alla användare ser samma värde.
-
-## Omfattning
-
-En uppgift kan vara otilldelad eller tilldelad exakt en befintlig användare.
-`Ingen` är standard vid skapande och den aktiva browseranvändaren förväljs
-inte. Frontend återanvänder användarlistan som redan hämtas vid appstart.
-
-På ett otilldelat väntande kort öppnar `Ta uppgift` ett användarval. Ett
-tilldelat väntande kort visar namnet och öppnar samma val. Ändringen skickas
-direkt till backend och kortet uppdateras först med den bekräftade responsen.
-Vid fel behålls den tidigare tilldelningen och ett lokalt felmeddelande visas.
-
-För `IN_PROGRESS` och `COMPLETED` visas ansvarig eller `Otilldelad` utan
-redigerbar kontroll.
-
-## Produktregler
-
-- En uppgift har högst en ansvarig.
-- Ansvarig är valfri och måste motsvara en befintlig användare.
-- Endast uppgifter med status `WAITING` får få ändrad ansvarig.
-- Tilldelning ändrar aldrig status, titel, beskrivning eller poäng.
-- Vem som helst kan välja valfri ansvarig; aktiv användare är inte
-  autentisering eller behörighetskontroll.
-
-## API-förändringar
-
-`POST /api/tasks` accepterar det valfria fältet `assigneeId`. Saknat fält eller
-`null` skapar en otilldelad uppgift. Ett UUID som inte motsvarar en användare
-ger `404 Not Found`.
-
-`PUT /api/tasks/{taskId}/assignee` ändrar endast ansvarig:
-
-```json
-{"assigneeId": "d56b54dd-31b0-4d71-8a10-82464be59a61"}
-```
-
-`{"assigneeId": null}` tar bort tilldelningen. Fältet måste finnas i requesten.
-Responsen är den uppdaterade uppgiften. Task-responser innehåller:
-
-```json
-{"assignee": {"id": "d56b54dd-31b0-4d71-8a10-82464be59a61", "name": "Anna"}}
-```
-
-Otilldelade uppgifter har `"assignee": null`. Ogiltigt UUID eller saknat fält
-ger `400`, okänd uppgift eller användare ger `404` och ändring av en uppgift
-som inte väntar ger `409`. Felen använder det befintliga formatet med `code`
-och `message`.
-
-## Databasförändringar
-
-`V4__add_task_assignee.sql` lägger till `task.assignee_id UUID NULL` med en
-främmande nyckel till `app_user.id`. Befintliga uppgifter blir otilldelade.
-Migreringen använder varken `ON DELETE CASCADE` eller `ON DELETE SET NULL`.
-
-JPA-modellen använder en lazy `ManyToOne`. Repositoryts listning och
-id-hämtning använder en entity graph för att hämta ansvarig tillsammans med
-uppgiften och undvika N+1-frågor när responsen byggs.
-
-## Frontendförändringar
-
-Skapandedialogen innehåller ett tilldelningsval med `Ingen` och samtliga
-användare. Valet bevaras tillsammans med övriga formulärvärden vid fel.
-
-Väntande kort har en separat tilldelningskontroll. Kontrollen är inaktiverad
-medan just det kortets request pågår; övriga delar av brädan förblir
-interaktiva. Serverns task-respons ersätter motsvarande uppgift i den befintliga
-listan utan att ändra ordningen.
-
-## Tester och verifiering
-
-Backendens integrationstester täcker skapande med och utan ansvarig,
-responsformat, okända id:n, tilldelning, byte, av-tilldelning, statuskonflikt
-och att övriga uppgiftsfält inte ändras.
-
-Frontendtesterna täcker standardval och användarlista i skapandedialogen,
-create-requestens `assigneeId`, kortens redigerbara och statiska lägen,
-tilldelningsrequest, vänteläge, serverbekräftad uppdatering, av-tilldelning och
-fel utan optimistisk ändring.
-
-Manuell verifiering ska omfatta skapande med och utan ansvarig, tilldelning,
-byte, av-tilldelning, bevarad status, omladdning, statiska kontroller för andra
-statusar, felrespons och projektets normala desktop- och mobilbredder.
-
-## Avgränsning mot Feature 5
-
-Feature 4 inför inget API eller UI för statusändring och inte regeln att
-`IN_PROGRESS` måste ha en ansvarig. Tilldelning leder inte automatiskt till
-`IN_PROGRESS`, och av-tilldelning leder inte automatiskt till `WAITING`.
-
-## Ingår inte
-
-Flera ansvariga, statusändring, drag-and-drop, generell redigering, radering,
-deadlines, återkommande uppgifter, poänghistorik, användaradministration,
-autentisering, behörighetskontroll och automatisk tilldelning ingår inte.
-
-## Kända begränsningar
-
-Användare kan ännu inte raderas, så relationens framtida beteende vid
-användarradering är inte beslutat. Frontend har ingen optimistisk uppdatering;
-det tidigare värdet ligger kvar tills backend svarar.
-
-## Relaterade commits
-
-Fylls i när featuren har committats.

docs/roadmap.md

@@ -43,12 +43,11 @@ Feature 0–2 är klara. Den aktuella applikationen har:
 - en bräda med Väntande, Pågående och Klart;
 - nya uppgifter som alltid skapas med status `WAITING`.
 
-Tilldelning av högst en ansvarig användare per uppgift är under utveckling. Det
+Det finns ännu inga uppgiftstilldelningar, statusändringar, drag-and-drop,
-finns ännu inga statusändringar, drag-and-drop, redigeringar, raderingar,
+redigeringar, raderingar, deadlines eller återkommande uppgifter.
-deadlines eller återkommande uppgifter.
 Nuvarande användarval är inte autentisering.
 
-**Feature 4 – Tilldelning av uppgifter är pågående.**
+**Feature 3 – Uppgiftspoäng är pågående.**
 
 ## Featureöversikt
 
@@ -58,7 +57,7 @@ Nuvarande användarval är inte autentisering.
 | 1 – Användarval | Klar | 0 | Centrala användare och lokalt aktivt användar-id |
 | 2 – Skapa uppgifter | Klar | 0–1 | Gemensamma uppgifter och trekolumnsbräda |
 | 3 – Uppgiftspoäng | Pågående | 2 | Poäng på uppgifter |
-| 4 – Tilldelning | Pågående | 1–2 | Valfri ansvarig användare |
+| 4 – Tilldelning | Planerad | 1–2 | Valfri ansvarig användare |
 | 5 – Statusändring | Planerad | 4 | Backendstyrda statusövergångar |
 | 6 – Drag-and-drop | Planerad | 5 | Kortflytt via status-API |
 | 7 – Radera uppgift | Planerad | 2 | Bekräftad radering |
@@ -131,7 +130,7 @@ databasen har inget permanent defaultvärde.
 
 ### Feature 4 – Tilldelning av uppgifter
 
-**Status:** Pågående
+**Status:** Planerad
 
 **Beroenden:** Feature 1 och Feature 2
 
@@ -142,10 +141,13 @@ databasen har inget permanent defaultvärde.
 - visa ansvarig på uppgiftskort;
 - kunna ändra ansvarig på en befintlig uppgift.
 
-En väntande uppgift får vara tilldelad eller otilldelad och har högst en
+En väntande uppgift får vara tilldelad eller otilldelad. Tilldelning införs före
-ansvarig. Tilldelning införs före statusändring eftersom en pågående uppgift
+statusändring eftersom en pågående uppgift senare måste ha en ansvarig.
-senare måste ha en ansvarig. Hur borttagna användare ska hanteras är fortsatt
+
-öppet tills användarradering införs.
+**Öppna frågor:**
+
+- om en uppgift ska ha endast en ansvarig;
+- hur borttagna användare ska hanteras när användarradering införs.
 
 ### Feature 5 – Statusändring och statusregler
 

frontend/src/App.test.tsx

@@ -22,7 +22,6 @@ const tasks = [
     description: 'Bottenvåningen',
     status: 'WAITING',
     points: 7,
-    assignee: null,
     createdAt: '2026-07-24T10:00:00Z',
   },
   {
@@ -31,7 +30,6 @@ const tasks = [
     description: null,
     status: 'IN_PROGRESS',
     points: 3,
-    assignee: { id: users[1].id, name: users[1].name },
     createdAt: '2026-07-24T10:01:00Z',
   },
   {
@@ -40,7 +38,6 @@ const tasks = [
     description: null,
     status: 'COMPLETED',
     points: 5,
-    assignee: null,
     createdAt: '2026-07-24T10:02:00Z',
   },
 ]
@@ -196,13 +193,6 @@ test('Ny uppgift öppnar modalen med fokus i titelfältet', async () => {
   expect(screen.getByRole('dialog', { name: 'Skapa ny uppgift' })).toBeInTheDocument()
   expect(screen.getByLabelText('Titel')).toHaveFocus()
   expect(screen.getByLabelText('Poäng')).toHaveValue(1)
-  expect(screen.getByLabelText('Tilldela')).toHaveValue('')
-  expect(within(screen.getByLabelText('Tilldela')).getByRole('option', { name: 'Ingen' }))
-    .toBeInTheDocument()
-  expect(within(screen.getByLabelText('Tilldela')).getByRole('option', { name: 'Urban' }))
-    .toBeInTheDocument()
-  expect(within(screen.getByLabelText('Tilldela')).getByRole('option', { name: 'Anna' }))
-    .toBeInTheDocument()
 })
 
 test.each([
@@ -256,7 +246,6 @@ test('en skapad uppgift visas längst ned i Väntande och modalen stängs', asyn
     description: 'Köket',
     status: 'WAITING',
     points: 7,
-    assignee: { id: users[1].id, name: users[1].name },
     createdAt: '2026-07-24T10:03:00Z',
   }
   window.localStorage.setItem('hemhub.activeUserId', users[0].id)
@@ -274,7 +263,6 @@ test('en skapad uppgift visas längst ned i Väntande och modalen stängs', asyn
     target: { value: '  Köket  ' },
   })
   fireEvent.change(screen.getByLabelText('Poäng'), { target: { value: '7' } })
-  fireEvent.change(screen.getByLabelText('Tilldela'), { target: { value: users[1].id } })
   fireEvent.click(screen.getByRole('button', { name: 'Skapa uppgift' }))
 
   await waitFor(() =>
@@ -282,148 +270,17 @@ test('en skapad uppgift visas längst ned i Väntande och modalen stängs', asyn
   )
   const waiting = screen.getByRole('region', { name: 'Väntande' })
   expect(within(waiting).getAllByRole('article').map((card) => card.textContent)).toEqual([
-    'Dammsuga7 pBottenvåningenTa uppgift',
+    'Dammsuga7 pBottenvåningen',
-    'Putsa fönster7 pKöketAnna',
+    'Putsa fönster7 pKöket',
   ])
   expect(fetchMock).toHaveBeenLastCalledWith('/api/tasks', {
     method: 'POST',
     headers: { 'Content-Type': 'application/json' },
-    body: JSON.stringify({
+    body: JSON.stringify({ title: 'Putsa fönster', description: 'Köket', points: 7 }),
-      title: 'Putsa fönster',
-      description: 'Köket',
-      points: 7,
-      assigneeId: users[1].id,
-    }),
   })
 
   fireEvent.click(screen.getByRole('button', { name: 'Ny uppgift' }))
   expect(screen.getByLabelText('Poäng')).toHaveValue(1)
-  expect(screen.getByLabelText('Tilldela')).toHaveValue('')
-})
-
-test('Ingen skickas som null när en uppgift skapas', async () => {
-  const createdTask = {
-    ...tasks[0],
-    id: '00000000-0000-0000-0000-000000000010',
-    title: 'Torka bordet',
-  }
-  window.localStorage.setItem('hemhub.activeUserId', users[0].id)
-  const fetchMock = vi.spyOn(globalThis, 'fetch')
-  fetchMock.mockResolvedValueOnce(jsonResponse(users))
-  fetchMock.mockResolvedValueOnce(jsonResponse([]))
-  fetchMock.mockResolvedValueOnce(jsonResponse(createdTask, 201))
-  render(<App />)
-
-  await waitFor(() => expect(fetchMock).toHaveBeenCalledTimes(2))
-  fireEvent.click(screen.getByRole('button', { name: 'Ny uppgift' }))
-  fireEvent.change(screen.getByLabelText('Titel'), { target: { value: 'Torka bordet' } })
-  fireEvent.click(screen.getByRole('button', { name: 'Skapa uppgift' }))
-
-  await waitFor(() => expect(fetchMock).toHaveBeenCalledTimes(3))
-  expect(fetchMock).toHaveBeenLastCalledWith('/api/tasks', {
-    method: 'POST',
-    headers: { 'Content-Type': 'application/json' },
-    body: JSON.stringify({
-      title: 'Torka bordet',
-      description: null,
-      points: 1,
-      assigneeId: null,
-    }),
-  })
-})
-
-test('kort visar ansvarig och endast väntande uppgifter har redigerbar kontroll', async () => {
-  window.localStorage.setItem('hemhub.activeUserId', users[0].id)
-  mockUsersAndTasks(users, tasks)
-  render(<App />)
-
-  const waitingCard = (await screen.findByText('Dammsuga')).closest('article')
-  const inProgressCard = screen.getByText('Diska').closest('article')
-  const completedCard = screen.getByText('Vattna blommor').closest('article')
-
-  expect(waitingCard).not.toBeNull()
-  expect(inProgressCard).not.toBeNull()
-  expect(completedCard).not.toBeNull()
-  expect(within(waitingCard!).getByRole('button', { name: 'Ändra ansvarig för Dammsuga' }))
-    .toHaveTextContent('Ta uppgift')
-  expect(within(inProgressCard!).getByText('Anna')).toBeInTheDocument()
-  expect(within(inProgressCard!).queryByRole('button', { name: /ansvarig/i })).not.toBeInTheDocument()
-  expect(within(completedCard!).getByText('Otilldelad')).toBeInTheDocument()
-})
-
-test('val av ansvarig anropar endpointen och uppdaterar kortet efter svar', async () => {
-  const updatedTask = { ...tasks[0], assignee: { id: users[1].id, name: users[1].name } }
-  let resolveAssignment!: (response: Response) => void
-  const assignmentResponse = new Promise<Response>((resolve) => {
-    resolveAssignment = resolve
-  })
-  window.localStorage.setItem('hemhub.activeUserId', users[0].id)
-  const fetchMock = vi.spyOn(globalThis, 'fetch')
-  fetchMock.mockResolvedValueOnce(jsonResponse(users))
-  fetchMock.mockResolvedValueOnce(jsonResponse([tasks[0]]))
-  fetchMock.mockReturnValueOnce(assignmentResponse)
-  render(<App />)
-
-  fireEvent.click(await screen.findByRole('button', { name: 'Ändra ansvarig för Dammsuga' }))
-  const select = screen.getByRole('combobox', { name: 'Ansvarig för Dammsuga' })
-  fireEvent.change(select, { target: { value: users[1].id } })
-
-  expect(select).toBeDisabled()
-  expect(fetchMock).toHaveBeenLastCalledWith(`/api/tasks/${tasks[0].id}/assignee`, {
-    method: 'PUT',
-    headers: { 'Content-Type': 'application/json' },
-    body: JSON.stringify({ assigneeId: users[1].id }),
-  })
-
-  resolveAssignment(jsonResponse(updatedTask))
-  await waitFor(() =>
-    expect(screen.getByRole('button', { name: 'Ändra ansvarig för Dammsuga' }))
-      .toHaveTextContent('Anna'),
-  )
-  expect(screen.queryByRole('combobox', { name: 'Ansvarig för Dammsuga' })).not.toBeInTheDocument()
-})
-
-test('val av Ingen av-tilldelar en väntande uppgift', async () => {
-  const assignedTask = { ...tasks[0], assignee: { id: users[1].id, name: users[1].name } }
-  const unassignedTask = { ...assignedTask, assignee: null }
-  window.localStorage.setItem('hemhub.activeUserId', users[0].id)
-  const fetchMock = vi.spyOn(globalThis, 'fetch')
-  fetchMock.mockResolvedValueOnce(jsonResponse(users))
-  fetchMock.mockResolvedValueOnce(jsonResponse([assignedTask]))
-  fetchMock.mockResolvedValueOnce(jsonResponse(unassignedTask))
-  render(<App />)
-
-  fireEvent.click(await screen.findByRole('button', { name: 'Ändra ansvarig för Dammsuga' }))
-  fireEvent.change(screen.getByRole('combobox', { name: 'Ansvarig för Dammsuga' }), {
-    target: { value: '' },
-  })
-
-  await screen.findByText('Ta uppgift')
-  expect(fetchMock).toHaveBeenLastCalledWith(`/api/tasks/${tasks[0].id}/assignee`, {
-    method: 'PUT',
-    headers: { 'Content-Type': 'application/json' },
-    body: JSON.stringify({ assigneeId: null }),
-  })
-})
-
-test('misslyckad tilldelning behåller ansvarig och visar fel', async () => {
-  const assignedTask = { ...tasks[0], assignee: { id: users[1].id, name: users[1].name } }
-  window.localStorage.setItem('hemhub.activeUserId', users[0].id)
-  const fetchMock = vi.spyOn(globalThis, 'fetch')
-  fetchMock.mockResolvedValueOnce(jsonResponse(users))
-  fetchMock.mockResolvedValueOnce(jsonResponse([assignedTask]))
-  fetchMock.mockResolvedValueOnce(
-    jsonResponse({ code: 'USER_NOT_FOUND', message: 'Användaren finns inte.' }, 404),
-  )
-  render(<App />)
-
-  fireEvent.click(await screen.findByRole('button', { name: 'Ändra ansvarig för Dammsuga' }))
-  fireEvent.change(screen.getByRole('combobox', { name: 'Ansvarig för Dammsuga' }), {
-    target: { value: users[0].id },
-  })
-
-  expect(await screen.findByRole('alert')).toHaveTextContent('Användaren finns inte.')
-  expect(screen.getByRole('combobox', { name: 'Ansvarig för Dammsuga' })).toHaveValue(users[1].id)
 })
 
 test.each([
@@ -464,11 +321,9 @@ test('formulärdata bevaras när skapande av uppgift misslyckas', async () => {
   const title = screen.getByLabelText('Titel')
   const description = screen.getByLabelText('Beskrivning (valfri)')
   const points = screen.getByLabelText('Poäng')
-  const assignee = screen.getByLabelText('Tilldela')
   fireEvent.change(title, { target: { value: 'Dammsuga' } })
   fireEvent.change(description, { target: { value: 'Bottenvåningen' } })
   fireEvent.change(points, { target: { value: '7' } })
-  fireEvent.change(assignee, { target: { value: users[1].id } })
   fireEvent.click(screen.getByRole('button', { name: 'Skapa uppgift' }))
 
   expect(await screen.findByRole('alert')).toHaveTextContent('Uppgiften är ogiltig.')
@@ -476,7 +331,6 @@ test('formulärdata bevaras när skapande av uppgift misslyckas', async () => {
   expect(title).toHaveValue('Dammsuga')
   expect(description).toHaveValue('Bottenvåningen')
   expect(points).toHaveValue(7)
-  expect(assignee).toHaveValue(users[1].id)
 })
 
 function mockUsersAndTasks(userResponse: unknown, taskResponse: unknown) {

frontend/src/App.tsx

@@ -79,7 +79,7 @@ function App() {
   }
 
   if (activeUser) {
-    return <TaskBoard activeUserName={activeUser.name} users={users} onLogOut={logOut} />
+    return <TaskBoard activeUserName={activeUser.name} onLogOut={logOut} />
   }
 
   if (showCreateUser) {

frontend/src/TaskBoard.tsx

@@ -2,20 +2,12 @@ import { FormEvent, MouseEvent, useEffect, useRef, useState } from 'react'
 
 type TaskStatus = 'WAITING' | 'IN_PROGRESS' | 'COMPLETED'
 
-type UserSummary = {
-  id: string
-  name: string
-}
-
-type Assignee = UserSummary
-
 type Task = {
   id: string
   title: string
   description: string | null
   status: TaskStatus
   points: number
-  assignee: Assignee | null
   createdAt: string
 }
 
@@ -25,7 +17,6 @@ type ApiError = {
 
 type TaskBoardProps = {
   activeUserName: string
-  users: UserSummary[]
   onLogOut: () => void
 }
 
@@ -35,13 +26,10 @@ const columns: { status: TaskStatus; title: string }[] = [
   { status: 'COMPLETED', title: 'Klart' },
 ]
 
-function TaskBoard({ activeUserName, users, onLogOut }: TaskBoardProps) {
+function TaskBoard({ activeUserName, onLogOut }: TaskBoardProps) {
   const [tasks, setTasks] = useState<Task[]>([])
   const [loadState, setLoadState] = useState<'loading' | 'ready' | 'error'>('loading')
   const [showCreateTask, setShowCreateTask] = useState(false)
-  const [editingAssigneeTaskId, setEditingAssigneeTaskId] = useState<string | null>(null)
-  const [savingAssigneeTaskIds, setSavingAssigneeTaskIds] = useState<Set<string>>(new Set())
-  const [assignmentErrors, setAssignmentErrors] = useState<Record<string, string>>({})
 
   const loadTasks = async () => {
     setLoadState('loading')
@@ -64,49 +52,6 @@ function TaskBoard({ activeUserName, users, onLogOut }: TaskBoardProps) {
     void loadTasks()
   }, [])
 
-  const updateAssignee = async (task: Task, assigneeId: string) => {
-    if (savingAssigneeTaskIds.has(task.id)) {
-      return
-    }
-
-    setSavingAssigneeTaskIds((current) => new Set(current).add(task.id))
-    setAssignmentErrors((current) => ({ ...current, [task.id]: '' }))
-
-    try {
-      const response = await fetch(`/api/tasks/${task.id}/assignee`, {
-        method: 'PUT',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ assigneeId: assigneeId || null }),
-      })
-
-      if (!response.ok) {
-        const apiError = (await response.json().catch(() => ({}))) as ApiError
-        setAssignmentErrors((current) => ({
-          ...current,
-          [task.id]: apiError.message ?? 'Det gick inte att ändra ansvarig. Försök igen.',
-        }))
-        return
-      }
-
-      const updatedTask = (await response.json()) as Task
-      setTasks((current) =>
-        current.map((currentTask) => (currentTask.id === updatedTask.id ? updatedTask : currentTask)),
-      )
-      setEditingAssigneeTaskId(null)
-    } catch {
-      setAssignmentErrors((current) => ({
-        ...current,
-        [task.id]: 'Det gick inte att ändra ansvarig. Försök igen.',
-      }))
-    } finally {
-      setSavingAssigneeTaskIds((current) => {
-        const next = new Set(current)
-        next.delete(task.id)
-        return next
-      })
-    }
-  }
-
   return (
     <main className="task-app">
       <header className="app-header">
@@ -153,15 +98,6 @@ function TaskBoard({ activeUserName, users, onLogOut }: TaskBoardProps) {
                       <span className="points-badge">{task.points} p</span>
                     </div>
                     {task.description && <p>{task.description}</p>}
-                    <AssigneeControl
-                      task={task}
-                      users={users}
-                      editing={editingAssigneeTaskId === task.id}
-                      saving={savingAssigneeTaskIds.has(task.id)}
-                      error={assignmentErrors[task.id]}
-                      onEdit={() => setEditingAssigneeTaskId(task.id)}
-                      onChange={(assigneeId) => void updateAssignee(task, assigneeId)}
-                    />
                   </article>
                 ))}
             </div>
@@ -171,7 +107,6 @@ function TaskBoard({ activeUserName, users, onLogOut }: TaskBoardProps) {
 
       {showCreateTask && (
         <CreateTaskModal
-          users={users}
           onClose={() => setShowCreateTask(false)}
           onCreated={(task) => {
             setTasks((currentTasks) => [...currentTasks, task])
@@ -183,110 +118,15 @@ function TaskBoard({ activeUserName, users, onLogOut }: TaskBoardProps) {
   )
 }
 
-type AssigneeControlProps = {
-  task: Task
-  users: UserSummary[]
-  editing: boolean
-  saving: boolean
-  error?: string
-  onEdit: () => void
-  onChange: (assigneeId: string) => void
-}
-
-function UserIcon() {
-  return (
-    <svg
-      className="user-icon"
-      viewBox="0 0 24 24"
-      width="18"
-      height="18"
-      aria-hidden="true"
-    >
-      <circle cx="12" cy="8" r="3.5" fill="none" stroke="currentColor" strokeWidth="1.8" />
-      <path
-        d="M5 20c.5-4 3-6 7-6s6.5 2 7 6"
-        fill="none"
-        stroke="currentColor"
-        strokeWidth="1.8"
-        strokeLinecap="round"
-      />
-    </svg>
-  )
-}
-
-function AssigneeControl({
-  task,
-  users,
-  editing,
-  saving,
-  error,
-  onEdit,
-  onChange,
-}: AssigneeControlProps) {
-  const displayName = task.assignee?.name ?? (task.status === 'WAITING' ? 'Ta uppgift' : 'Otilldelad')
-
-  if (task.status !== 'WAITING') {
-    return (
-      <div className="task-assignee task-assignee-static">
-        <UserIcon />
-        <span>{displayName}</span>
-      </div>
-    )
-  }
-
-  return (
-    <div className="task-assignment">
-      {editing ? (
-        <label className="assignee-select-label">
-          <span className="visually-hidden">Ansvarig för {task.title}</span>
-          <UserIcon />
-          <select
-            aria-label={`Ansvarig för ${task.title}`}
-            value={task.assignee?.id ?? ''}
-            disabled={saving}
-            autoFocus
-            onChange={(event) => onChange(event.target.value)}
-          >
-            <option value="">Ingen</option>
-            {users.map((user) => (
-              <option key={user.id} value={user.id}>
-                {user.name}
-              </option>
-            ))}
-          </select>
-        </label>
-      ) : (
-        <button
-          type="button"
-          className="task-assignee task-assignee-button"
-          disabled={saving}
-          onClick={onEdit}
-          aria-label={`Ändra ansvarig för ${task.title}`}
-        >
-          <UserIcon />
-          <span>{displayName}</span>
-        </button>
-      )}
-      {error && (
-        <p className="assignment-error error" role="alert">
-          {error}
-        </p>
-      )}
-    </div>
-  )
-}
-
 type CreateTaskModalProps = {
-  users: UserSummary[]
   onClose: () => void
   onCreated: (task: Task) => void
 }
 
-function CreateTaskModal({ users, onClose, onCreated }: CreateTaskModalProps) {
+function CreateTaskModal({ onClose, onCreated }: CreateTaskModalProps) {
   const [title, setTitle] = useState('')
   const [description, setDescription] = useState('')
   const [points, setPoints] = useState('1')
-  const [assigneeId, setAssigneeId] = useState('')
   const [error, setError] = useState('')
   const [isSubmitting, setIsSubmitting] = useState(false)
   const isSubmittingRef = useRef(false)
@@ -351,7 +191,6 @@ function CreateTaskModal({ users, onClose, onCreated }: CreateTaskModalProps) {
           title: trimmedTitle,
           description: trimmedDescription || null,
           points: numericPoints,
-          assigneeId: assigneeId || null,
         }),
       })
 
@@ -425,23 +264,6 @@ function CreateTaskModal({ users, onClose, onCreated }: CreateTaskModalProps) {
             1–99 poäng beroende på hur tidskrävande, besvärlig eller viktig uppgiften är.
           </p>
 
-          <label className="field-label-uppercase" htmlFor="task-assignee">
-            Tilldela
-          </label>
-          <select
-            id="task-assignee"
-            value={assigneeId}
-            disabled={isSubmitting}
-            onChange={(event) => setAssigneeId(event.target.value)}
-          >
-            <option value="">Ingen</option>
-            {users.map((user) => (
-              <option key={user.id} value={user.id}>
-                {user.name}
-              </option>
-            ))}
-          </select>
-
           {error && (
             <p className="error" role="alert">
               {error}

frontend/src/styles.css

@@ -23,7 +23,6 @@ h1 {
 
 button,
 input,
-select,
 textarea {
   font: inherit;
 }
@@ -39,7 +38,6 @@ button {
 
 button:disabled,
 input:disabled,
-select:disabled,
 textarea:disabled {
   cursor: not-allowed;
   opacity: 0.65;
@@ -72,15 +70,6 @@ input {
   border-radius: 0.4rem;
 }
 
-select {
-  box-sizing: border-box;
-  width: 100%;
-  padding: 0.6rem;
-  border: 1px solid #9ca3af;
-  border-radius: 0.4rem;
-  background: white;
-}
-
 textarea {
   box-sizing: border-box;
   width: 100%;
@@ -185,68 +174,6 @@ textarea {
   margin: 0;
 }
 
-.task-assignment {
-  margin-top: 0.9rem;
-}
-
-.task-assignee {
-  display: inline-flex;
-  align-items: center;
-  gap: 0.4rem;
-  color: #475569;
-  font-size: 0.9rem;
-}
-
-.task-assignee-button {
-  padding: 0.25rem 0;
-  color: #2563eb;
-  background: transparent;
-}
-
-.task-assignee-static {
-  margin-top: 0.9rem;
-}
-
-.user-icon {
-  flex: 0 0 auto;
-}
-
-.assignee-select-label {
-  display: flex;
-  align-items: center;
-  gap: 0.4rem;
-}
-
-.assignee-select-label select {
-  width: auto;
-  min-width: 9rem;
-}
-
-.assignment-error {
-  margin-top: 0.4rem;
-  font-size: 0.85rem;
-}
-
-.field-label-uppercase {
-  color: #64748b;
-  font-size: 0.8rem;
-  font-weight: 700;
-  letter-spacing: 0.08em;
-  text-transform: uppercase;
-}
-
-.visually-hidden {
-  position: absolute;
-  width: 1px;
-  height: 1px;
-  padding: 0;
-  margin: -1px;
-  overflow: hidden;
-  clip: rect(0, 0, 0, 0);
-  white-space: nowrap;
-  border: 0;
-}
-
 .task-card-header {
   display: flex;
   align-items: flex-start;