Compare commits
26 Commits
9957383e88
...
main
| Author | SHA1 | Date | |
|---|---|---|---|
| b6460a3924 | |||
| 5df0146672 | |||
| f296d15446 | |||
| 5dea4c4027 | |||
| 2696195e74 | |||
| c3c64482c0 | |||
| ddd706536e | |||
| dd145db12f | |||
| 5b9e562722 | |||
| 65a6488c0b | |||
| 6570aad4a2 | |||
| 85afc3d3a3 | |||
| d78611f5f7 | |||
| aaebe888f3 | |||
| 2e62261f49 | |||
| 059d4da921 | |||
| 1654b54a22 | |||
| bad6b5afca | |||
| 170994b44d | |||
| a2ed8a64d6 | |||
| 1462f9dc0c | |||
| f9246d4463 | |||
| 2f7b99fb21 | |||
| 3f152eeccc | |||
| 050f248857 | |||
| 1ec7a72908 |
3
.gitignore
vendored
3
.gitignore
vendored
@ -1,5 +1,8 @@
|
||||
# Backend
|
||||
backend/target/
|
||||
backend/data/
|
||||
backend/*.mv.db
|
||||
backend/*.trace.db
|
||||
|
||||
# Frontend
|
||||
frontend/node_modules/
|
||||
|
||||
@ -9,4 +9,11 @@
|
||||
- Affärsregler ska senare säkerställas i backend och inte enbart i frontend.
|
||||
- Kod, tester och dokumentation ska hållas uppdaterade tillsammans.
|
||||
- Större arkitekturella beslut ska diskuteras innan de införs.
|
||||
|
||||
- Aktuell arkitektur, utvecklingsprocess, beslut och featurehistorik dokumenteras
|
||||
under `docs/`.
|
||||
- En feature ska uppdatera berörda dokument så att repositoryt förblir projektets
|
||||
facit även efter att feature-branchen har raderats.
|
||||
- Nästa feature ska väljas från `docs/roadmap.md`.
|
||||
- Roadmapen ska uppdateras innan en feature delas, flyttas, ersätts eller läggs
|
||||
till. En enskild dialog får inte etablera en alternativ featureplan utan att
|
||||
repositoryts roadmap uppdateras.
|
||||
|
||||
21
README.md
21
README.md
@ -6,6 +6,15 @@ innehåller två separata applikationer:
|
||||
- en backend byggd med Java 21, Spring Boot och Maven
|
||||
- en frontend byggd med React, TypeScript, Vite och pnpm
|
||||
|
||||
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, tilldela, ändra status på och permanent radera gemensamma
|
||||
hushållsuppgifter. Uppgiftskort kan flyttas mellan brädans statuskolumner med
|
||||
drag-and-drop eller med de befintliga statusknapparna. Radering kräver
|
||||
bekräftelse och genomförs först när backend har bekräftat operationen.
|
||||
|
||||
## Starta backend
|
||||
|
||||
Backend startar på port 8080.
|
||||
@ -44,3 +53,15 @@ cd frontend
|
||||
pnpm test
|
||||
```
|
||||
|
||||
## Dokumentation
|
||||
|
||||
Projektets aktuella arkitektur, utvecklingsprocess, övergripande beslut och
|
||||
featurehistorik finns i [`docs/`](docs/):
|
||||
|
||||
- [arkitektur](docs/architecture.md)
|
||||
- [roadmap och planerad featureordning](docs/roadmap.md)
|
||||
- [utvecklingsprocess](docs/development.md)
|
||||
- [arkitekturbeslut](docs/decisions/)
|
||||
- [implementerade features](docs/features/)
|
||||
|
||||
Dokumentationen ska uppdateras tillsammans med implementation och tester.
|
||||
|
||||
@ -26,6 +26,19 @@
|
||||
<groupId>org.springframework.boot</groupId>
|
||||
<artifactId>spring-boot-starter-web</artifactId>
|
||||
</dependency>
|
||||
<dependency>
|
||||
<groupId>org.springframework.boot</groupId>
|
||||
<artifactId>spring-boot-starter-data-jpa</artifactId>
|
||||
</dependency>
|
||||
<dependency>
|
||||
<groupId>org.springframework.boot</groupId>
|
||||
<artifactId>spring-boot-starter-flyway</artifactId>
|
||||
</dependency>
|
||||
<dependency>
|
||||
<groupId>com.h2database</groupId>
|
||||
<artifactId>h2</artifactId>
|
||||
<scope>runtime</scope>
|
||||
</dependency>
|
||||
<dependency>
|
||||
<groupId>org.springframework.boot</groupId>
|
||||
<artifactId>spring-boot-starter-test</artifactId>
|
||||
@ -42,4 +55,3 @@
|
||||
</plugins>
|
||||
</build>
|
||||
</project>
|
||||
|
||||
|
||||
5
backend/src/main/java/se/rubble/hemhub/api/ApiError.java
Normal file
5
backend/src/main/java/se/rubble/hemhub/api/ApiError.java
Normal file
@ -0,0 +1,5 @@
|
||||
package se.rubble.hemhub.api;
|
||||
|
||||
public record ApiError(String code, String message) {
|
||||
}
|
||||
|
||||
@ -0,0 +1,85 @@
|
||||
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.InvalidTaskStatusException;
|
||||
import se.rubble.hemhub.task.AssigneeNotFoundException;
|
||||
import se.rubble.hemhub.task.TaskNotFoundException;
|
||||
import se.rubble.hemhub.task.TaskRequiresAssigneeException;
|
||||
import se.rubble.hemhub.user.InvalidUserNameException;
|
||||
import se.rubble.hemhub.user.UserNameAlreadyExistsException;
|
||||
|
||||
@RestControllerAdvice
|
||||
public class ApiExceptionHandler {
|
||||
|
||||
@ExceptionHandler(InvalidUserNameException.class)
|
||||
public ResponseEntity<ApiError> handleInvalidUserName() {
|
||||
return ResponseEntity.badRequest()
|
||||
.body(new ApiError(
|
||||
"INVALID_USER_NAME",
|
||||
"Namnet måste innehålla mellan 1 och 50 tecken."));
|
||||
}
|
||||
|
||||
@ExceptionHandler(UserNameAlreadyExistsException.class)
|
||||
public ResponseEntity<ApiError> handleDuplicateUserName() {
|
||||
return ResponseEntity.status(HttpStatus.CONFLICT)
|
||||
.body(new ApiError(
|
||||
"USER_NAME_ALREADY_EXISTS",
|
||||
"En användare med det namnet finns redan."));
|
||||
}
|
||||
|
||||
@ExceptionHandler(InvalidTaskException.class)
|
||||
public ResponseEntity<ApiError> handleInvalidTask(InvalidTaskException exception) {
|
||||
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(InvalidTaskStatusException.class)
|
||||
public ResponseEntity<ApiError> handleInvalidTaskStatus() {
|
||||
return ResponseEntity.badRequest()
|
||||
.body(new ApiError(
|
||||
"INVALID_TASK_STATUS",
|
||||
"Status måste vara WAITING, IN_PROGRESS eller COMPLETED."));
|
||||
}
|
||||
|
||||
@ExceptionHandler(TaskRequiresAssigneeException.class)
|
||||
public ResponseEntity<ApiError> handleTaskRequiresAssignee() {
|
||||
return ResponseEntity.status(HttpStatus.CONFLICT)
|
||||
.body(new ApiError(
|
||||
"TASK_REQUIRES_ASSIGNEE",
|
||||
"En pågående uppgift måste ha en ansvarig."));
|
||||
}
|
||||
}
|
||||
@ -0,0 +1,4 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
public class AssigneeNotFoundException extends RuntimeException {
|
||||
}
|
||||
@ -0,0 +1,22 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
import tools.jackson.databind.JsonNode;
|
||||
|
||||
public record CreateTaskRequest(
|
||||
String title,
|
||||
String description,
|
||||
JsonNode points,
|
||||
JsonNode assigneeId) {
|
||||
|
||||
Integer integerPoints() {
|
||||
if (points == null || !points.isIntegralNumber() || !points.canConvertToInt()) {
|
||||
return null;
|
||||
}
|
||||
|
||||
return points.intValue();
|
||||
}
|
||||
|
||||
UUIDValue parsedAssigneeId() {
|
||||
return UUIDValue.optional(assigneeId);
|
||||
}
|
||||
}
|
||||
@ -0,0 +1,8 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
public class InvalidTaskAssignmentException extends RuntimeException {
|
||||
|
||||
public InvalidTaskAssignmentException(String message) {
|
||||
super(message);
|
||||
}
|
||||
}
|
||||
@ -0,0 +1,9 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
public class InvalidTaskException extends RuntimeException {
|
||||
|
||||
public InvalidTaskException(String message) {
|
||||
super(message);
|
||||
}
|
||||
}
|
||||
|
||||
@ -0,0 +1,4 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
public class InvalidTaskStatusException extends RuntimeException {
|
||||
}
|
||||
118
backend/src/main/java/se/rubble/hemhub/task/Task.java
Normal file
118
backend/src/main/java/se/rubble/hemhub/task/Task.java
Normal file
@ -0,0 +1,118 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
import java.time.Instant;
|
||||
import java.util.UUID;
|
||||
|
||||
import jakarta.persistence.Column;
|
||||
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")
|
||||
class Task {
|
||||
|
||||
@Id
|
||||
private UUID id;
|
||||
|
||||
@Column(nullable = false, length = 100)
|
||||
private String title;
|
||||
|
||||
@Column(length = 500)
|
||||
private String description;
|
||||
|
||||
@Enumerated(EnumType.STRING)
|
||||
@Column(nullable = false, length = 20)
|
||||
private TaskStatus status;
|
||||
|
||||
@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;
|
||||
|
||||
protected Task() {
|
||||
}
|
||||
|
||||
Task(
|
||||
UUID id,
|
||||
String title,
|
||||
String description,
|
||||
TaskStatus status,
|
||||
int points,
|
||||
User assignee,
|
||||
Instant createdAt) {
|
||||
if (points < 1 || points > 99) {
|
||||
throw new InvalidTaskException(
|
||||
"Poäng måste vara ett heltal mellan 1 och 99.");
|
||||
}
|
||||
if (status == TaskStatus.IN_PROGRESS && assignee == null) {
|
||||
throw new TaskRequiresAssigneeException();
|
||||
}
|
||||
|
||||
this.id = id;
|
||||
this.title = title;
|
||||
this.description = description;
|
||||
this.status = status;
|
||||
this.points = points;
|
||||
this.assignee = assignee;
|
||||
this.createdAt = createdAt;
|
||||
}
|
||||
|
||||
UUID getId() {
|
||||
return id;
|
||||
}
|
||||
|
||||
String getTitle() {
|
||||
return title;
|
||||
}
|
||||
|
||||
String getDescription() {
|
||||
return description;
|
||||
}
|
||||
|
||||
TaskStatus getStatus() {
|
||||
return status;
|
||||
}
|
||||
|
||||
int getPoints() {
|
||||
return points;
|
||||
}
|
||||
|
||||
User getAssignee() {
|
||||
return assignee;
|
||||
}
|
||||
|
||||
void changeAssignee(User assignee) {
|
||||
if (status == TaskStatus.IN_PROGRESS && assignee == null) {
|
||||
throw new TaskRequiresAssigneeException();
|
||||
}
|
||||
|
||||
this.assignee = assignee;
|
||||
}
|
||||
|
||||
void changeStatus(TaskStatus targetStatus, User automaticAssignee) {
|
||||
if (targetStatus == TaskStatus.IN_PROGRESS && assignee == null) {
|
||||
if (automaticAssignee == null) {
|
||||
throw new TaskRequiresAssigneeException();
|
||||
}
|
||||
assignee = automaticAssignee;
|
||||
}
|
||||
|
||||
status = targetStatus;
|
||||
}
|
||||
|
||||
Instant getCreatedAt() {
|
||||
return createdAt;
|
||||
}
|
||||
}
|
||||
@ -0,0 +1,75 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
import java.util.List;
|
||||
import java.util.UUID;
|
||||
|
||||
import org.springframework.http.HttpStatus;
|
||||
import org.springframework.web.bind.annotation.DeleteMapping;
|
||||
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;
|
||||
import org.springframework.web.bind.annotation.RestController;
|
||||
|
||||
@RestController
|
||||
@RequestMapping("/api/tasks")
|
||||
public class TaskController {
|
||||
|
||||
private final TaskService taskService;
|
||||
|
||||
TaskController(TaskService taskService) {
|
||||
this.taskService = taskService;
|
||||
}
|
||||
|
||||
@GetMapping
|
||||
public List<TaskResponse> findAll() {
|
||||
return taskService.findAll();
|
||||
}
|
||||
|
||||
@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(),
|
||||
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());
|
||||
}
|
||||
|
||||
@PutMapping("/{taskId}/status")
|
||||
public TaskResponse updateStatus(
|
||||
@PathVariable UUID taskId,
|
||||
@RequestBody(required = false) UpdateTaskStatusRequest request) {
|
||||
if (request == null) {
|
||||
throw new InvalidTaskStatusException();
|
||||
}
|
||||
|
||||
return taskService.updateStatus(
|
||||
taskId,
|
||||
request.parsedStatus(),
|
||||
request.parsedActiveUserId().value());
|
||||
}
|
||||
|
||||
@DeleteMapping("/{taskId}")
|
||||
@ResponseStatus(HttpStatus.NO_CONTENT)
|
||||
public void delete(@PathVariable UUID taskId) {
|
||||
taskService.delete(taskId);
|
||||
}
|
||||
}
|
||||
@ -0,0 +1,4 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
public class TaskNotFoundException extends RuntimeException {
|
||||
}
|
||||
@ -0,0 +1,16 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
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);
|
||||
}
|
||||
@ -0,0 +1,4 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
public class TaskRequiresAssigneeException extends RuntimeException {
|
||||
}
|
||||
@ -0,0 +1,32 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
import java.time.Instant;
|
||||
import java.util.UUID;
|
||||
|
||||
public record TaskResponse(
|
||||
UUID id,
|
||||
String title,
|
||||
String description,
|
||||
TaskStatus status,
|
||||
int points,
|
||||
AssigneeResponse assignee,
|
||||
Instant createdAt) {
|
||||
|
||||
static TaskResponse from(Task task) {
|
||||
return new TaskResponse(
|
||||
task.getId(),
|
||||
task.getTitle(),
|
||||
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());
|
||||
}
|
||||
}
|
||||
}
|
||||
135
backend/src/main/java/se/rubble/hemhub/task/TaskService.java
Normal file
135
backend/src/main/java/se/rubble/hemhub/task/TaskService.java
Normal file
@ -0,0 +1,135 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
import java.time.Clock;
|
||||
import java.time.Instant;
|
||||
import java.util.List;
|
||||
import java.util.UUID;
|
||||
|
||||
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) {
|
||||
this(taskRepository, userRepository, Clock.systemUTC());
|
||||
}
|
||||
|
||||
TaskService(TaskRepository taskRepository, UserRepository userRepository, Clock clock) {
|
||||
this.taskRepository = taskRepository;
|
||||
this.userRepository = userRepository;
|
||||
this.clock = clock;
|
||||
}
|
||||
|
||||
@Transactional(readOnly = true)
|
||||
List<TaskResponse> findAll() {
|
||||
return taskRepository.findAllByOrderByCreatedAtAscIdAsc().stream()
|
||||
.map(TaskResponse::from)
|
||||
.toList();
|
||||
}
|
||||
|
||||
@Transactional
|
||||
TaskResponse create(
|
||||
String requestedTitle,
|
||||
String requestedDescription,
|
||||
Integer requestedPoints,
|
||||
UUID requestedAssigneeId) {
|
||||
String title = requestedTitle == null ? "" : requestedTitle.trim();
|
||||
String description = normalizeDescription(requestedDescription);
|
||||
|
||||
if (title.isEmpty() || codePointLength(title) > 100) {
|
||||
throw new InvalidTaskException(
|
||||
"Titeln måste innehålla mellan 1 och 100 tecken.");
|
||||
}
|
||||
|
||||
if (description != null && codePointLength(description) > 500) {
|
||||
throw new InvalidTaskException(
|
||||
"Beskrivningen får innehålla högst 500 tecken.");
|
||||
}
|
||||
|
||||
if (requestedPoints == null) {
|
||||
throw new InvalidTaskException(
|
||||
"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);
|
||||
User assignee = findAssignee(requestedAssigneeId);
|
||||
|
||||
task.changeAssignee(assignee);
|
||||
return TaskResponse.from(task);
|
||||
}
|
||||
|
||||
@Transactional
|
||||
TaskResponse updateStatus(
|
||||
UUID taskId,
|
||||
TaskStatus targetStatus,
|
||||
UUID activeUserId) {
|
||||
Task task = taskRepository.findOneById(taskId)
|
||||
.orElseThrow(TaskNotFoundException::new);
|
||||
|
||||
User automaticAssignee = null;
|
||||
if (targetStatus == TaskStatus.IN_PROGRESS && task.getAssignee() == null) {
|
||||
if (activeUserId == null) {
|
||||
throw new TaskRequiresAssigneeException();
|
||||
}
|
||||
automaticAssignee = findAssignee(activeUserId);
|
||||
}
|
||||
|
||||
task.changeStatus(targetStatus, automaticAssignee);
|
||||
return TaskResponse.from(task);
|
||||
}
|
||||
|
||||
@Transactional
|
||||
void delete(UUID taskId) {
|
||||
Task task = taskRepository.findById(taskId)
|
||||
.orElseThrow(TaskNotFoundException::new);
|
||||
taskRepository.delete(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;
|
||||
}
|
||||
|
||||
String description = requestedDescription.trim();
|
||||
return description.isEmpty() ? null : description;
|
||||
}
|
||||
|
||||
private static int codePointLength(String value) {
|
||||
return value.codePointCount(0, value.length());
|
||||
}
|
||||
}
|
||||
@ -0,0 +1,8 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
public enum TaskStatus {
|
||||
WAITING,
|
||||
IN_PROGRESS,
|
||||
COMPLETED
|
||||
}
|
||||
|
||||
25
backend/src/main/java/se/rubble/hemhub/task/UUIDValue.java
Normal file
25
backend/src/main/java/se/rubble/hemhub/task/UUIDValue.java
Normal file
@ -0,0 +1,25 @@
|
||||
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.");
|
||||
}
|
||||
}
|
||||
}
|
||||
@ -0,0 +1,16 @@
|
||||
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;
|
||||
}
|
||||
}
|
||||
@ -0,0 +1,23 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
import tools.jackson.databind.JsonNode;
|
||||
import tools.jackson.databind.node.JsonNodeType;
|
||||
|
||||
public record UpdateTaskStatusRequest(JsonNode status, JsonNode activeUserId) {
|
||||
|
||||
TaskStatus parsedStatus() {
|
||||
if (status == null || status.getNodeType() != JsonNodeType.STRING) {
|
||||
throw new InvalidTaskStatusException();
|
||||
}
|
||||
|
||||
try {
|
||||
return TaskStatus.valueOf(status.stringValue());
|
||||
} catch (IllegalArgumentException exception) {
|
||||
throw new InvalidTaskStatusException();
|
||||
}
|
||||
}
|
||||
|
||||
UUIDValue parsedActiveUserId() {
|
||||
return UUIDValue.optional(activeUserId);
|
||||
}
|
||||
}
|
||||
@ -0,0 +1,5 @@
|
||||
package se.rubble.hemhub.user;
|
||||
|
||||
public record CreateUserRequest(String name) {
|
||||
}
|
||||
|
||||
@ -0,0 +1,5 @@
|
||||
package se.rubble.hemhub.user;
|
||||
|
||||
public class InvalidUserNameException extends RuntimeException {
|
||||
}
|
||||
|
||||
48
backend/src/main/java/se/rubble/hemhub/user/User.java
Normal file
48
backend/src/main/java/se/rubble/hemhub/user/User.java
Normal file
@ -0,0 +1,48 @@
|
||||
package se.rubble.hemhub.user;
|
||||
|
||||
import java.time.Instant;
|
||||
import java.util.UUID;
|
||||
|
||||
import jakarta.persistence.Column;
|
||||
import jakarta.persistence.Entity;
|
||||
import jakarta.persistence.Id;
|
||||
import jakarta.persistence.Table;
|
||||
|
||||
@Entity
|
||||
@Table(name = "app_user")
|
||||
public class User {
|
||||
|
||||
@Id
|
||||
private UUID id;
|
||||
|
||||
@Column(nullable = false, length = 50)
|
||||
private String name;
|
||||
|
||||
@Column(name = "normalized_name", nullable = false, length = 150, unique = true)
|
||||
private String normalizedName;
|
||||
|
||||
@Column(name = "created_at", nullable = false)
|
||||
private Instant createdAt;
|
||||
|
||||
protected User() {
|
||||
}
|
||||
|
||||
User(UUID id, String name, String normalizedName, Instant createdAt) {
|
||||
this.id = id;
|
||||
this.name = name;
|
||||
this.normalizedName = normalizedName;
|
||||
this.createdAt = createdAt;
|
||||
}
|
||||
|
||||
public UUID getId() {
|
||||
return id;
|
||||
}
|
||||
|
||||
public String getName() {
|
||||
return name;
|
||||
}
|
||||
|
||||
Instant getCreatedAt() {
|
||||
return createdAt;
|
||||
}
|
||||
}
|
||||
@ -0,0 +1,34 @@
|
||||
package se.rubble.hemhub.user;
|
||||
|
||||
import java.util.List;
|
||||
|
||||
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.RequestBody;
|
||||
import org.springframework.web.bind.annotation.RequestMapping;
|
||||
import org.springframework.web.bind.annotation.ResponseStatus;
|
||||
import org.springframework.web.bind.annotation.RestController;
|
||||
|
||||
@RestController
|
||||
@RequestMapping("/api/users")
|
||||
public class UserController {
|
||||
|
||||
private final UserService userService;
|
||||
|
||||
UserController(UserService userService) {
|
||||
this.userService = userService;
|
||||
}
|
||||
|
||||
@GetMapping
|
||||
public List<UserResponse> findAll() {
|
||||
return userService.findAll();
|
||||
}
|
||||
|
||||
@PostMapping
|
||||
@ResponseStatus(HttpStatus.CREATED)
|
||||
public UserResponse create(@RequestBody(required = false) CreateUserRequest request) {
|
||||
return userService.create(request == null ? null : request.name());
|
||||
}
|
||||
}
|
||||
|
||||
@ -0,0 +1,5 @@
|
||||
package se.rubble.hemhub.user;
|
||||
|
||||
public class UserNameAlreadyExistsException extends RuntimeException {
|
||||
}
|
||||
|
||||
@ -0,0 +1,10 @@
|
||||
package se.rubble.hemhub.user;
|
||||
|
||||
import java.util.UUID;
|
||||
|
||||
import org.springframework.data.jpa.repository.JpaRepository;
|
||||
|
||||
public interface UserRepository extends JpaRepository<User, UUID> {
|
||||
|
||||
boolean existsByNormalizedName(String normalizedName);
|
||||
}
|
||||
@ -0,0 +1,12 @@
|
||||
package se.rubble.hemhub.user;
|
||||
|
||||
import java.time.Instant;
|
||||
import java.util.UUID;
|
||||
|
||||
public record UserResponse(UUID id, String name, Instant createdAt) {
|
||||
|
||||
static UserResponse from(User user) {
|
||||
return new UserResponse(user.getId(), user.getName(), user.getCreatedAt());
|
||||
}
|
||||
}
|
||||
|
||||
66
backend/src/main/java/se/rubble/hemhub/user/UserService.java
Normal file
66
backend/src/main/java/se/rubble/hemhub/user/UserService.java
Normal file
@ -0,0 +1,66 @@
|
||||
package se.rubble.hemhub.user;
|
||||
|
||||
import java.time.Clock;
|
||||
import java.time.Instant;
|
||||
import java.util.Comparator;
|
||||
import java.util.List;
|
||||
import java.util.Locale;
|
||||
import java.util.UUID;
|
||||
|
||||
import org.springframework.beans.factory.annotation.Autowired;
|
||||
import org.springframework.dao.DataIntegrityViolationException;
|
||||
import org.springframework.stereotype.Service;
|
||||
import org.springframework.transaction.annotation.Transactional;
|
||||
|
||||
@Service
|
||||
class UserService {
|
||||
|
||||
private static final Comparator<User> BY_DISPLAY_NAME =
|
||||
Comparator.comparing(User::getName, String.CASE_INSENSITIVE_ORDER)
|
||||
.thenComparing(User::getName)
|
||||
.thenComparing(User::getId);
|
||||
|
||||
private final UserRepository userRepository;
|
||||
private final Clock clock;
|
||||
|
||||
@Autowired
|
||||
UserService(UserRepository userRepository) {
|
||||
this(userRepository, Clock.systemUTC());
|
||||
}
|
||||
|
||||
UserService(UserRepository userRepository, Clock clock) {
|
||||
this.userRepository = userRepository;
|
||||
this.clock = clock;
|
||||
}
|
||||
|
||||
@Transactional(readOnly = true)
|
||||
List<UserResponse> findAll() {
|
||||
return userRepository.findAll().stream()
|
||||
.sorted(BY_DISPLAY_NAME)
|
||||
.map(UserResponse::from)
|
||||
.toList();
|
||||
}
|
||||
|
||||
@Transactional
|
||||
UserResponse create(String requestedName) {
|
||||
String name = requestedName == null ? "" : requestedName.trim();
|
||||
|
||||
if (name.isEmpty() || name.codePointCount(0, name.length()) > 50) {
|
||||
throw new InvalidUserNameException();
|
||||
}
|
||||
|
||||
String normalizedName = name.toLowerCase(Locale.ROOT);
|
||||
|
||||
if (userRepository.existsByNormalizedName(normalizedName)) {
|
||||
throw new UserNameAlreadyExistsException();
|
||||
}
|
||||
|
||||
User user = new User(UUID.randomUUID(), name, normalizedName, Instant.now(clock));
|
||||
|
||||
try {
|
||||
return UserResponse.from(userRepository.saveAndFlush(user));
|
||||
} catch (DataIntegrityViolationException exception) {
|
||||
throw new UserNameAlreadyExistsException();
|
||||
}
|
||||
}
|
||||
}
|
||||
6
backend/src/main/resources/application.properties
Normal file
6
backend/src/main/resources/application.properties
Normal file
@ -0,0 +1,6 @@
|
||||
spring.datasource.url=jdbc:h2:mem:hemhub;MODE=PostgreSQL;DATABASE_TO_LOWER=TRUE;DEFAULT_NULL_ORDERING=HIGH;DB_CLOSE_DELAY=-1
|
||||
spring.datasource.username=sa
|
||||
spring.datasource.password=
|
||||
spring.jpa.hibernate.ddl-auto=validate
|
||||
spring.jpa.open-in-view=false
|
||||
spring.flyway.enabled=true
|
||||
@ -0,0 +1,7 @@
|
||||
CREATE TABLE app_user (
|
||||
id UUID PRIMARY KEY,
|
||||
name VARCHAR(50) NOT NULL,
|
||||
normalized_name VARCHAR(150) NOT NULL,
|
||||
created_at TIMESTAMP WITH TIME ZONE NOT NULL,
|
||||
CONSTRAINT uk_app_user_normalized_name UNIQUE (normalized_name)
|
||||
);
|
||||
@ -0,0 +1,8 @@
|
||||
CREATE TABLE task (
|
||||
id UUID PRIMARY KEY,
|
||||
title VARCHAR(100) NOT NULL,
|
||||
description VARCHAR(500),
|
||||
status VARCHAR(20) NOT NULL,
|
||||
created_at TIMESTAMP WITH TIME ZONE NOT NULL
|
||||
);
|
||||
|
||||
@ -0,0 +1,8 @@
|
||||
ALTER TABLE task ADD COLUMN points INTEGER;
|
||||
|
||||
UPDATE task SET points = 1 WHERE points IS NULL;
|
||||
|
||||
ALTER TABLE task ALTER COLUMN points SET NOT NULL;
|
||||
|
||||
ALTER TABLE task
|
||||
ADD CONSTRAINT ck_task_points_range CHECK (points BETWEEN 1 AND 99);
|
||||
@ -0,0 +1,6 @@
|
||||
ALTER TABLE task
|
||||
ADD COLUMN assignee_id UUID;
|
||||
|
||||
ALTER TABLE task
|
||||
ADD CONSTRAINT fk_task_assignee
|
||||
FOREIGN KEY (assignee_id) REFERENCES app_user (id);
|
||||
342
backend/src/test/java/se/rubble/hemhub/task/TaskApiTest.java
Normal file
342
backend/src/test/java/se/rubble/hemhub/task/TaskApiTest.java
Normal file
@ -0,0 +1,342 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
import java.time.Instant;
|
||||
import java.util.UUID;
|
||||
|
||||
import org.junit.jupiter.api.BeforeEach;
|
||||
import org.junit.jupiter.api.Test;
|
||||
import org.springframework.beans.factory.annotation.Autowired;
|
||||
import org.springframework.boot.test.context.SpringBootTest;
|
||||
import org.springframework.http.MediaType;
|
||||
import org.springframework.test.web.servlet.MockMvc;
|
||||
import org.springframework.test.web.servlet.ResultActions;
|
||||
import org.springframework.test.web.servlet.setup.MockMvcBuilders;
|
||||
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;
|
||||
|
||||
@SpringBootTest
|
||||
class TaskApiTest {
|
||||
|
||||
@Autowired
|
||||
private WebApplicationContext context;
|
||||
|
||||
@Autowired
|
||||
private TaskRepository taskRepository;
|
||||
|
||||
private MockMvc mockMvc;
|
||||
|
||||
@BeforeEach
|
||||
void setUp() {
|
||||
taskRepository.deleteAll();
|
||||
mockMvc = MockMvcBuilders.webAppContextSetup(context).build();
|
||||
}
|
||||
|
||||
@Test
|
||||
void createsWaitingTaskWithTrimmedValues() throws Exception {
|
||||
mockMvc.perform(post("/api/tasks")
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.content("""
|
||||
{
|
||||
"title": " Dammsuga ",
|
||||
"description": " Bottenvåningen ",
|
||||
"points": 7
|
||||
}
|
||||
"""))
|
||||
.andExpect(status().isCreated())
|
||||
.andExpect(jsonPath("$.id").isString())
|
||||
.andExpect(jsonPath("$.title").value("Dammsuga"))
|
||||
.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"))
|
||||
.andExpect(status().isOk())
|
||||
.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")
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.content("""
|
||||
{"title": "Dammsuga", "description": " ", "points": 1}
|
||||
"""))
|
||||
.andExpect(status().isCreated())
|
||||
.andExpect(jsonPath("$.description").value((Object) null));
|
||||
}
|
||||
|
||||
@Test
|
||||
void rejectsBlankAndTooLongTitles() throws Exception {
|
||||
assertInvalidTask("""
|
||||
{"title": " ", "points": 1}
|
||||
""");
|
||||
assertInvalidTask(
|
||||
"{\"title\": \"%s\", \"points\": 1}".formatted("a".repeat(101)));
|
||||
}
|
||||
|
||||
@Test
|
||||
void rejectsTooLongDescription() throws Exception {
|
||||
assertInvalidTask("""
|
||||
{"title": "Dammsuga", "description": "%s", "points": 1}
|
||||
""".formatted("a".repeat(501)));
|
||||
}
|
||||
|
||||
@Test
|
||||
void acceptsPointBoundaries() throws Exception {
|
||||
createTaskWithPoints(1)
|
||||
.andExpect(status().isCreated())
|
||||
.andExpect(jsonPath("$.points").value(1));
|
||||
createTaskWithPoints(99)
|
||||
.andExpect(status().isCreated())
|
||||
.andExpect(jsonPath("$.points").value(99));
|
||||
}
|
||||
|
||||
@Test
|
||||
void rejectsMissingNullAndOutOfRangePoints() throws Exception {
|
||||
assertInvalidTask("""
|
||||
{"title": "Saknas"}
|
||||
""");
|
||||
assertInvalidTask("""
|
||||
{"title": "Null", "points": null}
|
||||
""");
|
||||
assertInvalidTask("""
|
||||
{"title": "Noll", "points": 0}
|
||||
""");
|
||||
assertInvalidTask("""
|
||||
{"title": "Negativ", "points": -1}
|
||||
""");
|
||||
assertInvalidTask("""
|
||||
{"title": "För stor", "points": 100}
|
||||
""");
|
||||
}
|
||||
|
||||
@Test
|
||||
void rejectsNonIntegerPoints() throws Exception {
|
||||
assertInvalidTask("""
|
||||
{"title": "Decimal", "points": 1.5}
|
||||
""");
|
||||
assertInvalidTask("""
|
||||
{"title": "Text", "points": "sju"}
|
||||
""");
|
||||
}
|
||||
|
||||
@Test
|
||||
void listsTasksOldestFirstWithIdAsTieBreaker() throws Exception {
|
||||
Instant older = Instant.parse("2026-07-24T10:00:00Z");
|
||||
Instant newer = Instant.parse("2026-07-24T11:00:00Z");
|
||||
UUID firstId = UUID.fromString("00000000-0000-0000-0000-000000000001");
|
||||
UUID secondId = UUID.fromString("00000000-0000-0000-0000-000000000002");
|
||||
UUID newestId = UUID.fromString("00000000-0000-0000-0000-000000000003");
|
||||
|
||||
taskRepository.save(new Task(
|
||||
newestId, "Nyast", null, TaskStatus.WAITING, 3, null, newer));
|
||||
taskRepository.save(new Task(
|
||||
secondId, "Andra", null, TaskStatus.WAITING, 2, null, older));
|
||||
taskRepository.save(new Task(
|
||||
firstId, "Första", null, TaskStatus.COMPLETED, 1, null, older));
|
||||
|
||||
mockMvc.perform(get("/api/tasks"))
|
||||
.andExpect(status().isOk())
|
||||
.andExpect(jsonPath("$[0].title").value("Första"))
|
||||
.andExpect(jsonPath("$[0].points").value(1))
|
||||
.andExpect(jsonPath("$[1].title").value("Andra"))
|
||||
.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"));
|
||||
}
|
||||
|
||||
private ResultActions createTaskWithPoints(int points) throws Exception {
|
||||
return mockMvc.perform(post("/api/tasks")
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.content("{\"title\": \"Uppgift %d\", \"points\": %d}"
|
||||
.formatted(points, points)));
|
||||
}
|
||||
|
||||
private void assertInvalidTask(String body) throws Exception {
|
||||
mockMvc.perform(post("/api/tasks")
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.content(body))
|
||||
.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));
|
||||
}
|
||||
}
|
||||
@ -0,0 +1,128 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
import java.time.Instant;
|
||||
import java.util.UUID;
|
||||
|
||||
import org.junit.jupiter.api.BeforeEach;
|
||||
import org.junit.jupiter.api.Test;
|
||||
import org.junit.jupiter.params.ParameterizedTest;
|
||||
import org.junit.jupiter.params.provider.EnumSource;
|
||||
import org.springframework.beans.factory.annotation.Autowired;
|
||||
import org.springframework.boot.test.context.SpringBootTest;
|
||||
import org.springframework.http.MediaType;
|
||||
import org.springframework.test.web.servlet.MockMvc;
|
||||
import org.springframework.test.web.servlet.setup.MockMvcBuilders;
|
||||
import org.springframework.web.context.WebApplicationContext;
|
||||
|
||||
import se.rubble.hemhub.user.User;
|
||||
import se.rubble.hemhub.user.UserRepository;
|
||||
|
||||
import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.delete;
|
||||
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.result.MockMvcResultMatchers.content;
|
||||
import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath;
|
||||
import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
|
||||
|
||||
@SpringBootTest
|
||||
class TaskDeletionApiTest {
|
||||
|
||||
@Autowired
|
||||
private WebApplicationContext context;
|
||||
|
||||
@Autowired
|
||||
private TaskRepository taskRepository;
|
||||
|
||||
@Autowired
|
||||
private UserRepository userRepository;
|
||||
|
||||
private MockMvc mockMvc;
|
||||
|
||||
@BeforeEach
|
||||
void setUp() {
|
||||
taskRepository.deleteAll();
|
||||
userRepository.deleteAll();
|
||||
mockMvc = MockMvcBuilders.webAppContextSetup(context).build();
|
||||
}
|
||||
|
||||
@ParameterizedTest
|
||||
@EnumSource(TaskStatus.class)
|
||||
void deletesTaskInEveryStatus(TaskStatus statusValue) throws Exception {
|
||||
User assignee = createUser();
|
||||
Task task = saveTask(statusValue, assignee);
|
||||
|
||||
mockMvc.perform(delete("/api/tasks/{taskId}", task.getId()))
|
||||
.andExpect(status().isNoContent())
|
||||
.andExpect(content().string(""));
|
||||
|
||||
mockMvc.perform(get("/api/tasks"))
|
||||
.andExpect(status().isOk())
|
||||
.andExpect(jsonPath("$").isEmpty());
|
||||
org.junit.jupiter.api.Assertions.assertTrue(userRepository.existsById(assignee.getId()));
|
||||
}
|
||||
|
||||
@Test
|
||||
void deletesOnlyRequestedTask() throws Exception {
|
||||
User assignee = createUser();
|
||||
Task deleted = saveTask(TaskStatus.WAITING, assignee);
|
||||
Task remaining = saveTask(TaskStatus.COMPLETED, null);
|
||||
|
||||
mockMvc.perform(delete("/api/tasks/{taskId}", deleted.getId()))
|
||||
.andExpect(status().isNoContent());
|
||||
|
||||
mockMvc.perform(get("/api/tasks"))
|
||||
.andExpect(status().isOk())
|
||||
.andExpect(jsonPath("$.length()").value(1))
|
||||
.andExpect(jsonPath("$[0].id").value(remaining.getId().toString()))
|
||||
.andExpect(jsonPath("$[0].status").value("COMPLETED"));
|
||||
org.junit.jupiter.api.Assertions.assertTrue(userRepository.existsById(assignee.getId()));
|
||||
}
|
||||
|
||||
@Test
|
||||
void returnsNotFoundForUnknownAndAlreadyDeletedTask() throws Exception {
|
||||
Task task = saveTask(TaskStatus.WAITING, null);
|
||||
|
||||
mockMvc.perform(delete("/api/tasks/{taskId}", task.getId()))
|
||||
.andExpect(status().isNoContent());
|
||||
mockMvc.perform(delete("/api/tasks/{taskId}", task.getId()))
|
||||
.andExpect(status().isNotFound())
|
||||
.andExpect(jsonPath("$.code").value("TASK_NOT_FOUND"));
|
||||
mockMvc.perform(delete(
|
||||
"/api/tasks/{taskId}",
|
||||
"00000000-0000-0000-0000-000000000099"))
|
||||
.andExpect(status().isNotFound())
|
||||
.andExpect(jsonPath("$.code").value("TASK_NOT_FOUND"));
|
||||
}
|
||||
|
||||
@Test
|
||||
void keepsExistingBadRequestForInvalidUuid() throws Exception {
|
||||
mockMvc.perform(delete("/api/tasks/{taskId}", "inte-ett-uuid"))
|
||||
.andExpect(status().isBadRequest())
|
||||
.andExpect(jsonPath("$.code").value("INVALID_TASK_ASSIGNMENT"));
|
||||
}
|
||||
|
||||
private User createUser() throws Exception {
|
||||
String response = mockMvc.perform(post("/api/users")
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.content("""
|
||||
{"name": "Urban"}
|
||||
"""))
|
||||
.andExpect(status().isCreated())
|
||||
.andReturn()
|
||||
.getResponse()
|
||||
.getContentAsString();
|
||||
String id = com.jayway.jsonpath.JsonPath.read(response, "$.id");
|
||||
return userRepository.findById(UUID.fromString(id)).orElseThrow();
|
||||
}
|
||||
|
||||
private Task saveTask(TaskStatus statusValue, User assignee) {
|
||||
return taskRepository.save(new Task(
|
||||
UUID.randomUUID(),
|
||||
"Dammsuga",
|
||||
"Bottenvåningen",
|
||||
statusValue,
|
||||
7,
|
||||
assignee,
|
||||
Instant.parse("2026-07-28T09:00:00Z")));
|
||||
}
|
||||
}
|
||||
@ -0,0 +1,255 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
import java.time.Instant;
|
||||
import java.util.UUID;
|
||||
|
||||
import org.junit.jupiter.api.BeforeEach;
|
||||
import org.junit.jupiter.api.Test;
|
||||
import org.junit.jupiter.params.ParameterizedTest;
|
||||
import org.junit.jupiter.params.provider.CsvSource;
|
||||
import org.junit.jupiter.params.provider.EnumSource;
|
||||
import org.springframework.beans.factory.annotation.Autowired;
|
||||
import org.springframework.boot.test.context.SpringBootTest;
|
||||
import org.springframework.http.MediaType;
|
||||
import org.springframework.test.web.servlet.MockMvc;
|
||||
import org.springframework.test.web.servlet.ResultActions;
|
||||
import org.springframework.test.web.servlet.setup.MockMvcBuilders;
|
||||
import org.springframework.web.context.WebApplicationContext;
|
||||
|
||||
import se.rubble.hemhub.user.User;
|
||||
import se.rubble.hemhub.user.UserRepository;
|
||||
|
||||
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;
|
||||
|
||||
@SpringBootTest
|
||||
class TaskStatusApiTest {
|
||||
|
||||
private static final Instant CREATED_AT = Instant.parse("2026-07-27T10:15:30Z");
|
||||
|
||||
@Autowired
|
||||
private WebApplicationContext context;
|
||||
|
||||
@Autowired
|
||||
private TaskRepository taskRepository;
|
||||
|
||||
@Autowired
|
||||
private UserRepository userRepository;
|
||||
|
||||
private MockMvc mockMvc;
|
||||
|
||||
@BeforeEach
|
||||
void setUp() {
|
||||
taskRepository.deleteAll();
|
||||
mockMvc = MockMvcBuilders.webAppContextSetup(context).build();
|
||||
}
|
||||
|
||||
@ParameterizedTest
|
||||
@CsvSource({
|
||||
"WAITING, IN_PROGRESS",
|
||||
"WAITING, COMPLETED",
|
||||
"IN_PROGRESS, WAITING",
|
||||
"IN_PROGRESS, COMPLETED",
|
||||
"COMPLETED, WAITING",
|
||||
"COMPLETED, IN_PROGRESS"
|
||||
})
|
||||
void allowsEveryDirectStatusTransition(TaskStatus initial, TaskStatus target)
|
||||
throws Exception {
|
||||
User assignee = createUser();
|
||||
Task task = saveTask(initial, assignee);
|
||||
|
||||
updateStatus(task.getId(), target, assignee.getId())
|
||||
.andExpect(status().isOk())
|
||||
.andExpect(jsonPath("$.status").value(target.name()))
|
||||
.andExpect(jsonPath("$.title").value("Dammsuga"))
|
||||
.andExpect(jsonPath("$.description").value("Bottenvåningen"))
|
||||
.andExpect(jsonPath("$.points").value(7))
|
||||
.andExpect(jsonPath("$.createdAt").value(CREATED_AT.toString()))
|
||||
.andExpect(jsonPath("$.assignee.id").value(assignee.getId().toString()));
|
||||
}
|
||||
|
||||
@ParameterizedTest
|
||||
@EnumSource(TaskStatus.class)
|
||||
void acceptsCurrentStatusAsIdempotentTarget(TaskStatus statusValue) throws Exception {
|
||||
User assignee = createUser();
|
||||
Task task = saveTask(statusValue, assignee);
|
||||
|
||||
updateStatus(task.getId(), statusValue, null)
|
||||
.andExpect(status().isOk())
|
||||
.andExpect(jsonPath("$.status").value(statusValue.name()))
|
||||
.andExpect(jsonPath("$.assignee.id").value(assignee.getId().toString()));
|
||||
}
|
||||
|
||||
@Test
|
||||
void automaticallyAssignsActiveUserWhenUnassignedTaskStarts() throws Exception {
|
||||
User activeUser = createUser();
|
||||
Task task = saveTask(TaskStatus.WAITING, null);
|
||||
|
||||
updateStatus(task.getId(), TaskStatus.IN_PROGRESS, activeUser.getId())
|
||||
.andExpect(status().isOk())
|
||||
.andExpect(jsonPath("$.status").value("IN_PROGRESS"))
|
||||
.andExpect(jsonPath("$.assignee.id").value(activeUser.getId().toString()))
|
||||
.andExpect(jsonPath("$.assignee.name").value(activeUser.getName()));
|
||||
}
|
||||
|
||||
@Test
|
||||
void keepsExistingAssigneeAndDoesNotResolveActiveUser() throws Exception {
|
||||
User assignee = createUser();
|
||||
Task task = saveTask(TaskStatus.WAITING, assignee);
|
||||
|
||||
updateStatus(
|
||||
task.getId(),
|
||||
TaskStatus.IN_PROGRESS,
|
||||
UUID.fromString("00000000-0000-0000-0000-000000000099"))
|
||||
.andExpect(status().isOk())
|
||||
.andExpect(jsonPath("$.assignee.id").value(assignee.getId().toString()));
|
||||
}
|
||||
|
||||
@Test
|
||||
void requiresValidActiveUserWhenUnassignedTaskStarts() throws Exception {
|
||||
Task task = saveTask(TaskStatus.COMPLETED, null);
|
||||
|
||||
updateStatus(task.getId(), TaskStatus.IN_PROGRESS, null)
|
||||
.andExpect(status().isConflict())
|
||||
.andExpect(jsonPath("$.code").value("TASK_REQUIRES_ASSIGNEE"));
|
||||
|
||||
updateStatus(
|
||||
task.getId(),
|
||||
TaskStatus.IN_PROGRESS,
|
||||
UUID.fromString("00000000-0000-0000-0000-000000000099"))
|
||||
.andExpect(status().isNotFound())
|
||||
.andExpect(jsonPath("$.code").value("USER_NOT_FOUND"));
|
||||
}
|
||||
|
||||
@ParameterizedTest
|
||||
@EnumSource(TaskStatus.class)
|
||||
void allowsAssigningAndChangingAssigneeInEveryStatus(TaskStatus statusValue)
|
||||
throws Exception {
|
||||
User first = createUser();
|
||||
User second = createUser();
|
||||
Task task = saveTask(statusValue, first);
|
||||
|
||||
updateAssignee(task.getId(), second.getId())
|
||||
.andExpect(status().isOk())
|
||||
.andExpect(jsonPath("$.status").value(statusValue.name()))
|
||||
.andExpect(jsonPath("$.assignee.id").value(second.getId().toString()));
|
||||
}
|
||||
|
||||
@ParameterizedTest
|
||||
@EnumSource(value = TaskStatus.class, names = {"WAITING", "COMPLETED"})
|
||||
void allowsRemovingAssigneeOutsideInProgress(TaskStatus statusValue) throws Exception {
|
||||
Task task = saveTask(statusValue, createUser());
|
||||
|
||||
removeAssignee(task.getId())
|
||||
.andExpect(status().isOk())
|
||||
.andExpect(jsonPath("$.status").value(statusValue.name()))
|
||||
.andExpect(jsonPath("$.assignee").value((Object) null));
|
||||
}
|
||||
|
||||
@Test
|
||||
void rejectsRemovingAssigneeFromInProgressTask() throws Exception {
|
||||
User assignee = createUser();
|
||||
Task task = saveTask(TaskStatus.IN_PROGRESS, assignee);
|
||||
|
||||
removeAssignee(task.getId())
|
||||
.andExpect(status().isConflict())
|
||||
.andExpect(jsonPath("$.code").value("TASK_REQUIRES_ASSIGNEE"))
|
||||
.andExpect(jsonPath("$.message")
|
||||
.value("En pågående uppgift måste ha en ansvarig."));
|
||||
}
|
||||
|
||||
@Test
|
||||
void validatesStatusRequestAndTaskId() throws Exception {
|
||||
Task task = saveTask(TaskStatus.WAITING, null);
|
||||
|
||||
rawStatusUpdate(task.getId().toString(), "{}")
|
||||
.andExpect(status().isBadRequest())
|
||||
.andExpect(jsonPath("$.code").value("INVALID_TASK_STATUS"));
|
||||
rawStatusUpdate(task.getId().toString(), """
|
||||
{"status": null}
|
||||
""")
|
||||
.andExpect(status().isBadRequest())
|
||||
.andExpect(jsonPath("$.code").value("INVALID_TASK_STATUS"));
|
||||
rawStatusUpdate(task.getId().toString(), """
|
||||
{"status": "UNKNOWN"}
|
||||
""")
|
||||
.andExpect(status().isBadRequest())
|
||||
.andExpect(jsonPath("$.code").value("INVALID_TASK_STATUS"));
|
||||
rawStatusUpdate(task.getId().toString(), """
|
||||
{"status": "IN_PROGRESS", "activeUserId": "inte-ett-uuid"}
|
||||
""")
|
||||
.andExpect(status().isBadRequest())
|
||||
.andExpect(jsonPath("$.code").value("INVALID_TASK_ASSIGNMENT"));
|
||||
rawStatusUpdate(task.getId().toString(), "{")
|
||||
.andExpect(status().isBadRequest());
|
||||
rawStatusUpdate("00000000-0000-0000-0000-000000000099", """
|
||||
{"status": "WAITING"}
|
||||
""")
|
||||
.andExpect(status().isNotFound())
|
||||
.andExpect(jsonPath("$.code").value("TASK_NOT_FOUND"));
|
||||
}
|
||||
|
||||
private User createUser() throws Exception {
|
||||
String name = "U" + UUID.randomUUID();
|
||||
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 userRepository.findById(UUID.fromString(id)).orElseThrow();
|
||||
}
|
||||
|
||||
private Task saveTask(TaskStatus statusValue, User assignee) {
|
||||
return taskRepository.save(new Task(
|
||||
UUID.randomUUID(),
|
||||
"Dammsuga",
|
||||
"Bottenvåningen",
|
||||
statusValue,
|
||||
7,
|
||||
assignee,
|
||||
CREATED_AT));
|
||||
}
|
||||
|
||||
private ResultActions updateStatus(
|
||||
UUID taskId,
|
||||
TaskStatus target,
|
||||
UUID activeUserId) throws Exception {
|
||||
String activeUserJson = activeUserId == null
|
||||
? ""
|
||||
: ", \"activeUserId\": \"%s\"".formatted(activeUserId);
|
||||
return rawStatusUpdate(
|
||||
taskId.toString(),
|
||||
"""
|
||||
{"status": "%s"%s}
|
||||
""".formatted(target.name(), activeUserJson));
|
||||
}
|
||||
|
||||
private ResultActions rawStatusUpdate(String taskId, String body) throws Exception {
|
||||
return mockMvc.perform(put("/api/tasks/{taskId}/status", taskId)
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.content(body));
|
||||
}
|
||||
|
||||
private ResultActions updateAssignee(UUID taskId, UUID assigneeId) throws Exception {
|
||||
return mockMvc.perform(put("/api/tasks/{taskId}/assignee", taskId)
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.content("""
|
||||
{"assigneeId": "%s"}
|
||||
""".formatted(assigneeId)));
|
||||
}
|
||||
|
||||
private ResultActions removeAssignee(UUID taskId) throws Exception {
|
||||
return mockMvc.perform(put("/api/tasks/{taskId}/assignee", taskId)
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.content("""
|
||||
{"assigneeId": null}
|
||||
"""));
|
||||
}
|
||||
}
|
||||
42
backend/src/test/java/se/rubble/hemhub/task/TaskTest.java
Normal file
42
backend/src/test/java/se/rubble/hemhub/task/TaskTest.java
Normal file
@ -0,0 +1,42 @@
|
||||
package se.rubble.hemhub.task;
|
||||
|
||||
import java.time.Instant;
|
||||
import java.util.UUID;
|
||||
|
||||
import org.junit.jupiter.api.Test;
|
||||
|
||||
import static org.junit.jupiter.api.Assertions.assertThrows;
|
||||
|
||||
class TaskTest {
|
||||
|
||||
@Test
|
||||
void rejectsPointsOutsideAllowedRange() {
|
||||
assertThrows(InvalidTaskException.class, () -> taskWithPoints(0));
|
||||
assertThrows(InvalidTaskException.class, () -> taskWithPoints(100));
|
||||
}
|
||||
|
||||
@Test
|
||||
void rejectsInProgressTaskWithoutAssignee() {
|
||||
assertThrows(
|
||||
TaskRequiresAssigneeException.class,
|
||||
() -> new Task(
|
||||
UUID.randomUUID(),
|
||||
"Dammsuga",
|
||||
null,
|
||||
TaskStatus.IN_PROGRESS,
|
||||
3,
|
||||
null,
|
||||
Instant.parse("2026-07-26T12:00:00Z")));
|
||||
}
|
||||
|
||||
private Task taskWithPoints(int points) {
|
||||
return new Task(
|
||||
UUID.randomUUID(),
|
||||
"Dammsuga",
|
||||
null,
|
||||
TaskStatus.WAITING,
|
||||
points,
|
||||
null,
|
||||
Instant.parse("2026-07-26T12:00:00Z"));
|
||||
}
|
||||
}
|
||||
108
backend/src/test/java/se/rubble/hemhub/user/UserApiTest.java
Normal file
108
backend/src/test/java/se/rubble/hemhub/user/UserApiTest.java
Normal file
@ -0,0 +1,108 @@
|
||||
package se.rubble.hemhub.user;
|
||||
|
||||
import org.junit.jupiter.api.BeforeEach;
|
||||
import org.junit.jupiter.api.Test;
|
||||
import org.springframework.beans.factory.annotation.Autowired;
|
||||
import org.springframework.boot.test.context.SpringBootTest;
|
||||
import org.springframework.http.MediaType;
|
||||
import org.springframework.test.web.servlet.MockMvc;
|
||||
import org.springframework.test.web.servlet.setup.MockMvcBuilders;
|
||||
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.result.MockMvcResultMatchers.jsonPath;
|
||||
import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
|
||||
|
||||
@SpringBootTest
|
||||
class UserApiTest {
|
||||
|
||||
@Autowired
|
||||
private WebApplicationContext context;
|
||||
|
||||
@Autowired
|
||||
private UserRepository userRepository;
|
||||
|
||||
private MockMvc mockMvc;
|
||||
|
||||
@BeforeEach
|
||||
void setUp() {
|
||||
userRepository.deleteAll();
|
||||
mockMvc = MockMvcBuilders.webAppContextSetup(context).build();
|
||||
}
|
||||
|
||||
@Test
|
||||
void listsNoUsersWhenDatabaseIsEmpty() throws Exception {
|
||||
mockMvc.perform(get("/api/users"))
|
||||
.andExpect(status().isOk())
|
||||
.andExpect(jsonPath("$").isEmpty());
|
||||
}
|
||||
|
||||
@Test
|
||||
void createsAndListsUserWithTrimmedName() throws Exception {
|
||||
mockMvc.perform(post("/api/users")
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.content("""
|
||||
{"name": " Urban "}
|
||||
"""))
|
||||
.andExpect(status().isCreated())
|
||||
.andExpect(jsonPath("$.id").isString())
|
||||
.andExpect(jsonPath("$.name").value("Urban"))
|
||||
.andExpect(jsonPath("$.createdAt").isString())
|
||||
.andExpect(jsonPath("$.normalizedName").doesNotExist());
|
||||
|
||||
mockMvc.perform(get("/api/users"))
|
||||
.andExpect(status().isOk())
|
||||
.andExpect(jsonPath("$[0].name").value("Urban"));
|
||||
}
|
||||
|
||||
@Test
|
||||
void rejectsInvalidNames() throws Exception {
|
||||
mockMvc.perform(post("/api/users")
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.content("""
|
||||
{"name": " "}
|
||||
"""))
|
||||
.andExpect(status().isBadRequest())
|
||||
.andExpect(jsonPath("$.code").value("INVALID_USER_NAME"));
|
||||
|
||||
mockMvc.perform(post("/api/users")
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.content("{\"name\": \"%s\"}".formatted("a".repeat(51))))
|
||||
.andExpect(status().isBadRequest())
|
||||
.andExpect(jsonPath("$.code").value("INVALID_USER_NAME"));
|
||||
}
|
||||
|
||||
@Test
|
||||
void rejectsDuplicateNameIgnoringCase() throws Exception {
|
||||
createUser("Urban");
|
||||
|
||||
mockMvc.perform(post("/api/users")
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.content("""
|
||||
{"name": "urban"}
|
||||
"""))
|
||||
.andExpect(status().isConflict())
|
||||
.andExpect(jsonPath("$.code").value("USER_NAME_ALREADY_EXISTS"));
|
||||
}
|
||||
|
||||
@Test
|
||||
void listsUsersSortedByDisplayName() throws Exception {
|
||||
createUser("Urban");
|
||||
createUser("Anna");
|
||||
createUser("Bertil");
|
||||
|
||||
mockMvc.perform(get("/api/users"))
|
||||
.andExpect(status().isOk())
|
||||
.andExpect(jsonPath("$[0].name").value("Anna"))
|
||||
.andExpect(jsonPath("$[1].name").value("Bertil"))
|
||||
.andExpect(jsonPath("$[2].name").value("Urban"));
|
||||
}
|
||||
|
||||
private void createUser(String name) throws Exception {
|
||||
mockMvc.perform(post("/api/users")
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.content("{\"name\": \"%s\"}".formatted(name)))
|
||||
.andExpect(status().isCreated());
|
||||
}
|
||||
}
|
||||
7
backend/src/test/resources/application.properties
Normal file
7
backend/src/test/resources/application.properties
Normal file
@ -0,0 +1,7 @@
|
||||
spring.datasource.url=jdbc:h2:mem:hemhub-test;MODE=PostgreSQL;DATABASE_TO_LOWER=TRUE;DEFAULT_NULL_ORDERING=HIGH;DB_CLOSE_DELAY=-1
|
||||
spring.datasource.username=sa
|
||||
spring.datasource.password=
|
||||
spring.jpa.hibernate.ddl-auto=validate
|
||||
spring.jpa.open-in-view=false
|
||||
spring.flyway.enabled=true
|
||||
|
||||
225
docs/architecture.md
Normal file
225
docs/architecture.md
Normal file
@ -0,0 +1,225 @@
|
||||
# HemHubs arkitektur
|
||||
|
||||
Detta dokument beskriver den arkitektur som kan verifieras i repositoryts kod,
|
||||
tester och konfiguration. Historiska implementationssteg finns under
|
||||
[`features/`](features/) och övergripande beslut under
|
||||
[`decisions/`](decisions/).
|
||||
|
||||
## Aktuell implementation
|
||||
|
||||
### Monorepo
|
||||
|
||||
HemHub ligger i ett Git-repository med två separata applikationer:
|
||||
|
||||
```text
|
||||
hemhub/
|
||||
├── backend/
|
||||
├── frontend/
|
||||
└── docs/
|
||||
```
|
||||
|
||||
Applikationerna har egna byggverktyg och beroenden. De delar inte källkod eller
|
||||
byggprocess.
|
||||
|
||||
### Frontend
|
||||
|
||||
Frontend finns i `frontend/` och använder React 19, TypeScript, Vite, pnpm och
|
||||
dnd-kit-ekosystemets aktuella React-adapter. 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;
|
||||
- serverbekräftade statusändringar genom knappar på uppgiftskorten;
|
||||
- optimistiska statusflyttar genom drag-and-drop mellan brädans kolumner;
|
||||
- bekräftad och serverbekräftad permanent radering av uppgifter;
|
||||
- klientnära validering och begripliga felmeddelanden;
|
||||
- uppgiftsbrädan med kolumnerna Väntande, Pågående och Klart.
|
||||
|
||||
Tillståndet hanteras lokalt i React-komponenter. Ingen router eller separat
|
||||
global state-lösning används.
|
||||
|
||||
### Backend
|
||||
|
||||
Backend finns i `backend/` och använder Java 21, Spring Boot 4.1.0, Maven,
|
||||
Spring Web, Spring Data JPA och Flyway. Maven Wrapper ingår i repositoryt.
|
||||
|
||||
Backend ansvarar för API, slutlig validering, skapande av UUID och tidsstämplar,
|
||||
persistens samt sortering av returnerade användare och uppgifter.
|
||||
|
||||
### Kommunikation
|
||||
|
||||
Alla applikationsendpoints ligger under `/api`. Frontend använder enbart
|
||||
relativa adresser, exempelvis `/api/users` och `/api/tasks`.
|
||||
|
||||
Vid lokal utveckling kör Vite normalt på port 5173 och proxar `/api` till
|
||||
`http://localhost:8080`, där Spring Boot körs. Ingen generell
|
||||
CORS-konfiguration finns i backend. Webbläsaren anropar därmed Vites origin,
|
||||
och utvecklingsservern vidarebefordrar API-anropen.
|
||||
|
||||
Aktuella endpoints:
|
||||
|
||||
- `GET /api/health`
|
||||
- `GET /api/users`
|
||||
- `POST /api/users`
|
||||
- `GET /api/tasks`
|
||||
- `POST /api/tasks`
|
||||
- `PUT /api/tasks/{taskId}/assignee`
|
||||
- `PUT /api/tasks/{taskId}/status`
|
||||
- `DELETE /api/tasks/{taskId}`
|
||||
|
||||
### Databas och migreringar
|
||||
|
||||
Lokal körning använder en H2-databas i minnet. Databasen finns under
|
||||
backendprocessens livstid och lokal utvecklingsdata återställs när backend
|
||||
startas om. Automatiska backendtester använder en separat H2-databas i minnet.
|
||||
|
||||
Båda anslutningarna använder H2:s `MODE=PostgreSQL`,
|
||||
`DATABASE_TO_LOWER=TRUE` och `DEFAULT_NULL_ORDERING=HIGH`. Det är en verifierbar
|
||||
kompatibilitetsinställning, inte samma sak som att applikationen har verifierats
|
||||
mot PostgreSQL.
|
||||
|
||||
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.
|
||||
|
||||
### Domänmodell
|
||||
|
||||
#### Användare
|
||||
|
||||
En användare lagras i tabellen `app_user` med:
|
||||
|
||||
- `id`: UUID;
|
||||
- `name`: visningsnamn, högst 50 tecken;
|
||||
- `normalized_name`: trimmat namn i gemener, internt och unikt;
|
||||
- `created_at`: en `Instant`, lagrad som `TIMESTAMP WITH TIME ZONE`.
|
||||
|
||||
`normalized_name` exponeras inte via API. Användare returneras alfabetiskt efter
|
||||
visningsnamn med deterministiska sekundära jämförelser.
|
||||
|
||||
#### Uppgift
|
||||
|
||||
En uppgift lagras i tabellen `task` med:
|
||||
|
||||
- `id`: UUID;
|
||||
- `title`: obligatorisk titel, högst 100 tecken;
|
||||
- `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
|
||||
ansvarig användare. Relationen hämtas tillsammans med uppgifterna när de listas,
|
||||
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. Tilldelnings-API:t kan tilldela eller byta
|
||||
ansvarig i samtliga statusar. Ansvarig kan tas bort i `WAITING` och `COMPLETED`,
|
||||
men inte i `IN_PROGRESS`. Tilldelning ändrar aldrig uppgiftens status.
|
||||
|
||||
Alla direkta statusövergångar är tillåtna och samma målstatus är idempotent.
|
||||
`IN_PROGRESS` kräver en ansvarig. När en otilldelad uppgift påbörjas skickar
|
||||
frontend aktiv användares id, och backend tilldelar användaren och ändrar status
|
||||
i samma transaktion. En befintlig ansvarig byts aldrig av statusoperationen.
|
||||
|
||||
Uppgifter raderas fysiskt genom task-repositoryt. Det finns ingen
|
||||
mjukraderingsflagga, papperskorg eller återställningsmodell. Radering av en
|
||||
uppgift påverkar inte dess ansvariga användare.
|
||||
|
||||
### Aktiv användare
|
||||
|
||||
Användarlistan hämtas från backend. Frontend lagrar endast den valda
|
||||
användarens UUID i webbläsarens `localStorage` under nyckeln
|
||||
`hemhub.activeUserId`.
|
||||
|
||||
Vid start verifieras det lagrade id:t mot backendens aktuella användarlista. Ett
|
||||
giltigt val återanvänds i samma browser. Ett ogiltigt val tas bort. Valet är
|
||||
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`,
|
||||
saknade uppgifter eller användare till `404 Not Found` och dubbletter eller
|
||||
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
|
||||
misslyckade API-anrop. Status- och tilldelningsfel visas lokalt på berört kort;
|
||||
statusknappar och tilldelning uppdateras först med backendens bekräftade
|
||||
respons. Drag-and-drop flyttar kortet optimistiskt men återställer hela den
|
||||
tidigare uppgiften vid fel. Vid framgång ersätts alltid det lokala värdet med
|
||||
backendens fullständiga respons. Status- och tilldelningsanrop delar låsning per
|
||||
task-id, så det berörda kortet blockeras utan att resten av brädan låses.
|
||||
Drag-and-drop återanvänder backendens befintliga status-API oförändrat.
|
||||
|
||||
Radering är serverbekräftad och använder samma låsning per task-id. Kortet och
|
||||
bekräftelsedialogen ligger kvar tills backend svarar. Vid `204 No Content`
|
||||
tas kortet bort lokalt. Ett `404`-svar tas endast som bekräftelse på att kortet
|
||||
redan saknas när felkoden är `TASK_NOT_FOUND`; övriga fel behåller kortet och
|
||||
dialogen för ett nytt försök.
|
||||
|
||||
### Teststrategi
|
||||
|
||||
Backend har JUnit 5-tester:
|
||||
|
||||
- ett fristående MockMvc-test för health-endpointen;
|
||||
- Spring Boot-integrationstester via MockMvc mot H2 in-memory för användar- och
|
||||
uppgifts-API.
|
||||
|
||||
Frontend använder Vitest, jsdom och React Testing Library. `fetch` och
|
||||
`localStorage` ersätts i testerna, så frontendtesterna kräver inte en körande
|
||||
backend. Drag-and-drop-adaptern översätter bibliotekshändelser till task-id och
|
||||
status, så stateflöden kan testas utan att simulera fysisk layout i jsdom.
|
||||
Produktionsbygget kör TypeScript-kompilering följt av Vite.
|
||||
|
||||
### Produktionsdeployment
|
||||
|
||||
Ingen produktionsdeployment är implementerad i repositoryt. Det finns inga
|
||||
Dockerfiler, pipelinefiler eller produktionsspecifika Nginx-, Watchtower- eller
|
||||
databaskonfigurationer. H2 används både lokalt och i automatiska tester; någon
|
||||
PostgreSQL-konfiguration finns ännu inte.
|
||||
|
||||
## Beslutad planerad riktning
|
||||
|
||||
Repositoryt anger att affärsregler även framöver ska säkerställas i backend och
|
||||
att större arkitekturella beslut ska diskuteras innan de införs.
|
||||
|
||||
Följande produktionsriktning är beslutad men ännu inte implementerad:
|
||||
|
||||
- PostgreSQL ska användas som produktionsdatabas.
|
||||
- Frontend och backend ska paketeras som separata Docker-images.
|
||||
- Källkoden ligger i Gitea.
|
||||
- Drone ska bygga och publicera images till ett privat registry.
|
||||
- Watchtower ska uppdatera de körande tjänsterna när nya images publiceras.
|
||||
- Nginx kan användas som reverse proxy framför tjänsterna.
|
||||
- Produktionsmiljön ska köras på Ubuntu-servern Biff.
|
||||
|
||||
Den planerade riktningen beskrivs även i
|
||||
[`005-production-deployment-direction.md`](decisions/005-production-deployment-direction.md).
|
||||
Punkterna ovan beskriver målbilden och ska inte tolkas som att motsvarande
|
||||
konfiguration redan finns eller har verifierats.
|
||||
|
||||
## Fortfarande öppna detaljer
|
||||
|
||||
Följande har inte fastställts i dokumentationen och ska beslutas i samband med
|
||||
att produktionslösningen implementeras:
|
||||
|
||||
- exakt containerstruktur och tjänsteindelning;
|
||||
- image-namn och taggningsstrategi;
|
||||
- produktions-URL;
|
||||
- hantering och distribution av secrets;
|
||||
- exakt Nginx-konfiguration;
|
||||
- exakt Drone-, registry-, Watchtower- och deploymentkonfiguration.
|
||||
|
||||
Miljöspecifika adresser, credentials och secrets ska inte lagras i dessa
|
||||
arkitekturdokument.
|
||||
28
docs/decisions/001-monorepo.md
Normal file
28
docs/decisions/001-monorepo.md
Normal file
@ -0,0 +1,28 @@
|
||||
# 001 – Monorepo med separata applikationer
|
||||
|
||||
## Status
|
||||
|
||||
Accepterat
|
||||
|
||||
## Datum
|
||||
|
||||
2026-07-23
|
||||
|
||||
## Sammanhang
|
||||
|
||||
HemHub behöver en webbläsarklient och ett server-API. Båda delarna utvecklas
|
||||
inkrementellt och behöver kunna versionshanteras och dokumenteras tillsammans.
|
||||
|
||||
## Beslut
|
||||
|
||||
Frontend och backend ligger i samma Git-repository, i katalogerna `frontend/`
|
||||
respektive `backend/`. De är separata applikationer med egna byggverktyg,
|
||||
beroenden och startkommandon.
|
||||
|
||||
## Konsekvenser
|
||||
|
||||
- En feature kan ändra frontend, backend, tester och dokumentation atomärt.
|
||||
- En gemensam historik beskriver hela systemet.
|
||||
- Applikationerna kan startas och testas oberoende.
|
||||
- Repositoryt har ingen gemensam rotbyggprocess; relevanta kommandon körs i
|
||||
respektive applikationskatalog.
|
||||
32
docs/decisions/002-same-origin-api-proxy.md
Normal file
32
docs/decisions/002-same-origin-api-proxy.md
Normal file
@ -0,0 +1,32 @@
|
||||
# 002 – Relativa API-adresser och lokal utvecklingsproxy
|
||||
|
||||
## Status
|
||||
|
||||
Accepterat
|
||||
|
||||
## Datum
|
||||
|
||||
2026-07-23
|
||||
|
||||
## Sammanhang
|
||||
|
||||
Frontend körs lokalt med Vite på port 5173 och backend med Spring Boot på port
|
||||
8080. Frontend behöver nå API:t utan miljöspecifika, hårdkodade backendadresser
|
||||
i applikationskoden.
|
||||
|
||||
## Beslut
|
||||
|
||||
Frontend använder relativa API-adresser under `/api`. Vites utvecklingsserver
|
||||
proxar `/api` till `http://localhost:8080`.
|
||||
|
||||
Ingen generell CORS-konfiguration införs i backend så länge webbläsaren anropar
|
||||
Vites origin och Vite vidarebefordrar anropet.
|
||||
|
||||
## Konsekvenser
|
||||
|
||||
- Frontendkoden innehåller inte en lokal fullständig backend-URL.
|
||||
- Lokal utveckling kräver att backend är tillgänglig på port 8080 för
|
||||
API-anrop via proxyn.
|
||||
- En separat CORS-policy behöver inte underhållas för nuvarande lokala flöde.
|
||||
- En framtida driftlösning måste ge `/api` en motsvarande same-origin-väg eller
|
||||
medföra ett nytt dokumenterat beslut.
|
||||
34
docs/decisions/003-central-users-local-active-user.md
Normal file
34
docs/decisions/003-central-users-local-active-user.md
Normal file
@ -0,0 +1,34 @@
|
||||
# 003 – Centrala användare och lokalt val av aktiv användare
|
||||
|
||||
## Status
|
||||
|
||||
Accepterat
|
||||
|
||||
## Datum
|
||||
|
||||
2026-07-24
|
||||
|
||||
## Sammanhang
|
||||
|
||||
HemHub behöver veta vem som använder gränssnittet, men har ännu ingen
|
||||
autentisering. Användarlistan ska vara gemensam medan själva valet kan vara
|
||||
lokalt för den aktuella browsern.
|
||||
|
||||
## Beslut
|
||||
|
||||
Användare lagras centralt via backend och hämtas från `/api/users`. Frontend
|
||||
lagrar endast vald användares UUID i `localStorage` med nyckeln
|
||||
`hemhub.activeUserId`.
|
||||
|
||||
Vid appstart jämförs det lokala id:t med backendens användarlista. Ett giltigt id
|
||||
återanvänds och ett ogiltigt id tas bort. `Logga ut` tar bort nyckeln och visar
|
||||
användarvalet igen.
|
||||
|
||||
## Konsekvenser
|
||||
|
||||
- Samma browser kan återanvända sitt senaste giltiga användarval.
|
||||
- En annan browser eller en rensad browserlagring måste välja användare igen.
|
||||
- Endast id lagras lokalt; aktuellt namn kommer från backendens lista.
|
||||
- Valet synkroniseras inte mellan browsers eller enheter.
|
||||
- Lösningen identifierar en användare i gränssnittet men ger ingen säker
|
||||
autentisering, session eller behörighetskontroll.
|
||||
30
docs/decisions/004-feature-branch-workflow.md
Normal file
30
docs/decisions/004-feature-branch-workflow.md
Normal file
@ -0,0 +1,30 @@
|
||||
# 004 – Kortlivade feature-branches
|
||||
|
||||
## Status
|
||||
|
||||
Accepterat
|
||||
|
||||
## Sammanhang
|
||||
|
||||
HemHub utvecklas inkrementellt med avgränsade ändringar. Historiska
|
||||
feature-branches ska kunna raderas efter merge utan att projektets motiv och
|
||||
aktuella läge försvinner.
|
||||
|
||||
## Beslut
|
||||
|
||||
Varje feature eller avgränsad ändring utvecklas på en kortlivad branch som
|
||||
skapas från uppdaterad `main`. Kod, tester och relevant dokumentation ingår i
|
||||
samma ändring.
|
||||
|
||||
Commit och push görs först efter uttrycklig instruktion. Merge sker efter
|
||||
verifiering, och `main` ska representera verifierad kod. Därefter kan branchen
|
||||
raderas.
|
||||
|
||||
## Konsekvenser
|
||||
|
||||
- Pågående arbete isoleras från `main`.
|
||||
- En feature kan granskas och verifieras som en sammanhållen ändring.
|
||||
- Dokumentationen måste uppdateras före merge så att raderade branches inte
|
||||
behövs för att förstå projektet.
|
||||
- Övergripande beslut bevaras i `docs/decisions/` och faktisk featurehistorik i
|
||||
`docs/features/`.
|
||||
52
docs/decisions/005-production-deployment-direction.md
Normal file
52
docs/decisions/005-production-deployment-direction.md
Normal file
@ -0,0 +1,52 @@
|
||||
# 005 – Riktning för produktionsdeployment
|
||||
|
||||
## Status
|
||||
|
||||
Accepterat som planerad riktning, ännu inte implementerat
|
||||
|
||||
## Sammanhang
|
||||
|
||||
HemHub använder i nuläget H2 för lokal utveckling och tester. Repositoryt saknar
|
||||
fortfarande container-, pipeline- och produktionskonfiguration, men den
|
||||
övergripande målbilden för byggande och drift behöver vara dokumenterad innan
|
||||
den implementeras.
|
||||
|
||||
Källkoden ligger i Gitea och den planerade produktionsmiljön är Ubuntu-servern
|
||||
Biff.
|
||||
|
||||
## Beslut
|
||||
|
||||
- PostgreSQL ska användas som produktionsdatabas.
|
||||
- Frontend och backend ska paketeras som Docker-images.
|
||||
- Drone ska bygga och publicera images till ett privat registry.
|
||||
- Watchtower ska uppdatera tjänsterna när nya images publiceras.
|
||||
- Nginx kan användas som reverse proxy.
|
||||
|
||||
Detta ADR fastställer komponenterna och ansvarsfördelningen på övergripande
|
||||
nivå. Det inför inte någon konfiguration och innebär inte att lösningen redan
|
||||
har driftverifierats.
|
||||
|
||||
## Konsekvenser
|
||||
|
||||
- Kommande produktionsarbete behöver införa och verifiera PostgreSQL-stöd,
|
||||
Dockerpaketering och en Drone-baserad leveranskedja.
|
||||
- Images behöver kunna publiceras till ett privat registry som Biff kan nå.
|
||||
- Uppdateringsflödet behöver utformas så att Watchtower kan hämta och starta nya
|
||||
images på ett kontrollerat sätt.
|
||||
- Nginx är ett möjligt reverse proxy-lager, inte en fastställd detaljkonfiguration.
|
||||
- Lokal utveckling och automatiska tester fortsätter använda H2 tills ett
|
||||
separat beslut eller en feature ändrar detta.
|
||||
|
||||
## Öppna detaljer
|
||||
|
||||
Följande beslutas först när produktionslösningen implementeras:
|
||||
|
||||
- exakt containerstruktur;
|
||||
- image-namn och taggningsstrategi;
|
||||
- produktions-URL;
|
||||
- secrets och hur de tillförs till pipeline och tjänster;
|
||||
- exakt Nginx-konfiguration;
|
||||
- exakt Drone-, registry-, Watchtower- och deploymentkonfiguration.
|
||||
|
||||
IP-adresser, credentials och andra miljöspecifika känsliga värden ska inte
|
||||
dokumenteras här.
|
||||
63
docs/development.md
Normal file
63
docs/development.md
Normal file
@ -0,0 +1,63 @@
|
||||
# Utvecklingsprocess
|
||||
|
||||
Repositoryt är projektets facit. ChatGPT- eller Codex-dialoger kan användas som
|
||||
arbetsyta, men implementation, tester och dokumentation ska tillsammans göra
|
||||
projektets läge begripligt utan tidigare dialoger eller raderade branches.
|
||||
|
||||
## Arbetssätt
|
||||
|
||||
- Använd en kortlivad branch per feature eller annan avgränsad ändring.
|
||||
- Skapa branchen från en uppdaterad `main`.
|
||||
- En feature per ChatGPT-dialog är en praktisk arbetsform, inte en
|
||||
dokumentationskälla.
|
||||
- Välj nästa feature från [`roadmap.md`](roadmap.md).
|
||||
- Uppdatera roadmapen innan en feature delas, flyttas, ersätts eller läggs till.
|
||||
En dialog får inte skapa en parallell featureplan som saknas i repositoryt.
|
||||
- Skapa eller uppdatera feature-dokumentet inom samma feature.
|
||||
- Ge Codex en tydligt avgränsad specifikation.
|
||||
- Implementera endast uttryckliga krav och undvik spekulativ funktionalitet.
|
||||
- Kör relevanta tester före commit och gör manuell verifiering när beteendet
|
||||
motiverar det.
|
||||
- Commit och push sker först efter uttrycklig instruktion.
|
||||
- Merge sker först när ändringen har verifierats.
|
||||
- Uppdatera arkitektur- och beslutsdokument när övergripande beslut förändras.
|
||||
|
||||
`main` ska innehålla verifierad kod. När en feature har mergats ska dess branch
|
||||
kunna raderas utan att projektkunskap går förlorad.
|
||||
|
||||
## Rekommenderad featureprocess
|
||||
|
||||
1. Uppdatera `main`.
|
||||
2. Välj nästa feature från roadmapen och dokumentera först eventuell ändring av
|
||||
planen.
|
||||
3. Skapa en avgränsad branch.
|
||||
4. Skapa eller uppdatera feature-dokumentet.
|
||||
5. Implementera specifikationen.
|
||||
6. Kör relevanta automatiska tester och bygge.
|
||||
7. Gör manuell verifiering där det är relevant.
|
||||
8. Uppdatera dokumentationen så att den beskriver den faktiska lösningen.
|
||||
9. Commit och push efter uttrycklig instruktion.
|
||||
10. Merge efter verifiering.
|
||||
11. Radera den mergade branchen.
|
||||
|
||||
## Verifiering före merge
|
||||
|
||||
För nuvarande projekt bör verifieringen normalt omfatta:
|
||||
|
||||
```bash
|
||||
cd backend
|
||||
./mvnw test
|
||||
```
|
||||
|
||||
```bash
|
||||
cd frontend
|
||||
pnpm test
|
||||
pnpm build
|
||||
```
|
||||
|
||||
Kör även `git diff --check` och granska `git status --short`. Manuell lokal
|
||||
verifiering av berörda flöden kompletterar, men ersätter inte, automatiska
|
||||
tester.
|
||||
|
||||
Om ett befintligt test misslyckas av ett skäl utanför ändringens omfattning ska
|
||||
det rapporteras; produktionskod ska inte ändras enbart för att dölja felet.
|
||||
76
docs/features/000-project-foundation.md
Normal file
76
docs/features/000-project-foundation.md
Normal file
@ -0,0 +1,76 @@
|
||||
# Feature 0 – Projektgrund
|
||||
|
||||
## Status
|
||||
|
||||
Färdig och mergad till `main`.
|
||||
|
||||
## Bakgrund
|
||||
|
||||
HemHub behövde en minimal projektgrund för inkrementell utveckling av en
|
||||
webbapplikation med separat frontend och backend.
|
||||
|
||||
## Mål
|
||||
|
||||
Skapa körbara React- och Spring Boot-applikationer, koppla ihop dem lokalt och
|
||||
etablera grundläggande tester och dokumentation.
|
||||
|
||||
## Omfattning
|
||||
|
||||
- monorepo med `backend/` och `frontend/`;
|
||||
- Java 21, Spring Boot och Maven Wrapper;
|
||||
- React, TypeScript, Vite och pnpm;
|
||||
- health-endpoint och en tillfällig frontendstatus;
|
||||
- Vite-proxy och grundtester;
|
||||
- `README.md`, `AGENTS.md` och `.gitignore`.
|
||||
|
||||
## Avgränsningar
|
||||
|
||||
Feature 0 införde ingen databas, domänmodell, autentisering, deployment,
|
||||
containerkonfiguration eller produktionskonfiguration.
|
||||
|
||||
## Beslut
|
||||
|
||||
Frontend och backend skapades som separata applikationer i samma repository.
|
||||
Frontend använder relativa `/api`-adresser, och Vite proxar dem lokalt till
|
||||
backend på port 8080. Ingen generell CORS-konfiguration infördes.
|
||||
|
||||
## Implementerad lösning
|
||||
|
||||
Backend skapades med Spring Boot 4.1.0, Java 21, Spring Web och Maven Wrapper.
|
||||
Frontend skapades med React 19, TypeScript, Vite och pnpm.
|
||||
|
||||
Den ursprungliga startsidan anropade health-endpointen och visade backendstatus
|
||||
eller ett anslutningsfel. Senare features har ersatt denna startsida, men
|
||||
health-endpointen och dess test finns kvar.
|
||||
|
||||
## API-förändringar
|
||||
|
||||
`GET /api/health` infördes och returnerar:
|
||||
|
||||
```json
|
||||
{"status":"UP"}
|
||||
```
|
||||
|
||||
## Databasförändringar
|
||||
|
||||
Inga.
|
||||
|
||||
## Frontendförändringar
|
||||
|
||||
En minimal startsida visade rubriken HemHub, att frontend hade startat och
|
||||
resultatet från `/api/health`. Vite konfigurerades att proxya `/api` till
|
||||
`http://localhost:8080`.
|
||||
|
||||
## Tester och verifiering
|
||||
|
||||
Ett MockMvc-test verifierar status 200 och `status: UP`. Det ursprungliga
|
||||
frontendtestet verifierade rubriken HemHub med mockat API-anrop.
|
||||
|
||||
## Kända begränsningar
|
||||
|
||||
Projektgrunden innehöll ingen användar- eller uppgiftsfunktionalitet. Den
|
||||
ursprungliga health-vyn är inte längre appens aktiva vy.
|
||||
|
||||
## Relaterade commits
|
||||
|
||||
- `9957383e88b08dc006d2eaeb2a513c7769bd5705` – `Initialize HemHub project foundation`
|
||||
99
docs/features/001-user-selection.md
Normal file
99
docs/features/001-user-selection.md
Normal file
@ -0,0 +1,99 @@
|
||||
# Feature 1 – Användarval
|
||||
|
||||
## Status
|
||||
|
||||
Färdig och mergad till `main`.
|
||||
|
||||
## Bakgrund
|
||||
|
||||
HemHub behövde centralt lagrade användare och ett enkelt sätt att välja vem som
|
||||
använder applikationen, utan att införa autentisering.
|
||||
|
||||
## Mål
|
||||
|
||||
Göra det möjligt att lista och skapa användare, välja en aktiv användare,
|
||||
återanvända valet i samma browser och lämna den aktiva vyn.
|
||||
|
||||
## Omfattning
|
||||
|
||||
- persistens, API och validering för användare;
|
||||
- startflöden för tom och befintlig användarlista;
|
||||
- lokalt lagrad aktiv användare;
|
||||
- användarval, skapande och felhantering;
|
||||
- automatiserade backend- och frontendtester.
|
||||
|
||||
## Avgränsningar
|
||||
|
||||
Ingen autentisering, lösenord, roll, behörighet, e-post, avatar,
|
||||
hushållsrelation, redigering eller radering infördes.
|
||||
|
||||
## Beslut
|
||||
|
||||
Backend är slutlig auktoritet för namnvalidering. Namn normaliseras separat för
|
||||
skiftlägesokänslig unikhet. Frontend lagrar endast UUID under
|
||||
`hemhub.activeUserId` och verifierar det mot den hämtade användarlistan.
|
||||
|
||||
## Implementerad lösning
|
||||
|
||||
Vid appstart hämtar frontend alltid användarna. En tom lista leder direkt till
|
||||
formuläret Skapa användare. Om användare finns men inget giltigt lokalt val
|
||||
finns visas Vem är du?.
|
||||
|
||||
Val eller lyckat skapande sparar användarens id och aktiverar användaren. Ett
|
||||
ogiltigt lagrat id rensas utan tekniskt fel. Feature 1:s tillfälliga startsida
|
||||
och kontrollen Byt användare ersattes i Feature 2 av uppgiftsbrädan och
|
||||
kontrollen Logga ut; lagringsmekanismen är oförändrad.
|
||||
|
||||
## API-förändringar
|
||||
|
||||
- `GET /api/users` returnerar alla användare.
|
||||
- `POST /api/users` skapar en användare och returnerar `201 Created`.
|
||||
|
||||
API-responsen innehåller `id`, `name` och `createdAt`. `normalizedName` exponeras
|
||||
inte.
|
||||
|
||||
Tomt namn eller namn längre än 50 Unicode-kodpunkter ger `400` med
|
||||
`INVALID_USER_NAME`. Ett dubblettnamn utan hänsyn till stora och små bokstäver
|
||||
ger `409` med `USER_NAME_ALREADY_EXISTS`.
|
||||
|
||||
## Databasförändringar
|
||||
|
||||
Flyway-migreringen `V1__create_users.sql` skapade tabellen `app_user`:
|
||||
|
||||
- UUID som primärnyckel;
|
||||
- `name VARCHAR(50)`;
|
||||
- unikt `normalized_name VARCHAR(150)`;
|
||||
- `created_at TIMESTAMP WITH TIME ZONE`.
|
||||
|
||||
Lokalt används filbaserad H2 och i tester H2 in-memory. Backend genererar UUID
|
||||
och `createdAt` med en UTC-klocka.
|
||||
|
||||
## Frontendförändringar
|
||||
|
||||
Frontend fick laddnings-, fel-, användarvals- och användarskapandevyer.
|
||||
Skapandeformuläret trimmar namnet, gör en enkel längdkontroll, blockerar
|
||||
dubbelsubmit och behåller inmatningen vid fel.
|
||||
|
||||
Nuvarande utloggning tar bort `hemhub.activeUserId`, rensar aktiv användare och
|
||||
visar Vem är du? även om endast en användare finns.
|
||||
|
||||
## Tester och verifiering
|
||||
|
||||
Backendens integrationstester verifierar tom lista, skapande och listning,
|
||||
trimning, ogiltiga namn, skiftlägesokänsliga dubbletter och alfabetisk
|
||||
sortering.
|
||||
|
||||
Frontendtesterna verifierar tom lista, användarval, automatisk aktivering efter
|
||||
skapande, bevarad inmatning vid fel, hämtfel med återförsök, ogiltigt lagrat id
|
||||
och utloggning. API-anropen mockas.
|
||||
|
||||
## Kända begränsningar
|
||||
|
||||
Aktiv användare är ett lokalt gränssnittsval, inte säker autentisering. Valet
|
||||
synkroniseras inte mellan browsers eller enheter. Användare kan inte redigeras
|
||||
eller raderas.
|
||||
|
||||
## Relaterade commits
|
||||
|
||||
- `1ec7a729085d456d3185a1c74d02f99f40de0e8d` – `feat: add user selection flow`
|
||||
- `050f248857a01db2dc236a0ca35982fd70dab3d6` – merge till `main`
|
||||
108
docs/features/002-task-creation.md
Normal file
108
docs/features/002-task-creation.md
Normal file
@ -0,0 +1,108 @@
|
||||
# Feature 2 – Skapa uppgifter
|
||||
|
||||
## Status
|
||||
|
||||
Färdig och mergad till `main`.
|
||||
|
||||
## Bakgrund
|
||||
|
||||
Efter införandet av aktiv användare behövde HemHub en första gemensam
|
||||
uppgiftsmodell och en enkel bräda för att skapa och visa uppgifter.
|
||||
|
||||
## Mål
|
||||
|
||||
Låta en aktiv användare se tre statuskolumner, skapa en uppgift med titel och
|
||||
valfri beskrivning samt se den sparade uppgiften efter omladdning.
|
||||
|
||||
## Omfattning
|
||||
|
||||
- persistent uppgiftsmodell och Flyway-migrering;
|
||||
- API för att skapa och lista uppgifter;
|
||||
- bräda med Väntande, Pågående och Klart;
|
||||
- modal för att skapa uppgifter;
|
||||
- laddnings-, validerings- och felhantering;
|
||||
- automatiserade backend- och frontendtester.
|
||||
|
||||
## Avgränsningar
|
||||
|
||||
Ingen ändring av status, drag-and-drop, tilldelning, användarrelation, poäng,
|
||||
deadline, återkommande uppgift, redigering, radering, sökning, filtrering eller
|
||||
paginering infördes.
|
||||
|
||||
## Beslut
|
||||
|
||||
Alla användare ser samma uppgifter; uppgiftsmodellen har ingen relation till en
|
||||
användare. Backend väljer alltid status `WAITING` vid skapande. Listningen
|
||||
sorteras i backend efter `createdAt ASC, id ASC`, och frontend bevarar den
|
||||
ordningen.
|
||||
|
||||
## Implementerad lösning
|
||||
|
||||
JPA-entiteten `Task` innehåller UUID, titel, valfri beskrivning, status och
|
||||
skapandetid. Backend genererar UUID och `createdAt` med en UTC-klocka.
|
||||
|
||||
Frontend visar uppgiftsbrädan när ett giltigt aktivt användarval finns. Uppgifter
|
||||
hämtas vid montering, grupperas efter status och visas med endast titel och
|
||||
eventuell beskrivning. Tomma kolumner saknar tomlägestext.
|
||||
|
||||
## API-förändringar
|
||||
|
||||
- `GET /api/tasks` returnerar samtliga uppgifter, äldst först och med UUID som
|
||||
sekundär sorteringsnyckel.
|
||||
- `POST /api/tasks` skapar en uppgift och returnerar `201 Created`.
|
||||
|
||||
Titel trimmas, är obligatorisk och får omfatta högst 100 Unicode-kodpunkter.
|
||||
Beskrivning trimmas, får omfatta högst 500 Unicode-kodpunkter och lagras som
|
||||
`null` om den är tom. Ogiltiga anrop ger `400` med felkoden `INVALID_TASK`.
|
||||
|
||||
## Databasförändringar
|
||||
|
||||
Flyway-migreringen `V2__create_tasks.sql` skapade tabellen `task`:
|
||||
|
||||
- `id UUID PRIMARY KEY`;
|
||||
- `title VARCHAR(100) NOT NULL`;
|
||||
- `description VARCHAR(500)`;
|
||||
- `status VARCHAR(20) NOT NULL`;
|
||||
- `created_at TIMESTAMP WITH TIME ZONE NOT NULL`.
|
||||
|
||||
Status lagras som text genom `@Enumerated(EnumType.STRING)`. Databasen har ingen
|
||||
check constraint för enumvärden.
|
||||
|
||||
## Frontendförändringar
|
||||
|
||||
Feature 1:s tillfälliga aktiva vy ersattes med uppgiftsbrädan. Sidhuvudet visar
|
||||
aktiv användares namn, Logga ut och Ny uppgift.
|
||||
|
||||
Skapandemodalen innehåller titel och valfri beskrivning. Titelfältet får fokus
|
||||
när modalen öppnas. När inget submit-anrop pågår kan den stängas med kryss,
|
||||
Escape eller klick på bakgrunden. Normal stängning avmonterar komponenten och
|
||||
nollställer därmed formuläret.
|
||||
|
||||
Vid submit gör frontend samma grundläggande längdkontroller, skickar trimmade
|
||||
värden och blockerar uppenbara dubbelsubmit. Vid fel stannar modalen öppen med
|
||||
bevarad inmatning. Vid framgång läggs API-svaret sist i den befintliga listan,
|
||||
vilket placerar den nya `WAITING`-uppgiften längst ned i Väntande utan att
|
||||
sortera om backendens ordning.
|
||||
|
||||
## Tester och verifiering
|
||||
|
||||
Backendens integrationstester verifierar skapande, `WAITING`, trimning, tom
|
||||
beskrivning som `null`, längdvalidering samt sorteringen `createdAt ASC, id ASC`.
|
||||
|
||||
Frontendtesterna verifierar bräda och statusgruppering, tomma kolumner,
|
||||
modalöppning och fokus, skapande, ordning efter skapande, bevarad formulärdata
|
||||
vid API-fel samt utloggning. Parametriserade testfall verifierar också stängning
|
||||
med kryss, Escape och bakgrundsklick samt att formuläret är rensat när modalen
|
||||
öppnas igen.
|
||||
|
||||
## Kända begränsningar
|
||||
|
||||
Statusvärden utöver `WAITING` kan visas om de redan finns i databasen, men inget
|
||||
nuvarande API eller gränssnitt kan flytta en uppgift mellan kolumnerna. Det
|
||||
finns ingen koppling mellan uppgifter och skapande eller aktiv användare.
|
||||
Modalen har ingen fokusfälla eller explicit fokusåterställning.
|
||||
|
||||
## Relaterade commits
|
||||
|
||||
- `3f152eecccdd88f840066543bf9321b81b4cead8` – `feat: add task creation board`
|
||||
- `2f7b99fb21c57c2e9c5f019a2b5073458e41c939` – merge till `main`
|
||||
443
docs/features/003-task-points.md
Normal file
443
docs/features/003-task-points.md
Normal file
@ -0,0 +1,443 @@
|
||||
# Feature 3 – Uppgiftspoäng
|
||||
|
||||
## Status
|
||||
|
||||
Färdig och mergad till `main`.
|
||||
|
||||
## Bakgrund
|
||||
|
||||
HemHub ska på sikt kunna använda spelifiering för att uppmuntra
|
||||
familjemedlemmar att utföra uppgifter. Exempel på framtida funktioner kan vara
|
||||
mål, achievements och belöningar baserade på hur många poäng en användare
|
||||
samlar under en viss period.
|
||||
|
||||
Feature 3 inför den grundläggande poänginformationen på uppgiften. Funktionen
|
||||
registrerar endast uppgiftens poängvärde. Intjäning av poäng och övrig
|
||||
spelifiering införs i senare features.
|
||||
|
||||
## Mål
|
||||
|
||||
Feature 3 ska:
|
||||
|
||||
- lägga till ett obligatoriskt poängvärde på varje uppgift;
|
||||
- låta användaren ange poäng när en uppgift skapas;
|
||||
- visa poängen på uppgiftskortet;
|
||||
- validera poängen konsekvent i frontend och backend;
|
||||
- dokumentera hur lokal utvecklingsdata hanteras.
|
||||
|
||||
## Betydelsen av poäng
|
||||
|
||||
Poängen uttrycker uppgiftens samlade värde utifrån hur:
|
||||
|
||||
- tidskrävande uppgiften är;
|
||||
- besvärlig uppgiften är;
|
||||
- viktig uppgiften är.
|
||||
|
||||
När poängintjäning införs i en senare feature ska samma värde motsvara hur många
|
||||
poäng användaren får när uppgiften slutförs.
|
||||
|
||||
Poängen är inte en exakt tidsuppskattning. En snabb men viktig uppgift kan
|
||||
därför ha ett högre poängvärde än en längre men mindre betydelsefull uppgift.
|
||||
|
||||
Feature 3 registrerar endast poängvärdet. Den ska inte registrera:
|
||||
|
||||
- vem som har tjänat poängen;
|
||||
- om poängen har delats ut;
|
||||
- när poängen har tjänats in;
|
||||
- någon historik över poäng.
|
||||
|
||||
## Poängskala
|
||||
|
||||
Poäng ska vara ett heltal mellan 1 och 99, inklusive gränsvärdena.
|
||||
|
||||
Alla heltal i intervallet är tillåtna. Feature 3 inför inte någon fast skala med
|
||||
fördefinierade steg.
|
||||
|
||||
Giltiga exempel:
|
||||
|
||||
- 1
|
||||
- 7
|
||||
- 25
|
||||
- 99
|
||||
|
||||
Ogiltiga exempel:
|
||||
|
||||
- inget värde;
|
||||
- `null`;
|
||||
- 0;
|
||||
- negativa tal;
|
||||
- 100 eller högre;
|
||||
- decimaltal;
|
||||
- text som inte kan tolkas som ett heltal.
|
||||
|
||||
En fast poängskala kan införas senare om erfarenhet från användningen visar att
|
||||
det är lämpligt.
|
||||
|
||||
## Avgränsning
|
||||
|
||||
Feature 3 omfattar endast:
|
||||
|
||||
- uppgiftens titel;
|
||||
- uppgiftens valfria beskrivning;
|
||||
- uppgiftens obligatoriska poängvärde;
|
||||
- visning av poäng på uppgiftskortet.
|
||||
|
||||
Feature 3 ska inte införa:
|
||||
|
||||
- tilldelning av uppgifter;
|
||||
- ändring av uppgiftsstatus;
|
||||
- drag-and-drop;
|
||||
- redigering av befintliga uppgifter;
|
||||
- radering av uppgifter;
|
||||
- deadlines;
|
||||
- återkommande uppgifter;
|
||||
- poänghistorik;
|
||||
- användares poängsaldo;
|
||||
- topplistor;
|
||||
- statistik;
|
||||
- mål;
|
||||
- achievements;
|
||||
- belöningar;
|
||||
- automatisk utdelning av poäng när en uppgift slutförs.
|
||||
|
||||
Dessa funktioner hanteras i senare features enligt roadmapen.
|
||||
|
||||
## Användarflöde
|
||||
|
||||
När användaren öppnar dialogen för att skapa en uppgift ska formuläret
|
||||
innehålla:
|
||||
|
||||
- titel;
|
||||
- beskrivning;
|
||||
- poäng.
|
||||
|
||||
Poängfältet ska initialt innehålla värdet `1`.
|
||||
|
||||
Användaren kan behålla standardvärdet eller ange ett annat heltal mellan 1 och
|
||||
99.
|
||||
|
||||
När uppgiften skapas ska frontend alltid skicka poängvärdet uttryckligen till
|
||||
backend. Backend ska inte själv fylla i ett saknat värde.
|
||||
|
||||
Efter att en uppgift har skapats framgångsrikt ska formuläret återställas.
|
||||
Poängfältet ska då återgå till `1`.
|
||||
|
||||
Om dialogen stängs och senare öppnas igen ska poängfältet också börja på `1`.
|
||||
|
||||
## Skapandedialog
|
||||
|
||||
Poäng ska anges med ett vanligt numeriskt inmatningsfält.
|
||||
|
||||
Fältet ska ha:
|
||||
|
||||
- etiketten `Poäng`;
|
||||
- initialt värde `1`;
|
||||
- minsta värde `1`;
|
||||
- högsta värde `99`;
|
||||
- heltalssteg.
|
||||
|
||||
En kort hjälptext kan visas:
|
||||
|
||||
> 1–99 poäng beroende på hur tidskrävande, besvärlig eller viktig uppgiften är.
|
||||
|
||||
Fältet får tillfälligt vara tomt medan användaren redigerar värdet. Frontend ska
|
||||
inte automatiskt återställa värdet till `1` medan användaren skriver.
|
||||
|
||||
Validering ska främst ske när användaren försöker skicka formuläret. Avancerad
|
||||
validering vid varje tangenttryckning ingår inte i denna feature.
|
||||
|
||||
## Frontendvalidering
|
||||
|
||||
Frontend ska blockera skapandeanropet om poängen inte är ett heltal mellan 1 och
|
||||
99.
|
||||
|
||||
Vid ett ogiltigt värde ska följande meddelande visas:
|
||||
|
||||
> Poäng måste vara ett heltal mellan 1 och 99.
|
||||
|
||||
Samma meddelande kan användas för:
|
||||
|
||||
- tomt värde;
|
||||
- värde under 1;
|
||||
- värde över 99;
|
||||
- decimaltal;
|
||||
- annat ogiltigt innehåll.
|
||||
|
||||
HTML-fältets attribut för minsta värde, högsta värde och heltalssteg får användas
|
||||
som stöd, men formulärlogiken ska också kontrollera värdet explicit.
|
||||
|
||||
Backend är alltid den slutliga garanten för valideringsreglerna.
|
||||
|
||||
## Visning på uppgiftskortet
|
||||
|
||||
Uppgiftens poäng ska visas på uppgiftskortet som en kompakt och dynamisk badge.
|
||||
|
||||
Badgen ska:
|
||||
|
||||
- renderas som en vanlig React- och HTML-komponent;
|
||||
- använda text och CSS;
|
||||
- läsa värdet från uppgiftens `points`;
|
||||
- visa värdet i formatet `{points} p`.
|
||||
|
||||
Exempel:
|
||||
|
||||
- `1 p`
|
||||
- `7 p`
|
||||
- `99 p`
|
||||
|
||||
Ingen genererad bild eller statisk grafik ska användas för själva poängvärdet.
|
||||
|
||||
Placering och visuell utformning ska följa projektets befintliga skärmbilder och
|
||||
nuvarande kortdesign. Poängindikatorn ska ligga i kortets metadataområde på
|
||||
motsvarande plats som poängindikatorn i designreferensen.
|
||||
|
||||
Mindre justeringar får göras för att passa den faktiska kortimplementationen.
|
||||
Feature 3 ska däremot inte införa en ny övergripande design för uppgiftskortet.
|
||||
|
||||
## API
|
||||
|
||||
Fältnamnet ska vara `points` genomgående i API, backend och frontend.
|
||||
|
||||
### Skapa uppgift
|
||||
|
||||
Requesten för att skapa en uppgift ska innehålla:
|
||||
|
||||
```json
|
||||
{
|
||||
"title": "Töm diskmaskinen",
|
||||
"description": "Ställ in allt i rätt skåp",
|
||||
"points": 3
|
||||
}
|
||||
```
|
||||
|
||||
`points` är obligatoriskt.
|
||||
|
||||
Backend ska inte tolka ett saknat värde som `1`.
|
||||
|
||||
### Uppgiftssvar
|
||||
|
||||
API-svar som innehåller en uppgift ska också innehålla `points`.
|
||||
|
||||
Exempel:
|
||||
|
||||
```json
|
||||
{
|
||||
"id": "00000000-0000-0000-0000-000000000000",
|
||||
"title": "Töm diskmaskinen",
|
||||
"description": "Ställ in allt i rätt skåp",
|
||||
"status": "WAITING",
|
||||
"points": 3,
|
||||
"createdAt": "2026-07-26T12:00:00Z"
|
||||
}
|
||||
```
|
||||
|
||||
Det gäller både:
|
||||
|
||||
- svaret efter att en uppgift skapats;
|
||||
- listning av uppgifter.
|
||||
|
||||
Det exakta API-formatet ska i övrigt följa den befintliga implementationen.
|
||||
|
||||
## Backendregler
|
||||
|
||||
En uppgift får aldrig existera med ett poängvärde utanför intervallet 1–99.
|
||||
|
||||
Regeln ska skyddas genom hela backend, inte bara i HTTP-lagret.
|
||||
|
||||
Beroende på repositoryts befintliga struktur ska valideringen tillämpas på
|
||||
relevanta nivåer, exempelvis:
|
||||
|
||||
- requestvalidering;
|
||||
- applikations- eller domänlogik;
|
||||
- entitetsmodell;
|
||||
- databasens schema.
|
||||
|
||||
Implementation ska följa projektets etablerade kodstruktur och inte introducera
|
||||
ett nytt arkitekturmönster enbart för denna feature.
|
||||
|
||||
## Felhantering
|
||||
|
||||
Ett ogiltigt eller saknat `points` ska ge:
|
||||
|
||||
```text
|
||||
400 Bad Request
|
||||
```
|
||||
|
||||
Backend ska använda projektets befintliga felformat och befintliga
|
||||
felhantering.
|
||||
|
||||
Feature 3 ska inte introducera en separat felmodell endast för poäng.
|
||||
|
||||
Backend får ge mer precisa valideringsdetaljer för exempelvis:
|
||||
|
||||
- saknat värde;
|
||||
- `null`;
|
||||
- värde under 1;
|
||||
- värde över 99.
|
||||
|
||||
Frontend behöver inte återge varje backenddetalj separat, utan kan visa det
|
||||
gemensamma användarmeddelandet:
|
||||
|
||||
> Poäng måste vara ett heltal mellan 1 och 99.
|
||||
|
||||
Vid andra eller oväntade backendfel ska frontend fortsätta använda projektets
|
||||
befintliga generella felhantering.
|
||||
|
||||
## Databas
|
||||
|
||||
Databasschemat ska innehålla ett obligatoriskt heltalsfält för uppgiftens poäng.
|
||||
|
||||
Det logiska slutläget är:
|
||||
|
||||
```text
|
||||
points INTEGER NOT NULL
|
||||
```
|
||||
|
||||
Databasen ska, om den befintliga schemahanteringen stödjer det, även skydda
|
||||
intervallet 1–99 med en motsvarande constraint.
|
||||
|
||||
Databasen ska inte ha ett permanent defaultvärde för nya uppgifter. Nya
|
||||
uppgifter ska alltid få ett uttryckligt poängvärde från applikationen.
|
||||
|
||||
Det förvalda värdet `1` är ett frontendbeteende och inte ett sätt för backend
|
||||
eller databasen att tyst komplettera ofullständiga anrop.
|
||||
|
||||
## Lokal utvecklingsdatabas
|
||||
|
||||
Den lokala utvecklingsdatabasen ska vara en in-memory H2-databas.
|
||||
|
||||
Databasen och dess innehåll ska återställas när backend startas om.
|
||||
|
||||
Lokal utvecklingsdata betraktas därför som tillfällig. Användare och uppgifter
|
||||
som skapats manuellt under utveckling behöver inte bevaras mellan starter.
|
||||
|
||||
Detta innebär att Feature 3 inte behöver migrera verkliga befintliga
|
||||
utvecklingsposter. En ny databas skapas direkt med det obligatoriska
|
||||
poängfältet.
|
||||
|
||||
Före Feature 3 var lokal H2 filbaserad. Feature 3 ändrar utvecklingsanslutningen
|
||||
till in-memory och uppdaterar utvecklingsdokumentationen i samma ändring.
|
||||
|
||||
## Schemahantering och framtida migrering
|
||||
|
||||
Att lokal utvecklingsdata inte bevaras innebär inte att framtida
|
||||
produktionsdata kan återställas vid varje release.
|
||||
|
||||
När HemHub börjar använda en beständig PostgreSQL-databas med data som ska
|
||||
bevaras måste schemaändringar hanteras med kontrollerade migreringar.
|
||||
|
||||
Feature 3 behöver inte införa eller färdigställa hela den framtida
|
||||
produktionsstrategin om den ännu inte finns i repositoryt.
|
||||
|
||||
Projektet använder redan Flyway och versionshanterade migreringar. Feature 3 ska
|
||||
därför lägga till en ny Flyway-migrering för poängfältet och inte ändra tidigare
|
||||
migreringar. Hibernate ska fortsatt validera schemat i stället för att skapa
|
||||
det.
|
||||
|
||||
Bytet till in-memory H2 innebär att befintliga lokala utvecklingsposter inte
|
||||
behöver bevaras eller fyllas på med poäng. Själva schemaändringen ska ändå
|
||||
hanteras som en kontrollerad migrering så att migrationshistoriken förblir
|
||||
sammanhängande inför framtida beständig data.
|
||||
|
||||
Repositoryts faktiska arkitektur och dokumentation har företräde.
|
||||
|
||||
## Backendtester
|
||||
|
||||
Backendtesterna verifierar att:
|
||||
|
||||
- en uppgift kan skapas med ett giltigt `points`;
|
||||
- det skapade API-svaret innehåller samma `points`;
|
||||
- listning av uppgifter innehåller `points`;
|
||||
- gränsvärdet `1` accepteras;
|
||||
- gränsvärdet `99` accepteras;
|
||||
- saknat `points` ger `400 Bad Request`;
|
||||
- `points: null` ger `400 Bad Request`;
|
||||
- `points: 0` ger `400 Bad Request`;
|
||||
- negativa värden ger `400 Bad Request`;
|
||||
- `points: 100` ger `400 Bad Request`.
|
||||
|
||||
Testerna följer den befintliga teststilen och utökar de tidigare
|
||||
uppgifts-API-testerna.
|
||||
|
||||
## Frontendtester
|
||||
|
||||
Frontendtesterna verifierar att:
|
||||
|
||||
- skapandedialogen öppnas med poängvärdet `1`;
|
||||
- ett giltigt poängvärde skickas i create-anropet;
|
||||
- tomt poängfält blockerar submit;
|
||||
- ett värde under 1 blockerar submit;
|
||||
- ett värde över 99 blockerar submit;
|
||||
- ett ogiltigt värde visar felmeddelandet;
|
||||
- formuläret återställs till poängvärdet `1` efter lyckad skapning;
|
||||
- ett uppgiftskort visar uppgiftens dynamiska poängbadge;
|
||||
- badgen visar värdet från uppgiftsdata, exempelvis `7 p`.
|
||||
|
||||
Testerna är inte beroende av en viss pixelplacering eller detaljerad CSS.
|
||||
|
||||
## Manuell verifiering
|
||||
|
||||
Följande verifierades manuellt:
|
||||
|
||||
1. Starta frontend och backend enligt projektets utvecklingsinstruktioner.
|
||||
2. Skapa en uppgift utan att ändra poängfältet.
|
||||
3. Verifiera att uppgiften får `1 p`.
|
||||
4. Skapa en uppgift med ett mellanvärde, exempelvis `7`.
|
||||
5. Verifiera att uppgiften får `7 p`.
|
||||
6. Skapa en uppgift med `99`.
|
||||
7. Verifiera att uppgiften får `99 p`.
|
||||
8. Försök skapa en uppgift med tomt poängfält.
|
||||
9. Verifiera att anropet blockeras och att rätt felmeddelande visas.
|
||||
10. Försök använda värdena `0` och `100`.
|
||||
11. Verifiera att båda avvisas.
|
||||
12. Kontrollera att poängbadgen följer projektets designreferens och fungerar
|
||||
med ett- och tvåsiffriga värden.
|
||||
13. Starta om backend.
|
||||
14. Verifiera att den lokala utvecklingsdatan inte finns kvar.
|
||||
|
||||
## Acceptanskriterier
|
||||
|
||||
Feature 3 är klar när:
|
||||
|
||||
- varje ny uppgift har ett obligatoriskt `points`;
|
||||
- `points` är ett heltal mellan 1 och 99;
|
||||
- frontendens standardvärde är `1`;
|
||||
- frontend alltid skickar `points` uttryckligen;
|
||||
- backend avvisar saknat eller ogiltigt `points`;
|
||||
- backend fyller inte automatiskt i ett saknat värde;
|
||||
- uppgiftens poäng returneras av API:t;
|
||||
- uppgiftens poäng visas dynamiskt på uppgiftskortet;
|
||||
- frontend- och backendtester täcker centrala giltiga och ogiltiga fall;
|
||||
- lokal H2 körs som in-memory och återställs vid omstart;
|
||||
- relevant dokumentation är uppdaterad;
|
||||
- Feature 3 inte inför funktionalitet som hör till senare features.
|
||||
|
||||
## Implementationsprinciper
|
||||
|
||||
När Feature 3 senare implementeras ska Codex först läsa:
|
||||
|
||||
```text
|
||||
AGENTS.md
|
||||
README.md
|
||||
docs/architecture.md
|
||||
docs/development.md
|
||||
docs/roadmap.md
|
||||
docs/decisions/
|
||||
docs/features/
|
||||
```
|
||||
|
||||
Codex ska även läsa relevant backendkod, frontendkod och befintliga tester innan
|
||||
ändringar görs.
|
||||
|
||||
Repositoryts faktiska kod och dokumentation har företräde framför antaganden i
|
||||
denna featurebeskrivning.
|
||||
|
||||
Dokumentation, implementation och tester ska uppdateras tillsammans.
|
||||
|
||||
Codex ska inte committa, pusha, skapa pull request eller merga utan uttrycklig
|
||||
instruktion.
|
||||
|
||||
## Relaterade commits
|
||||
|
||||
- `059d4da9214969ed3e28592178160da6de614b4d` – `feat: add task points`
|
||||
- `2e62261f49bb3142e28882483e41e0250ab11c5f` – merge till `main`
|
||||
123
docs/features/004-task-assignment.md
Normal file
123
docs/features/004-task-assignment.md
Normal file
@ -0,0 +1,123 @@
|
||||
# Feature 4 – Tilldelning av uppgifter
|
||||
|
||||
## Status
|
||||
|
||||
Färdig och mergad till `main`.
|
||||
|
||||
## 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 genomfördes för 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
|
||||
|
||||
- `aaebe888f3bae43b3413e9fa3893fec90afac8b4` – `feat: add task assignment`
|
||||
- `d78611f5f77374228f1f69b66734931a988f8355` – merge till `main`
|
||||
124
docs/features/005-task-status.md
Normal file
124
docs/features/005-task-status.md
Normal file
@ -0,0 +1,124 @@
|
||||
# Feature 5 – Statusändring och statusregler
|
||||
|
||||
## Status
|
||||
|
||||
Färdig och mergad till `main`.
|
||||
|
||||
## Bakgrund
|
||||
|
||||
Efter Feature 4 kan uppgifter vara otilldelade eller ha en ansvarig, men inget
|
||||
API eller gränssnitt kan ändra status. Feature 5 inför ett enkelt knappflöde
|
||||
före drag-and-drop och säkerställer statusreglerna i backend.
|
||||
|
||||
## Mål
|
||||
|
||||
- ändra status genom ett särskilt backend-API;
|
||||
- tillåta direkta övergångar mellan `WAITING`, `IN_PROGRESS` och `COMPLETED`;
|
||||
- kräva ansvarig för `IN_PROGRESS`;
|
||||
- automatiskt tilldela aktiv browseranvändare när en otilldelad uppgift
|
||||
påbörjas;
|
||||
- tillåta statusberoende ändringar av ansvarig;
|
||||
- använda serverbekräftade uppdateringar och vänteläge per kort.
|
||||
|
||||
## Status- och tilldelningsregler
|
||||
|
||||
Alla statusar får ändras direkt till varandra. Ett anrop med aktuell status som
|
||||
mål är giltigt och idempotent. En uppgift behöver inte passera
|
||||
`IN_PROGRESS` för att bli `COMPLETED`.
|
||||
|
||||
`WAITING` och `COMPLETED` får vara tilldelade eller otilldelade.
|
||||
`IN_PROGRESS` måste alltid ha en ansvarig. Det befintliga tilldelnings-API:t
|
||||
kan tilldela eller byta ansvarig i samtliga statusar och ta bort ansvarig i
|
||||
`WAITING` och `COMPLETED`. Ett försök att ta bort ansvarig i `IN_PROGRESS`
|
||||
avvisas med `409 TASK_REQUIRES_ASSIGNEE`. Tilldelning ändrar aldrig status.
|
||||
|
||||
När en otilldelad uppgift ändras till `IN_PROGRESS` skickar frontend aktiv
|
||||
användares id. Backend verifierar användaren, tilldelar den och ändrar status i
|
||||
samma transaktion. Om uppgiften redan har en ansvarig behålls den, och skickat
|
||||
`activeUserId` används inte för att byta ansvarig.
|
||||
|
||||
## API-förändringar
|
||||
|
||||
Status ändras med:
|
||||
|
||||
```http
|
||||
PUT /api/tasks/{taskId}/status
|
||||
```
|
||||
|
||||
```json
|
||||
{
|
||||
"status": "IN_PROGRESS",
|
||||
"activeUserId": "d56b54dd-31b0-4d71-8a10-82464be59a61"
|
||||
}
|
||||
```
|
||||
|
||||
`status` är obligatoriskt. `activeUserId` krävs endast när en otilldelad
|
||||
uppgift ska bli `IN_PROGRESS`. Responsen använder samma fullständiga
|
||||
task-format som övriga task-operationer.
|
||||
|
||||
Kända fel använder befintligt format med `code` och `message`:
|
||||
|
||||
- okänd uppgift: `404 TASK_NOT_FOUND`;
|
||||
- saknad, null eller okänd status: `400 INVALID_TASK_STATUS`;
|
||||
- okänd användare: `404 USER_NOT_FOUND`;
|
||||
- otilldelad `IN_PROGRESS`: `409 TASK_REQUIRES_ASSIGNEE`;
|
||||
- ogiltigt UUID-format: `400` med befintlig requestfelkod.
|
||||
|
||||
`USER_NOT_FOUND` och `INVALID_TASK_ASSIGNMENT` återanvänds från Feature 4 i
|
||||
stället för att införa parallella felkoder för samma användar-id.
|
||||
|
||||
## Databasförändringar
|
||||
|
||||
Inga. Befintliga kolumner för status och ansvarig är tillräckliga.
|
||||
|
||||
## Frontendförändringar
|
||||
|
||||
Varje kort visar två statusknappar:
|
||||
|
||||
- `WAITING`: `Påbörja`, `Markera klar`;
|
||||
- `IN_PROGRESS`: `Till Väntande`, `Markera klar`;
|
||||
- `COMPLETED`: `Till Väntande`, `Påbörja igen`.
|
||||
|
||||
Tilldelningskontrollen är redigerbar i alla statusar. Alternativet `Ingen`
|
||||
visas inte för `IN_PROGRESS`.
|
||||
|
||||
Status och tilldelning delar ett vänteläge per uppgift. Under ett anrop ligger
|
||||
kortet kvar i sin kolumn och båda kontrollerna på kortet är inaktiverade.
|
||||
Övriga kort är fortsatt interaktiva. Vid framgång ersätts uppgiften med
|
||||
serverresponsen; vid fel behålls tidigare data och felet visas lokalt.
|
||||
|
||||
## Tester och verifiering
|
||||
|
||||
Backendtesterna omfattar samtliga direkta övergångar, idempotens, automatisk
|
||||
tilldelning, bevarad ansvarig, statusvalidering, okända id:n, statusberoende
|
||||
tilldelning och oförändrade uppgiftsfält.
|
||||
|
||||
Frontendtesterna omfattar knapparnas statusmappning, requestformat,
|
||||
serverbekräftad flytt, automatisk tilldelning i responsen, lokalt vänteläge,
|
||||
dubbelsubmitsskydd, fel utan optimistisk ändring och statusberoende
|
||||
tilldelningsalternativ.
|
||||
|
||||
Manuell verifiering genomfördes genom ett sammanhängande flöde genom alla tre
|
||||
statusar, automatisk tilldelning, byte och borttagning av ansvarig, omladdning
|
||||
och centrala API-fel. Browserflödet verifierades i desktop- och mobilbredd utan
|
||||
upptäckta problem.
|
||||
|
||||
## Avgränsningar
|
||||
|
||||
Ingen drag-and-drop, radering, generell redigering, sortering, deadline,
|
||||
återkommande uppgift, status- eller poänghistorik, slutföranderegistrering,
|
||||
statistik, användaradministration, autentisering eller behörighetskontroll
|
||||
införs.
|
||||
|
||||
## Kända begränsningar
|
||||
|
||||
Aktiv användare är ett lokalt browserval och inte autentisering. Backend kan
|
||||
verifiera att id:t finns, men inte vem som faktiskt använder browsern.
|
||||
|
||||
Statusknapparna är ett första gränssnitt före Feature 6. Det finns ingen
|
||||
versionskontroll för konkurrerande uppdateringar utöver transaktioner och
|
||||
aktuell serverlogik.
|
||||
|
||||
## Relaterade commits
|
||||
|
||||
- `65a6488c0b268f49b1361591f025bdea8d67f754` – `feat: add task status transitions`
|
||||
551
docs/features/006-task-drag-and-drop.md
Normal file
551
docs/features/006-task-drag-and-drop.md
Normal file
@ -0,0 +1,551 @@
|
||||
# Feature 6 – Drag-and-drop
|
||||
|
||||
## Status
|
||||
|
||||
Färdig och mergad till main.
|
||||
|
||||
## Bakgrund
|
||||
|
||||
Feature 5 införde backendstyrda statusövergångar mellan `WAITING`,
|
||||
`IN_PROGRESS` och `COMPLETED`, ett särskilt status-API samt ett första
|
||||
knappbaserat gränssnitt för statusändring.
|
||||
|
||||
Feature 6 ska komplettera detta med drag-and-drop mellan brädans
|
||||
statuskolumner. Featuren ska återanvända den befintliga statusmodellen,
|
||||
status-API:t, tilldelningsreglerna och frontendens hantering av vänteläge och
|
||||
lokala fel. Den ska inte skapa en parallell statusmekanism.
|
||||
|
||||
## Mål
|
||||
|
||||
Feature 6 ska:
|
||||
|
||||
- låta användaren dra uppgiftskort mellan statuskolumner;
|
||||
- använda status-API:t från Feature 5;
|
||||
- ge omedelbar visuell återkoppling genom optimistisk flytt;
|
||||
- återställa kortet om backend avvisar statusändringen;
|
||||
- hantera automatisk tilldelning när en otilldelad uppgift dras till Pågående;
|
||||
- behålla befintliga statusknappar som ett tillfälligt alternativ;
|
||||
- fungera med mus och på en rimlig nivå med touch;
|
||||
- hålla dragmekanik, statuslogik och kortlayout tillräckligt separerade för
|
||||
framtida ändringar.
|
||||
|
||||
## Omfattning
|
||||
|
||||
Feature 6 är primärt en frontendfeature.
|
||||
|
||||
Backendens befintliga endpoint används:
|
||||
|
||||
```http
|
||||
PUT /api/tasks/{taskId}/status
|
||||
```
|
||||
|
||||
Requesten innehåller målstatus och kan innehålla aktiv användares id:
|
||||
|
||||
```json
|
||||
{
|
||||
"status": "IN_PROGRESS",
|
||||
"activeUserId": "d56b54dd-31b0-4d71-8a10-82464be59a61"
|
||||
}
|
||||
```
|
||||
|
||||
Backend ska endast ändras om granskning av den faktiska implementationen visar
|
||||
att en mindre korrigering behövs för att det befintliga kontraktet ska kunna
|
||||
återanvändas korrekt.
|
||||
|
||||
## Implementerad lösning
|
||||
|
||||
Frontend använder `@dnd-kit/react` 0.5.0 som aktuell React-adapter och
|
||||
`@dnd-kit/dom` 0.5.0 för en konfigurerad pointer-sensor. Legacy-paketen
|
||||
`@dnd-kit/core` och `@dnd-kit/sortable` används inte. Sortable-stöd behövs inte
|
||||
eftersom kort inte ordnas inom kolumner.
|
||||
|
||||
`TaskDragAndDrop.tsx` avgränsar bibliotekskopplingen. Adaptern:
|
||||
|
||||
- registrerar uppgiftskort som draggable och statuskolumner som droppable;
|
||||
- översätter ett lyckat drop-event till task-id och målstatus;
|
||||
- ignorerar avbrutna dragningar och ogiltiga mål;
|
||||
- använder sex pixlars aktiveringsavstånd för mus och penna;
|
||||
- använder 250 millisekunders fördröjning med åtta pixlars tolerans för touch;
|
||||
- behåller dnd-kits standardplugins och tangentbordssensor;
|
||||
- förlitar sig på sensorns standardskydd mot dragstart från interaktiva
|
||||
element.
|
||||
|
||||
Ingen särskild `touch-action`, collision detector eller drag-overlay har lagts
|
||||
till. Den befintliga responsiva layouten är oförändrad.
|
||||
|
||||
`TaskBoard` har fortsatt en gemensam requestväg och låsning per task-id för
|
||||
statusändringar. Ett presentationsval avgör beteendet:
|
||||
|
||||
- statusknappar använder `server-confirmed` och flyttar inte kortet före svar;
|
||||
- drag-and-drop använder `optimistic`, sparar hela tidigare uppgiften och
|
||||
flyttar kortet direkt;
|
||||
- lyckade anrop ersätter uppgiften med hela serverresponsen;
|
||||
- misslyckade optimistiska anrop återställer hela rollback-värdet och visar
|
||||
felet lokalt.
|
||||
|
||||
När en otilldelad uppgift dras till `IN_PROGRESS` visar det optimistiska värdet
|
||||
den aktiva användarens id och namn. En befintlig ansvarig behålls. Backendens
|
||||
befintliga status-API och Feature 5-regler återanvänds utan backendändringar.
|
||||
|
||||
Frontendens 44 tester, TypeScript-kompilering, Vites produktionsbygge och
|
||||
`git diff --check` passerar på feature-branchen. Manuell browserverifiering av
|
||||
de centrala drag-, server- och rollbackflödena är genomförd.
|
||||
|
||||
## Avgränsningar
|
||||
|
||||
Feature 6 ska inte införa:
|
||||
|
||||
- en ny statusmodell;
|
||||
- ett nytt eller parallellt status-API;
|
||||
- manuell sortering inom en kolumn;
|
||||
- persistent kortordning;
|
||||
- generell redigering av uppgifter;
|
||||
- radering;
|
||||
- deadlines;
|
||||
- återkommande uppgifter;
|
||||
- status- eller poänghistorik;
|
||||
- undo-funktion;
|
||||
- flera ansvariga;
|
||||
- autentisering eller behörigheter;
|
||||
- realtidsuppdatering mellan browsers;
|
||||
- en ny global state-lösning;
|
||||
- horisontell scrollning som ett särskilt mobilkoncept;
|
||||
- ett specialbyggt tangentbordsflöde för drag-and-drop.
|
||||
|
||||
Att flytta ett kort inom samma kolumn ska inte ändra någon ordning och ska inte
|
||||
ge ett backend-anrop.
|
||||
|
||||
## Befintliga statusregler
|
||||
|
||||
Feature 6 ska behandla följande regler som redan beslutade:
|
||||
|
||||
- statusarna är `WAITING`, `IN_PROGRESS` och `COMPLETED`;
|
||||
- alla direkta statusövergångar är tillåtna;
|
||||
- samma målstatus är giltig och idempotent;
|
||||
- `IN_PROGRESS` kräver ansvarig;
|
||||
- `WAITING` och `COMPLETED` får vara otilldelade;
|
||||
- när en otilldelad uppgift sätts till `IN_PROGRESS` skickar frontend aktiv
|
||||
användares id;
|
||||
- backend tilldelar då användaren och ändrar status atomärt;
|
||||
- om uppgiften redan har en ansvarig behålls denna;
|
||||
- statusoperationen byter aldrig en befintlig ansvarig;
|
||||
- backend returnerar hela den uppdaterade uppgiften;
|
||||
- serverns fullständiga task-respons är slutlig sanning.
|
||||
|
||||
## Uppdateringsstrategi
|
||||
|
||||
Drag-and-drop använder en kontrollerad optimistisk uppdatering.
|
||||
|
||||
När ett kort släpps i en annan statuskolumn ska frontend:
|
||||
|
||||
1. spara hela den nuvarande task-versionen som rollback-värde;
|
||||
2. skapa ett optimistiskt lokalt task-läge;
|
||||
3. visa kortet omedelbart i målkolumnen;
|
||||
4. markera kortet som upptaget;
|
||||
5. skicka statusanropet;
|
||||
6. vid framgång ersätta den optimistiska uppgiften med serverns fullständiga
|
||||
respons;
|
||||
7. vid fel återställa hela den tidigare task-versionen;
|
||||
8. visa felet lokalt på det återställda kortet.
|
||||
|
||||
Rollback ska återställa hela uppgiften, inte enbart statusfältet. Detta är
|
||||
viktigt eftersom en optimistisk flytt till `IN_PROGRESS` även kan innehålla en
|
||||
tillfällig antagen ansvarig.
|
||||
|
||||
Ingen generell infrastruktur för optimistiska uppdateringar ska införas.
|
||||
Lösningen ska hållas lokal till uppgiftslistan och drag-and-drop-flödet.
|
||||
Feature 5:s statusknappar ska fortsatt vara serverbekräftade: kortet ligger kvar
|
||||
i sin aktuella kolumn tills backend svarar.
|
||||
|
||||
## Otilldelad uppgift till Pågående
|
||||
|
||||
När en otilldelad uppgift dras till `IN_PROGRESS` ska frontend:
|
||||
|
||||
- skicka målstatus `IN_PROGRESS`;
|
||||
- skicka aktiv användares id;
|
||||
- optimistiskt visa aktiv användare som ansvarig;
|
||||
- låta backend tilldela användaren och ändra status i samma transaktion.
|
||||
|
||||
Om uppgiften redan har en ansvarig ska frontend inte optimistiskt ersätta denna
|
||||
med den aktiva användaren. Serverresponsen ersätter alltid det optimistiska
|
||||
antagandet.
|
||||
|
||||
Drag-and-drop ska inte visa ett nytt ansvarigval.
|
||||
|
||||
## Drag-and-drop-bibliotek
|
||||
|
||||
Feature 6 ska använda dnd-kit-ekosystemets aktuella stabila React-lösning.
|
||||
Featuredokumentet låser inte exakta paket eller API:n. Codex ska vid
|
||||
implementation verifiera den aktuella officiella dokumentationen, vilka paket
|
||||
som är aktuella respektive legacy, kompatibilitet med repositoryts
|
||||
React-version samt påverkan på Vitest/jsdom och React Testing Library.
|
||||
|
||||
Den valda lösningen ska stödja:
|
||||
|
||||
- draggable-kort;
|
||||
- droppable-statuskolumner;
|
||||
- pointer- och touchinteraktion;
|
||||
- aktiveringsvillkor;
|
||||
- avbruten dragning;
|
||||
- identifiering av målkolumn.
|
||||
|
||||
Biblioteket ska inte användas för:
|
||||
|
||||
- sortering inom kolumner;
|
||||
- persistent ordning;
|
||||
- egen domänmodell;
|
||||
- generell state-hantering;
|
||||
- parallell statuslogik.
|
||||
|
||||
Dragbibliotekets händelser ska översättas till en gemensam,
|
||||
bibliotekoberoende ingång till statusflödet.
|
||||
|
||||
## Statusknapparnas roll
|
||||
|
||||
Feature 5:s statusknappar behålls tills vidare som ett fullt fungerande
|
||||
alternativ. De betraktas inte som en permanent del av målbilden. Den
|
||||
ursprungliga visuella designen innehåller inte statusknappar, och de ska därför
|
||||
enkelt kunna tas bort efter utvärdering.
|
||||
|
||||
Implementation ska följa dessa principer:
|
||||
|
||||
- drag-and-drop får inte byggas ovanpå knappkomponenterna;
|
||||
- draglogik får inte placeras i knapparna;
|
||||
- statuslogik får inte dupliceras mellan knappar och drag-and-drop;
|
||||
- båda ska dela underliggande requestlogik, låsning per task-id, felhantering
|
||||
och ersättning med serverrespons;
|
||||
- drag-and-drop använder optimistisk visuell flytt, medan statusknapparna
|
||||
förblir serverbekräftade och låter kortet ligga kvar tills backend svarar;
|
||||
- den visuella presentationsstrategin får därför skilja sig mellan
|
||||
interaktionerna;
|
||||
- vänteläge, felhantering och serverrespons ska höra till uppgiften och det
|
||||
delade statusflödet, inte dupliceras per kontroll;
|
||||
- kortlayouten får inte strukturellt förutsätta att knapparna alltid finns.
|
||||
|
||||
Att ta bort statusknapparna senare ska inte kräva ändringar i status-API,
|
||||
dragmekanik, rollback eller task-state. Statusarkitekturen får inte vara kopplad
|
||||
till att knapparna finns kvar.
|
||||
|
||||
## Vänteläge
|
||||
|
||||
När statusanropet pågår ska det optimistiskt flyttade kortet tonas ned lätt.
|
||||
Ingen text som `Flyttar…` och ingen spinner krävs i första versionen.
|
||||
|
||||
Under vänteläget ska just detta kort inte kunna:
|
||||
|
||||
- dras igen;
|
||||
- initiera en ny statusändring;
|
||||
- ändra ansvarig.
|
||||
|
||||
Övriga kort ska förbli interaktiva.
|
||||
|
||||
Flera olika kort får ha statusanrop pågående samtidigt. Låsning, nedtoning,
|
||||
rollback och fel hanteras per task-id.
|
||||
|
||||
## Fel och återställning
|
||||
|
||||
Vid ett misslyckat statusanrop ska:
|
||||
|
||||
- hela den tidigare task-versionen återställas;
|
||||
- kortet återgå till ursprungskolumnen;
|
||||
- tidigare ansvarig återställas;
|
||||
- nedtoningen tas bort;
|
||||
- felet visas lokalt på kortet;
|
||||
- målkolumnen inte behålla någon tillfällig markering.
|
||||
|
||||
Tidigare klientdata ska inte delvis blandas med det misslyckade optimistiska
|
||||
tillståndet. Ett fel på ett kort ska inte blockera resten av brädan.
|
||||
|
||||
## Serverrespons
|
||||
|
||||
Vid lyckat statusanrop ska frontend alltid ersätta den optimistiska uppgiften
|
||||
med hela serverresponsen.
|
||||
|
||||
Det gäller även om serverresponsen skiljer sig från frontendens antagande vad
|
||||
gäller exempelvis:
|
||||
|
||||
- status;
|
||||
- ansvarig;
|
||||
- andra returnerade task-fält.
|
||||
|
||||
Servern är slutlig sanning.
|
||||
|
||||
## Dragyta
|
||||
|
||||
Kortets icke-interaktiva yta ska fungera som dragyta. Knappar, select och andra
|
||||
interaktiva element ska inte initiera dragning.
|
||||
|
||||
Implementation får avgöra sensoruppsättning, aktiveringströskel samt avstånd,
|
||||
fördröjning och tolerans. Vanliga klick och små fingerrörelser får inte
|
||||
oavsiktligt starta dragning, och kortets interaktiva kontroller ska fungera
|
||||
normalt.
|
||||
|
||||
Dragmekaniken ska implementeras så att ett separat draghandtag senare kan
|
||||
införas utan att statusoperation, API-anrop, rollback eller task-state behöver
|
||||
ändras. Det bör räcka att flytta bibliotekets draglisteners och tillhörande
|
||||
attribut från kortets rot till handtaget.
|
||||
|
||||
## Målkolumner
|
||||
|
||||
Kolumnerna behöver ingen stark eller permanent drop-markering. Den kolumn som
|
||||
kortet befinner sig över kan vid behov få en mycket diskret hover-effekt,
|
||||
exempelvis:
|
||||
|
||||
- en svag bakgrundsförändring;
|
||||
- en tunn kant;
|
||||
- annan lågmäld visuell återkoppling.
|
||||
|
||||
Feature 6 ska inte införa stora färgade drop-zoner eller en generell redesign
|
||||
av brädan. Om manuell verifiering visar att ingen markering behövs kan även den
|
||||
diskreta hover-effekten utelämnas.
|
||||
|
||||
## Drop i samma kolumn
|
||||
|
||||
Om ett kort släpps i kolumnen som motsvarar dess nuvarande status ska
|
||||
operationen vara no-op.
|
||||
|
||||
Det innebär:
|
||||
|
||||
- inget API-anrop;
|
||||
- ingen statusändring;
|
||||
- ingen ändring av ordning;
|
||||
- inget vänteläge;
|
||||
- inget felmeddelande.
|
||||
|
||||
## Avbruten dragning
|
||||
|
||||
Om en dragning avbryts eller avslutas utanför en giltig målkolumn ska
|
||||
operationen vara no-op. Kortet ska återgå till sin normala position utan
|
||||
API-anrop eller felindikering.
|
||||
|
||||
## Desktop och touch
|
||||
|
||||
Desktop är det primära användningsfallet för Feature 6.
|
||||
|
||||
Touch ska fungera på en rimlig grundnivå, men featuren ska inte införa en
|
||||
särskild mobil Kanban-design. Brädans befintliga responsiva layout ska
|
||||
behållas. Feature 6 ska inte införa horisontell scrollning. På smala skärmar
|
||||
får kolumnerna fortsätta använda repositoryts nuvarande responsiva layout,
|
||||
även om de staplas vertikalt.
|
||||
|
||||
Implementation får avgöra sensoruppsättning, aktiveringsvillkor, eventuell
|
||||
`touch-action`, collision detection och drag-overlay. Valen måste bevara normal
|
||||
vertikal scrollning på mobil och får inte göra interaktiva kortkontroller
|
||||
svåranvända.
|
||||
|
||||
Draglogiken ska:
|
||||
|
||||
- identifiera mål genom status, inte genom en fast skärmposition;
|
||||
- inte förutsätta att kolumnerna ligger horisontellt;
|
||||
- hållas separerad från layout-CSS;
|
||||
- kunna fortsätta fungera om kolumnlayouten senare ändras.
|
||||
|
||||
Om manuell verifiering visar att dragning mellan staplade kolumner fungerar
|
||||
dåligt kan mobilbeteendet ändras senare utan att status- eller rollbacklogiken
|
||||
görs om. Statusknapparna finns kvar som alternativ, särskilt där dragning är
|
||||
opraktisk.
|
||||
|
||||
## Tangentbord och tillgänglighet
|
||||
|
||||
Tangentbordsstyrd drag-and-drop ingår inte som grundkrav i Feature 6.
|
||||
Statusknapparna ska fortsatt ge en fungerande tangentbordsväg för
|
||||
statusändring. Drag-and-drop får därför inte vara den enda möjliga vägen.
|
||||
|
||||
Feature 6 ska ändå uppfylla grundläggande tillgänglighetskrav:
|
||||
|
||||
- interaktiva kontroller ska fortsatt gå att nå med tangentbord;
|
||||
- ett upptaget kort ska inte kunna aktiveras igen;
|
||||
- fokus ska inte tappas oförklarligt efter lyckad operation eller rollback;
|
||||
- begripliga etiketter ska bevaras;
|
||||
- dragbibliotekets standard-ARIA får användas;
|
||||
- ingen omfattande speciallösning för tangentbordsdragning ska byggas.
|
||||
|
||||
Förbättrat tangentbordsstöd för själva dragningen kan införas i en senare
|
||||
uppdatering.
|
||||
|
||||
## Gemensamt statusflöde
|
||||
|
||||
Frontend ska ha ett gemensamt, kontrolloberoende statusflöde för den
|
||||
underliggande statusändringen.
|
||||
|
||||
Det delade flödet ska ansvara för:
|
||||
|
||||
- kontroll av pågående operation för task-id;
|
||||
- requestformat;
|
||||
- aktiv användares id vid behov;
|
||||
- per-kort-vänteläge;
|
||||
- ersättning med serverrespons;
|
||||
- lokalt statusfel.
|
||||
|
||||
Drag-and-drop-flödet ska därutöver beräkna den optimistiska task-versionen,
|
||||
spara rollback-värdet och återställa hela den tidigare uppgiften vid fel.
|
||||
Statusknapparna ska inte göra en optimistisk flytt.
|
||||
|
||||
Drag-and-drop och statusknappar är separata sätt att ange målstatus till det
|
||||
delade flödet, men får använda olika visuell presentationsstrategi.
|
||||
Tilldelningskontrollen ska återanvända samma per-kort-låsning så att status och
|
||||
tilldelning inte kan ändras samtidigt på samma uppgift.
|
||||
|
||||
## Frontendtester
|
||||
|
||||
Frontendtesterna verifierar beteende och state utan att förutsätta fysisk
|
||||
layout eller exakta pointer-koordinater i jsdom. Dragadaptern mockas i
|
||||
brädtesterna, medan den bibliotekoberoende mappningen från draghändelse till
|
||||
task-id och målstatus testas separat.
|
||||
|
||||
Testerna täcker bland annat:
|
||||
|
||||
- korrekt statusanrop och optimistisk flytt till en annan kolumn;
|
||||
- låsning och nedtoning per task-id medan anropet pågår;
|
||||
- att andra kort kan ha samtidiga operationer;
|
||||
- automatisk optimistisk tilldelning till aktiv användare;
|
||||
- att en befintlig ansvarig behålls;
|
||||
- att hela serverresponsen ersätter det optimistiska värdet;
|
||||
- fullständig rollback och lokalt fel vid misslyckande;
|
||||
- no-op för samma status, avbruten dragning och ogiltigt mål;
|
||||
- att statusknapparnas befintliga serverbekräftade beteende är bevarat.
|
||||
|
||||
Totalt passerar 44 frontendtester. TypeScript-kompileringen och Vites
|
||||
produktionsbygge passerar också.
|
||||
|
||||
## Backendtester
|
||||
|
||||
Backend ändrades inte. Feature 5:s befintliga tester fortsätter därför att
|
||||
utgöra verifiering av:
|
||||
|
||||
- direkta statusövergångar;
|
||||
- idempotens;
|
||||
- automatisk tilldelning;
|
||||
- bevarad befintlig ansvarig;
|
||||
- statusvalidering;
|
||||
- statusberoende tilldelningsregler;
|
||||
- oförändrade övriga task-fält.
|
||||
|
||||
## Manuell verifiering
|
||||
|
||||
Manuell browserverifiering är genomförd. Följande verifierades:
|
||||
|
||||
- drag-and-drop mellan statuskolumner fungerar;
|
||||
- kortet flyttas optimistiskt och tonas ned under statusanropet;
|
||||
- en otilldelad uppgift som flyttas till Pågående får aktiv användare som
|
||||
ansvarig;
|
||||
- en befintlig ansvarig behålls;
|
||||
- serverns svar ersätter det optimistiska värdet;
|
||||
- statusknapparna fungerar fortsatt;
|
||||
- ett blockerat statusanrop visar
|
||||
`Det gick inte att ändra status. Försök igen.`;
|
||||
- kortet återställs till ursprungskolumnen vid fel;
|
||||
- tidigare ansvarig återställs, nedtoningen försvinner och kortet blir
|
||||
interaktivt igen;
|
||||
- övriga kort förblir interaktiva under operationen.
|
||||
|
||||
## Dokumentation
|
||||
|
||||
Feature 6 ska dokumenteras i:
|
||||
|
||||
```text
|
||||
docs/features/006-task-drag-and-drop.md
|
||||
```
|
||||
|
||||
Vid implementation ska även följande uppdateras när det är relevant:
|
||||
|
||||
```text
|
||||
README.md
|
||||
docs/architecture.md
|
||||
docs/roadmap.md
|
||||
```
|
||||
|
||||
Roadmapen markerar Feature 6 som `Klar` efter verifiering och merge.
|
||||
|
||||
Ett nytt ADR behövs endast om biblioteksvalet bedöms vara ett övergripande,
|
||||
långlivat frontendbeslut som påverkar fler delar av applikationen än Feature
|
||||
6. Om `dnd-kit` endast används lokalt för denna feature bör beslutet normalt
|
||||
dokumenteras i feature- och arkitekturdokumentationen.
|
||||
|
||||
## Acceptanskriterier
|
||||
|
||||
Feature 6 är klar när:
|
||||
|
||||
- kort kan dras mellan olika statuskolumner;
|
||||
- dragningen använder Feature 5:s status-API;
|
||||
- kortet flyttas optimistiskt till målkolumnen;
|
||||
- kortet tonas ned under serveranropet;
|
||||
- samma kort är låst för status, dragning och tilldelning under anropet;
|
||||
- andra kort förblir interaktiva;
|
||||
- otilldelad uppgift till Pågående använder aktiv användares id;
|
||||
- befintlig ansvarig behålls;
|
||||
- serverresponsen ersätter det optimistiska tillståndet;
|
||||
- fel återställer hela tidigare task-versionen;
|
||||
- kortet återgår till ursprungskolumnen vid fel;
|
||||
- felet visas lokalt på kortet;
|
||||
- drop i samma kolumn är no-op;
|
||||
- avbruten dragning är no-op;
|
||||
- ingen manuell eller persistent kortordning har införts;
|
||||
- statusknapparna fungerar fortsatt men är arkitekturellt frikopplade;
|
||||
- statusknapparna förblir serverbekräftade medan drag-and-drop är optimistisk;
|
||||
- statusknapparna kan tas bort senare utan att drag- eller statuslogiken byggs
|
||||
om;
|
||||
- kortets icke-interaktiva yta fungerar som dragyta;
|
||||
- interaktiva kortkontroller initierar inte dragning;
|
||||
- ett senare draghandtag kan införas utan ändring av statusflödet;
|
||||
- normal vertikal scrollning på mobil bevaras;
|
||||
- ingen horisontell scrollning har införts;
|
||||
- desktop fungerar väl;
|
||||
- touch fungerar på rimlig grundnivå;
|
||||
- tangentbordsdragning inte krävs;
|
||||
- statusändring fortsatt är möjlig med tangentbord genom statusknapparna;
|
||||
- frontendtesterna täcker centrala stateövergångar och felfall;
|
||||
- manuell verifiering täcker dragkänsla, touch, layout och rollback;
|
||||
- backend är oförändrad om inget konkret behov av justering hittas;
|
||||
- relevant dokumentation är uppdaterad.
|
||||
|
||||
## Implementationsprinciper
|
||||
|
||||
Vid implementationen användes följande dokumentation som tekniskt underlag:
|
||||
|
||||
```text
|
||||
AGENTS.md
|
||||
README.md
|
||||
docs/architecture.md
|
||||
docs/development.md
|
||||
docs/roadmap.md
|
||||
docs/decisions/
|
||||
docs/features/004-task-assignment.md
|
||||
docs/features/005-task-status.md
|
||||
```
|
||||
|
||||
Relevant frontendkod och tester granskades särskilt avseende:
|
||||
|
||||
- aktuell React-version;
|
||||
- frontendens installerade beroenden;
|
||||
- aktuell officiell dnd-kit-dokumentation;
|
||||
- vilka dnd-kit-paket som är aktuella respektive legacy;
|
||||
- dnd-kit-lösningens kompatibilitet med React-versionen;
|
||||
- påverkan på Vitest/jsdom och React Testing Library;
|
||||
- aktuell task-typ;
|
||||
- brädans kolumnstruktur;
|
||||
- uppgiftskortets komponentstruktur;
|
||||
- befintliga statusknappar;
|
||||
- statusanropets requestformat;
|
||||
- hur aktiv användare representeras;
|
||||
- hur tasks ersätts i state;
|
||||
- det gemensamma vänteläget per kort;
|
||||
- status- och tilldelningsfel;
|
||||
- tilldelningskontrollens inaktiveringslogik;
|
||||
- aktuell teststil;
|
||||
- vilka pointer- och draghändelser testmiljön stödjer.
|
||||
|
||||
Repositoryts faktiska kod och dokumentation har företräde framför antaganden i
|
||||
denna featurebeskrivning.
|
||||
|
||||
Kod, tester och relevant dokumentation ska uppdateras tillsammans.
|
||||
|
||||
Codex ska inte committa, pusha, skapa pull request eller merga utan uttrycklig
|
||||
instruktion.
|
||||
|
||||
## Relaterade commits
|
||||
|
||||
- Feature-commit:
|
||||
`c3c64482c062f144fd6cb6036e3c9db0afa5ec1e`
|
||||
- Merge-commit till `main`:
|
||||
`2696195e741c155a197ae9838d1938bdf2148cc2`
|
||||
442
docs/features/007-task-deletion.md
Normal file
442
docs/features/007-task-deletion.md
Normal file
@ -0,0 +1,442 @@
|
||||
# Feature 7 – Radera uppgift
|
||||
|
||||
## Status
|
||||
|
||||
Färdig och mergad till main.
|
||||
|
||||
## Bakgrund
|
||||
|
||||
HemHub stödjer skapande, visning, tilldelning, statusändring och drag-and-drop
|
||||
av uppgifter. Det saknas möjlighet att ta bort uppgifter som inte längre är
|
||||
relevanta eller som skapats av misstag.
|
||||
|
||||
Feature 7 inför permanent radering av en enskild uppgift. Radering hålls
|
||||
separat från generell redigering så att det destruktiva flödet, dess
|
||||
bekräftelse och felhantering kan implementeras och verifieras isolerat.
|
||||
|
||||
## Mål
|
||||
|
||||
Feature 7 ska:
|
||||
|
||||
- införa ett backend-API för permanent radering av en uppgift;
|
||||
- låta användaren initiera radering från uppgiftskortet;
|
||||
- kräva en tydlig bekräftelse före radering;
|
||||
- ta bort kortet från brädan först efter serverbekräftelse;
|
||||
- återanvända befintlig låsning och felhantering per task-id;
|
||||
- fungera tillsammans med statusändring, tilldelning och drag-and-drop utan
|
||||
parallella state- eller requestflöden.
|
||||
|
||||
## Omfattning
|
||||
|
||||
Feature 7 omfattar endast permanent radering av en befintlig uppgift.
|
||||
|
||||
En uppgift får raderas oavsett om dess status är `WAITING`, `IN_PROGRESS` eller
|
||||
`COMPLETED`. Uppgiftens ansvariga användare och aktiv browseranvändare påverkar
|
||||
inte möjligheten att radera. Det lokala användarvalet är inte autentisering
|
||||
eller behörighetskontroll.
|
||||
|
||||
## Avgränsningar
|
||||
|
||||
Feature 7 ska inte införa:
|
||||
|
||||
- mjuk radering, papperskorg, återställning eller undo;
|
||||
- arkivering, versions-, status- eller poänghistorik;
|
||||
- generell redigering;
|
||||
- batchradering eller markering av flera kort;
|
||||
- radering av användare;
|
||||
- behörigheter eller ägarskap;
|
||||
- realtidsuppdatering mellan browsers;
|
||||
- automatisk gallring;
|
||||
- persistent kortordning;
|
||||
- nya relationer till uppgifter;
|
||||
- generell cascade-logik för framtida modeller.
|
||||
|
||||
## Permanent radering
|
||||
|
||||
Radering är permanent. När användaren har bekräftat raderingen tas uppgiften
|
||||
bort ur databasen. Ingen `deleted`-flagga, `deletedAt`, dold arkiveringsmodell
|
||||
eller annan form av mjuk radering införs.
|
||||
|
||||
HemHub är en liten familjeapplikation utan revisionslogg, papperskorg eller
|
||||
återställningsflöde. En mjukraderingsmodell skulle därför öka komplexiteten
|
||||
utan ett tydligt nuvarande produktvärde.
|
||||
|
||||
Om framtida features för återkommande uppgifter eller poänghistorik behöver
|
||||
bevara information efter radering ska deras datamodeller och raderingsregler
|
||||
beslutas i respektive feature.
|
||||
|
||||
## Tillåtna statusar
|
||||
|
||||
Samtliga uppgifter får raderas oavsett status. Det krävs inte att en
|
||||
`IN_PROGRESS`-uppgift först flyttas till `WAITING`, och en `COMPLETED`-uppgift
|
||||
behandlas inte annorlunda än övriga uppgifter.
|
||||
|
||||
Bekräftelseflödet är samma för alla statusar. Ingen extra varning eller
|
||||
ytterligare bekräftelse införs för pågående uppgifter.
|
||||
|
||||
## Raderingskontroll
|
||||
|
||||
Raderingskontrollen ska visas som en diskret sopkorgsikon direkt på det
|
||||
befintliga uppgiftskortet, uppe till höger i ett eget åtgärdsområde. Feature 7
|
||||
inför inget kompakt eller expanderat kortläge.
|
||||
|
||||
Kontrollen ska:
|
||||
|
||||
- visas på befintliga uppgiftskort;
|
||||
- ha en tillgänglig etikett som identifierar uppgiften, exempelvis
|
||||
`Radera Töm diskmaskinen`;
|
||||
- öppna bekräftelsedialogen;
|
||||
- inte starta drag-and-drop;
|
||||
- ha en rimlig klickyta för touch;
|
||||
- ha neutral stil i normalläge och tydlig hover- och fokusmarkering;
|
||||
- vara inaktiverad när samma uppgift har en pågående operation.
|
||||
|
||||
Sopkorgen implementeras som inline-SVG enligt projektets befintliga
|
||||
ikonmönster. Feature 7 lägger inte till något ikonbibliotek.
|
||||
|
||||
Den destruktiva visuella betoningen ska primärt ligga i
|
||||
bekräftelsedialogen. Kontrollen ska kunna flyttas till en framtida meny eller
|
||||
detaljdialog utan att backend-API eller delete-flödet behöver göras om.
|
||||
|
||||
## Bekräftelsedialog
|
||||
|
||||
Radering bekräftas i en separat delete-modal. Den ska följa beteendemönstret i
|
||||
`CreateTaskModal`, men Feature 7 inför ingen gemensam modalkomponent och gör
|
||||
ingen bred modalrefaktorering. Dialogen ska visa:
|
||||
|
||||
- rubriken `Radera uppgift?`;
|
||||
- uppgiftens titel;
|
||||
- tydlig information om att raderingen är permanent;
|
||||
- knappen `Avbryt`;
|
||||
- den destruktivt utformade knappen `Radera`.
|
||||
|
||||
Exempel:
|
||||
|
||||
> Är du säker på att du vill radera **Töm diskmaskinen**? Uppgiften raderas
|
||||
> permanent och kan inte återställas.
|
||||
|
||||
Delete-modalen har inget stängningskryss. Innan delete-anropet har startat ska
|
||||
den kunna stängas med `Avbryt`, Escape eller klick på bakgrunden. Under
|
||||
pågående delete-anrop blockeras samtliga stängningsvägar.
|
||||
|
||||
Dialogen ska följa projektets befintliga modalstruktur och fokusprinciper.
|
||||
`Avbryt` får initialt fokus när dialogen öppnas; den destruktiva knappen
|
||||
`Radera` får inte initialt fokus. Båda knapparna ska vara
|
||||
tangentbordsåtkomliga.
|
||||
|
||||
## Backend-API
|
||||
|
||||
Radering sker genom:
|
||||
|
||||
```http
|
||||
DELETE /api/tasks/{taskId}
|
||||
```
|
||||
|
||||
### Lyckad radering
|
||||
|
||||
När uppgiften finns och raderas svarar backend med `204 No Content` utan body.
|
||||
|
||||
### Okänd uppgift
|
||||
|
||||
Om uppgiften inte finns svarar backend med `404 Not Found` och projektets
|
||||
befintliga felformat:
|
||||
|
||||
```text
|
||||
TASK_NOT_FOUND
|
||||
```
|
||||
|
||||
Det gäller även om samma task-id tidigare har raderats. Ett andra delete-anrop
|
||||
mot samma id ger därför `404 TASK_NOT_FOUND`.
|
||||
|
||||
### Ogiltigt task-id
|
||||
|
||||
Ett task-id som inte kan tolkas som UUID ger `400 Bad Request` med repositoryts
|
||||
nuvarande requestfel och felformat. Den befintliga felkoden
|
||||
`INVALID_TASK_ASSIGNMENT` ändras inte inom Feature 7. Feature 7 inför ingen
|
||||
separat felmodell för UUID-fel.
|
||||
|
||||
### Konflikter och transaktion
|
||||
|
||||
Radering är tillåten för samtliga statusar och oavsett ansvarig. Feature 7 har
|
||||
därför inget domänfall som ger `409 Conflict`.
|
||||
|
||||
Raderingen ska ske inom backendens normala transaktionsgräns och endast ta bort
|
||||
den identifierade uppgiften. Den får inte ändra eller radera ansvarig
|
||||
användare, andra användare eller andra uppgifter.
|
||||
|
||||
## Databas
|
||||
|
||||
Feature 7 raderar raden permanent ur tabellen `task`.
|
||||
|
||||
Nuvarande datamodell har inga dokumenterade beroendeentiteter som kräver en ny
|
||||
migrering eller särskild cascade-policy. Den befintliga relationen från
|
||||
`task.assignee_id` till `app_user.id` ska verifieras så att den inte hindrar
|
||||
radering av uppgiften. Den ansvariga användaren ska finnas kvar.
|
||||
|
||||
Ingen databasmigrering ska skapas om det faktiska schemat redan stödjer
|
||||
radering. Framtida relationer till uppgifter får definiera sin delete-policy
|
||||
när de införs.
|
||||
|
||||
## Frontendens uppdateringsstrategi
|
||||
|
||||
Frontend använder serverbekräftad radering. När användaren bekräftar ska
|
||||
frontend:
|
||||
|
||||
1. markera uppgiften som upptagen;
|
||||
2. behålla kortet i dess nuvarande kolumn;
|
||||
3. behålla bekräftelsedialogen öppen;
|
||||
4. skicka delete-anropet;
|
||||
5. vänta på serverns svar;
|
||||
6. vid `204 No Content` ta bort uppgiften ur den lokala task-listan;
|
||||
7. stänga dialogen;
|
||||
8. frigöra låsningen för task-id.
|
||||
|
||||
Kortet ska inte tas bort optimistiskt. Ingen rollback-modell behövs eftersom
|
||||
kortet ligger kvar under anropet.
|
||||
|
||||
## Vänteläge
|
||||
|
||||
När delete-anropet pågår ska:
|
||||
|
||||
- bekräftelsedialogen ligga kvar öppen;
|
||||
- `Radera` och `Avbryt` vara inaktiverade;
|
||||
- Escape och bakgrundsklick inte kunna stänga dialogen;
|
||||
- kortet ligga kvar i sin kolumn och tonas ned lätt;
|
||||
- alla interaktiva kontroller på samma kort vara inaktiverade.
|
||||
|
||||
Ingen spinner eller text som `Raderar…` krävs.
|
||||
|
||||
## Låsning och samspel med andra operationer
|
||||
|
||||
Delete ska återanvända den befintliga låsningen per task-id. När uppgiften har
|
||||
en pågående status-, tilldelnings- eller dragoperation ska radering inte kunna
|
||||
initieras.
|
||||
|
||||
När delete-anropet pågår ska samma uppgift inte kunna dras, ändra status, ändra
|
||||
ansvarig, öppna en ny raderingsdialog eller skicka ytterligare delete-anrop.
|
||||
Andra kort ska förbli interaktiva och kunna ha egna samtidiga operationer.
|
||||
|
||||
Feature 7 inför inget globalt vänteläge, separat delete-lås eller parallell
|
||||
requestmodell.
|
||||
|
||||
## Felhantering
|
||||
|
||||
### Vanliga delete-fel
|
||||
|
||||
Vid nätverksfel, serverfel eller annat vanligt delete-fel ska:
|
||||
|
||||
- kortet ligga kvar oförändrat;
|
||||
- dialogen ligga kvar öppen;
|
||||
- vänteläget avslutas;
|
||||
- kontrollerna aktiveras igen;
|
||||
- felmeddelandet
|
||||
`Det gick inte att radera uppgiften. Försök igen.` visas i dialogen;
|
||||
- användaren kunna försöka igen eller avbryta.
|
||||
|
||||
Delete-felet ska inte blandas med status- eller tilldelningsfel på kortet.
|
||||
|
||||
### `404 TASK_NOT_FOUND`
|
||||
|
||||
Om backend svarar med `404 TASK_NOT_FOUND` betraktas kortet som inaktuellt.
|
||||
Frontend ska då ta bort uppgiften ur den lokala task-listan, stänga dialogen
|
||||
och frigöra låsningen utan att visa det generella delete-felet.
|
||||
|
||||
Frontendens generella `ApiError`-typ utökas med ett valfritt `code`. Delete-
|
||||
flödet ska använda `code === "TASK_NOT_FOUND"` och status `404` för detta fall
|
||||
och får inte tolka meddelandetexten.
|
||||
|
||||
Andra typer av `404` ska inte behandlas som en redan borttagen uppgift.
|
||||
|
||||
## Frontendtester
|
||||
|
||||
Frontendtesterna ska minst verifiera:
|
||||
|
||||
- att sopkorgsknappen visas direkt på det befintliga uppgiftskortet;
|
||||
- att ikonen är inline-SVG och inte kräver ett ikonbibliotek;
|
||||
- tillgänglig etikett och rätt uppgift i bekräftelsedialogen;
|
||||
- information om permanent radering;
|
||||
- initialt fokus på `Avbryt`, aldrig på `Radera`;
|
||||
- att modalen saknar stängningskryss;
|
||||
- stängning med `Avbryt`, Escape och bakgrundsklick före anrop;
|
||||
- `DELETE /api/tasks/{taskId}` först efter bekräftelse;
|
||||
- att kort och dialog ligger kvar under anropet;
|
||||
- att dialogen inte kan stängas medan anropet pågår;
|
||||
- gemensam låsning för status, tilldelning, drag och radering;
|
||||
- att andra kort förblir interaktiva;
|
||||
- blockering av dubbla delete-anrop;
|
||||
- att `204 No Content` tar bort rätt kort och stänger dialogen;
|
||||
- att vanliga fel behåller kort och dialog samt kan återförsökas;
|
||||
- att `404 TASK_NOT_FOUND` tar bort det inaktuella kortet;
|
||||
- att ett annat `404`-fel inte feltolkas som `TASK_NOT_FOUND`;
|
||||
- att raderingskontrollen inte bryter drag-and-drop;
|
||||
- grundläggande tangentbordsfokus och knappaktivering.
|
||||
|
||||
Testerna ska verifiera beteende och state, inte exakt ikonplacering, färg eller
|
||||
pixelmått.
|
||||
|
||||
## Backendtester
|
||||
|
||||
Backendtesterna ska minst verifiera:
|
||||
|
||||
- radering i `WAITING`, `IN_PROGRESS` och `COMPLETED`;
|
||||
- `204 No Content` utan body;
|
||||
- att den raderade uppgiften inte längre finns i `GET /api/tasks`;
|
||||
- att andra uppgifter och den ansvariga användaren finns kvar oförändrade;
|
||||
- `404 TASK_NOT_FOUND` för okänt id och ett andra delete-anrop;
|
||||
- projektets befintliga `400`-fel för ogiltigt UUID-format.
|
||||
|
||||
Testerna ska följa repositoryts befintliga integrationsteststil.
|
||||
|
||||
## Manuell verifiering
|
||||
|
||||
Följande ska verifieras manuellt:
|
||||
|
||||
1. Sopkorgsknappens placering uppe till höger i ett eget åtgärdsområde,
|
||||
neutrala normalläge, touchyta, hover, fokus och tillgängliga etikett.
|
||||
2. Radering av uppgifter i samtliga tre statusar.
|
||||
3. Initialt fokus på `Avbryt`, inget stängningskryss samt avbrytande med knapp,
|
||||
Escape och bakgrundsklick före anrop.
|
||||
4. Rätt titel och information om permanent radering.
|
||||
5. Titel nära maximal längd.
|
||||
6. Blockering av dubbla delete-anrop.
|
||||
7. Vänteläge för kort och dialog under fördröjt svar.
|
||||
8. Låsning av drag, status, tilldelning och ny radering för samma kort.
|
||||
9. Fortsatt interaktion med andra kort.
|
||||
10. Vanligt serverfel, visat felmeddelande och nytt försök.
|
||||
11. `404 TASK_NOT_FOUND` och lokal borttagning av inaktuellt kort.
|
||||
12. Desktop- och mobilbredd samt tangentbordsaktivering.
|
||||
13. Omladdning efter lyckad radering så att uppgiften inte återkommer.
|
||||
|
||||
## Dokumentation
|
||||
|
||||
Feature 7 dokumenteras i:
|
||||
|
||||
```text
|
||||
docs/features/007-task-deletion.md
|
||||
```
|
||||
|
||||
Vid implementation ska `README.md`, `docs/architecture.md` och
|
||||
`docs/roadmap.md` uppdateras när det är relevant.
|
||||
|
||||
Roadmapen ska markera Feature 7 som `Klar` först efter implementation,
|
||||
automatiska tester, manuell verifiering och merge.
|
||||
|
||||
Ett nytt ADR behövs inte för permanent radering. Beslutet gäller den nuvarande
|
||||
task-livscykeln och etablerar inte en generell raderingspolicy för framtida
|
||||
entiteter.
|
||||
|
||||
## Acceptanskriterier
|
||||
|
||||
Feature 7 är klar när:
|
||||
|
||||
- en uppgift kan raderas permanent med `DELETE /api/tasks/{taskId}`;
|
||||
- lyckad radering ger `204 No Content`;
|
||||
- okänd eller redan raderad uppgift ger `404 TASK_NOT_FOUND`;
|
||||
- ogiltigt UUID-format följer befintlig felhantering;
|
||||
- alla tre statusar kan raderas oavsett ansvarig eller aktiv användare;
|
||||
- radering kräver en egen bekräftelsedialog;
|
||||
- dialogen visar rätt titel och anger att raderingen inte kan återställas;
|
||||
- `Avbryt` får initialt fokus och `Radera` får inte initialt fokus;
|
||||
- modalen saknar stängningskryss och blockerar alla stängningsvägar under
|
||||
anropet;
|
||||
- en diskret inline-SVG-sopkorg visas direkt på befintliga uppgiftskort;
|
||||
- raderingskontrollen startar inte drag;
|
||||
- kortet tas bort först efter serverbekräftelse;
|
||||
- samma task-id låses för status, tilldelning, drag och ny radering;
|
||||
- andra kort förblir interaktiva;
|
||||
- vanliga fel behåller kort och dialog och kan återförsökas;
|
||||
- `404 TASK_NOT_FOUND` tar bort det inaktuella lokala kortet;
|
||||
- ingen mjukradering, återställningsmodell eller onödig migrering införs;
|
||||
- backend- och frontendtester täcker centrala flöden;
|
||||
- manuell verifiering genomförs;
|
||||
- relevant dokumentation uppdateras.
|
||||
|
||||
## Implementerad lösning
|
||||
|
||||
Backendens task-controller och task-service har utökats med fysisk radering via
|
||||
`DELETE /api/tasks/{taskId}`. Servicen hämtar först uppgiften för att
|
||||
återanvända `TaskNotFoundException` och raderar därefter entiteten inom en
|
||||
transaktion. Ingen entitet, exception handler eller Flyway-migrering behövde
|
||||
ändras.
|
||||
|
||||
Frontendens `TaskBoard` använder samma per-task-lås som status- och
|
||||
tilldelningsoperationerna. Radering är serverbekräftad: kortet och dialogen
|
||||
ligger kvar medan anropet pågår, och kortet tas bort först efter `204 No
|
||||
Content`. Endast ett svar med både status `404` och felkoden
|
||||
`TASK_NOT_FOUND` tar bort ett känt inaktuellt kort. Övriga fel behåller kortet
|
||||
och dialogen så att användaren kan försöka igen.
|
||||
|
||||
`TaskCard` har inget kompakt eller expanderat läge. En neutral
|
||||
inline-SVG-knapp ligger direkt i kortets övre högra åtgärdsområde och stoppar
|
||||
pointer-händelsen innan den når dragytan. Den separata delete-modalen följer
|
||||
`CreateTaskModal`-mönstret utan en gemensam modalabstraktion. `Avbryt` får
|
||||
initialt fokus, modalen saknar stängningskryss och samtliga stängningsvägar
|
||||
blockeras under delete-anropet.
|
||||
|
||||
Frontendens generella `ApiError` innehåller nu ett valfritt `code`. Den
|
||||
befintliga backendhanteringen av felaktigt UUID är oförändrad och returnerar
|
||||
fortsatt `400 INVALID_TASK_ASSIGNMENT`.
|
||||
|
||||
## Tester och verifiering
|
||||
|
||||
Automatiskt verifierat:
|
||||
|
||||
- backendens fullständiga testsvit: 47 tester passerade;
|
||||
- frontendens fullständiga testsvit: 49 tester passerade;
|
||||
- frontendens produktionsbygge och TypeScript-kompilering passerade;
|
||||
- `git diff --check` passerade.
|
||||
|
||||
Backendtesterna ligger i den separata integrationstestklassen
|
||||
`TaskDeletionApiTest`. Frontendens delete-flöden testas tillsammans med övriga
|
||||
brädbeteenden i `App.test.tsx`.
|
||||
|
||||
Manuell browserverifiering genomfördes mot lokalt körande frontend och backend
|
||||
i Chrome. Följande verifierades:
|
||||
|
||||
- permanent radering och kvarstående borttagning efter omladdning för
|
||||
`WAITING`, `IN_PROGRESS` och `COMPLETED`;
|
||||
- lång titel, radbrytning och korrekt uppgiftstitel i dialogen;
|
||||
- initialt fokus på `Avbryt`, tabb-ordning till `Radera`, Escape och
|
||||
bakgrundsklick före anrop samt avsaknad av stängningskryss;
|
||||
- fördröjd delete-respons med kvarvarande och nedtonat kort, öppen låst modal
|
||||
och inaktiverade stängningsvägar;
|
||||
- gemensam låsning av drag, status, ansvarig och ny radering för samma kort,
|
||||
samtidigt som andra kort förblev interaktiva;
|
||||
- snabbt dubbelklick på `Radera` utan dubbla delete-anrop;
|
||||
- vanligt serverfel där kort och modal låg kvar, felet visades och ett nytt
|
||||
försök lyckades;
|
||||
- `404 TASK_NOT_FOUND`, där det inaktuella kortet togs bort lokalt;
|
||||
- neutral sopkorgsknapp med 40 × 40 pixlars klickyta, inline-SVG och
|
||||
pointer-hantering som inte startade drag;
|
||||
- desktopbredd 1440 × 1000 och mobilbredd 390 × 844 utan horisontell
|
||||
scrollning.
|
||||
|
||||
Inga problem upptäcktes i Feature 7-flödena.
|
||||
|
||||
## Relaterade commits
|
||||
|
||||
- Feature-commit: `f296d15`
|
||||
- Merge-commit: `5df0146`
|
||||
|
||||
## Implementationsprinciper
|
||||
|
||||
Före implementation ska Codex läsa:
|
||||
|
||||
```text
|
||||
AGENTS.md
|
||||
README.md
|
||||
docs/architecture.md
|
||||
docs/development.md
|
||||
docs/roadmap.md
|
||||
docs/decisions/
|
||||
docs/features/005-task-status.md
|
||||
docs/features/006-task-drag-and-drop.md
|
||||
```
|
||||
|
||||
Codex ska även läsa relevant backendkod, frontendkod och befintliga tester.
|
||||
Repositoryts faktiska kod, tester och dokumentation har företräde framför
|
||||
antaganden i detta dokument.
|
||||
|
||||
Implementation, tester och relevant dokumentation ska uppdateras tillsammans.
|
||||
Codex ska inte committa, pusha, skapa pull request eller merga utan uttrycklig
|
||||
instruktion.
|
||||
458
docs/roadmap.md
Normal file
458
docs/roadmap.md
Normal file
@ -0,0 +1,458 @@
|
||||
# HemHub roadmap
|
||||
|
||||
## Syfte
|
||||
|
||||
Roadmapen är HemHubs styrande plan för val och ordning av kommande features. Den
|
||||
utgår från den faktiska implementationen efter Feature 2 och från beslut som
|
||||
dokumenterats i arkitektur- och beslutsdokumenten.
|
||||
|
||||
Planen är ändringsbar. Ordningen uttrycker nuvarande prioritering och beroenden,
|
||||
inte ett löfte om att alla features måste genomföras oförändrade.
|
||||
|
||||
## Regler för användning
|
||||
|
||||
- Nästa feature ska väljas från denna roadmap.
|
||||
- Feature 0–2 behåller sina nummer och sin historiska betydelse.
|
||||
- Ändra roadmapen innan en feature delas, flyttas, ersätts eller läggs till.
|
||||
- Dokumentera motiv och beroendeförändringar innan utveckling påbörjas.
|
||||
- En ChatGPT- eller Codex-dialog får inte skapa en alternativ featureplan utan
|
||||
att roadmapen först uppdateras i repositoryt.
|
||||
- Håll varje feature tillräckligt liten för separat implementation och
|
||||
verifiering.
|
||||
- Beskriv mål och affärsregler här; bindande implementationsdetaljer hör till
|
||||
feature-specifikationen och relevanta beslutsdokument.
|
||||
- En designfeature producerar dokument och beslut, inte produktionskod, om inget
|
||||
annat uttryckligen beslutas.
|
||||
|
||||
Följande statusvärden används:
|
||||
|
||||
- **Klar** – implementerad, verifierad och mergad.
|
||||
- **Planerad** – ingår i nuvarande ordning men har inte påbörjats.
|
||||
- **Pågående** – utveckling pågår i en aktiv feature.
|
||||
- **Villkorad** – genomförs endast om det angivna villkoret uppfylls.
|
||||
- **Ersatt** – har ersatts av en dokumenterad annan feature eller plan.
|
||||
|
||||
## Nuvarande läge
|
||||
|
||||
Feature 0–7 är klara och finns på `main`. Den aktuella applikationen har:
|
||||
|
||||
- ett monorepo med separat React/Vite-frontend och Spring Boot-backend;
|
||||
- centralt lagrade användare och ett lokalt browserval av aktiv användare;
|
||||
- gemensamma uppgifter med titel, valfri beskrivning, status och poäng;
|
||||
- skapande och listning av uppgifter;
|
||||
- valfri tilldelning av högst en ansvarig användare per uppgift;
|
||||
- tilldelning och byte av ansvarig i samtliga statusar;
|
||||
- borttagning av ansvarig i `WAITING` och `COMPLETED`;
|
||||
- backendstyrda statusändringar mellan `WAITING`, `IN_PROGRESS` och `COMPLETED`;
|
||||
- automatisk tilldelning till aktiv användare när en otilldelad uppgift påbörjas;
|
||||
- drag-and-drop mellan statuskolumner med optimistisk flytt och rollback;
|
||||
- serverbekräftad permanent radering med bekräftelsedialog;
|
||||
- en bräda med Väntande, Pågående och Klart;
|
||||
- nya uppgifter som alltid skapas med status `WAITING`.
|
||||
|
||||
Tilldelning och status är separata egenskaper; tilldelningsflödet ändrar inte
|
||||
uppgiftens status. Alla direkta statusövergångar är tillåtna och
|
||||
`IN_PROGRESS` kräver ansvarig. Det finns ännu ingen redigering, deadline eller
|
||||
återkommande uppgift. Nuvarande användarval är inte autentisering.
|
||||
|
||||
**Feature 8 – Redigera uppgift är nästa planerade produktfeature.**
|
||||
|
||||
## Featureöversikt
|
||||
|
||||
| Feature och namn | Status | Beroenden | Huvudsakligt resultat |
|
||||
| --- | --- | --- | --- |
|
||||
| 0 – Projektgrund | Klar | – | Körbar frontend, backend och lokal API-koppling |
|
||||
| 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 | Klar | 2 | Poäng på uppgifter |
|
||||
| 4 – Tilldelning | Klar | 1–2 | Valfri ansvarig användare |
|
||||
| 5 – Statusändring | Klar | 4 | Backendstyrda statusövergångar |
|
||||
| 6 – Drag-and-drop | Klar | 5 | Kortflytt via status-API |
|
||||
| 7 – Radera uppgift | Klar | 2 | Bekräftad permanent radering |
|
||||
| 8 – Redigera uppgift | Planerad | 3 | Titel, beskrivning och poäng |
|
||||
| 9 – Deadline | Planerad | 2 | Valfri deadline och förseningsmarkering |
|
||||
| 10 – Sökning och filtrering | Planerad | 2; 4 för ansvarig; 9 för deadline | Sökning och filter på brädan |
|
||||
| 11 – Design av återkommande uppgifter | Planerad | 3–5, 9 | Beslut och plan, ingen produktionskod |
|
||||
| 12 – Återkommande uppgifter | Planerad | 5, 9, 11 | Implementerad återkommandemodell |
|
||||
| 13 – Poänghistorik och summering | Planerad | 3–5, 12 | Slutförandehistorik och summering |
|
||||
| 14 – PostgreSQL | Planerad | 0–13 | Verifierad produktionsdatabas |
|
||||
| 15 – Dockerpaketering | Planerad | 14 | Images och produktionslik lokal körning |
|
||||
| 16 – Pipeline och deployment | Planerad | 15 | Bygge, publicering och drift på Biff |
|
||||
| 17 – Autentisering | Villkorad | 16, extern åtkomst | Säker internetexponering |
|
||||
|
||||
## Genomförda features
|
||||
|
||||
### Feature 0 – Projektgrund
|
||||
|
||||
**Status:** Klar
|
||||
|
||||
Feature 0 etablerade monorepot, React/Vite-frontend, Spring Boot-backend,
|
||||
health-endpoint, lokal Vite-proxy och grundtester. Den faktiska lösningen
|
||||
beskrivs i
|
||||
[`000-project-foundation.md`](features/000-project-foundation.md).
|
||||
|
||||
### Feature 1 – Användarval
|
||||
|
||||
**Status:** Klar
|
||||
|
||||
Feature 1 införde skapande och listning av användare, val av aktiv användare och
|
||||
lokal lagring av användarens id. Lösningen är ett browserlokalt användarval,
|
||||
inte riktig autentisering. Den faktiska lösningen beskrivs i
|
||||
[`001-user-selection.md`](features/001-user-selection.md).
|
||||
|
||||
### Feature 2 – Skapa uppgifter
|
||||
|
||||
**Status:** Klar
|
||||
|
||||
Feature 2 införde en grundläggande uppgiftsmodell, API för att lista och skapa
|
||||
uppgifter samt en bräda med tre statuskolumner. Uppgifter har titel, valfri
|
||||
beskrivning och status; nya uppgifter skapas som `WAITING`. Den faktiska
|
||||
lösningen beskrivs i
|
||||
[`002-task-creation.md`](features/002-task-creation.md).
|
||||
|
||||
## Fas 1 – Komplettera den centrala uppgiftsmodellen
|
||||
|
||||
Fasen lägger till den domändata och de backendregler som behövs innan mer
|
||||
interaktiv brädhantering införs.
|
||||
|
||||
### Feature 3 – Uppgiftspoäng
|
||||
|
||||
**Status:** Klar
|
||||
|
||||
**Beroenden:** Feature 2
|
||||
|
||||
**Mål:**
|
||||
|
||||
- lägga till obligatoriska poäng på uppgifter;
|
||||
- välja och dokumentera poängskala;
|
||||
- ange poäng vid skapande;
|
||||
- visa poäng på uppgiftskort;
|
||||
- migrera befintliga uppgifter kontrollerat.
|
||||
|
||||
Feature 3 ligger först eftersom poäng blir ett centralt uppgiftsfält som senare
|
||||
ska kunna redigeras och historikföras.
|
||||
|
||||
Poängskalan är beslutad till alla heltal mellan 1 och 99. V3-migreringen ger
|
||||
eventuella befintliga uppgifter värdet `1` innan kolumnen görs obligatorisk;
|
||||
databasen har inget permanent defaultvärde.
|
||||
|
||||
### Feature 4 – Tilldelning av uppgifter
|
||||
|
||||
**Status:** Klar
|
||||
|
||||
**Beroenden:** Feature 1 och Feature 2
|
||||
|
||||
**Mål:**
|
||||
|
||||
- lägga till en valfri ansvarig användare;
|
||||
- tillåta ansvarig vid skapande;
|
||||
- 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
|
||||
ansvarig. Tilldelning införs före statusändring eftersom en pågående uppgift
|
||||
senare måste ha en ansvarig. Hur borttagna användare ska hanteras är fortsatt
|
||||
öppet tills användarradering införs.
|
||||
|
||||
### Feature 5 – Statusändring och statusregler
|
||||
|
||||
**Status:** Klar
|
||||
|
||||
**Beroenden:** Feature 4
|
||||
|
||||
**Mål:**
|
||||
|
||||
- införa backend-API för statusändring;
|
||||
- stödja `WAITING`, `IN_PROGRESS` och `COMPLETED`;
|
||||
- säkerställa statusregler i backend;
|
||||
- ge ett enkelt UI för statusändring före drag-and-drop.
|
||||
|
||||
`IN_PROGRESS` kräver en ansvarig användare. Statusflödet införs före
|
||||
drag-and-drop så att affärsregeln och API:t kan verifieras utan att samtidigt
|
||||
bygga en komplex interaktion. Feature 5 återanvänder Feature 4:s
|
||||
tilldelningsmodell och särskilda API för ansvarig; statusändring sker i ett
|
||||
separat statusflöde.
|
||||
|
||||
En otilldelad uppgift som sätts till `IN_PROGRESS` tilldelas automatiskt den
|
||||
aktiva browseranvändaren. En befintlig ansvarig behålls. Ansvarig kan bytas men
|
||||
inte tas bort medan uppgiften är pågående.
|
||||
|
||||
### Feature 6 – Drag-and-drop
|
||||
|
||||
**Status:** Klar
|
||||
|
||||
**Beroenden:** Feature 5
|
||||
|
||||
**Mål:**
|
||||
|
||||
- flytta uppgiftskort mellan statuskolumner;
|
||||
- använda status-API:t från Feature 5;
|
||||
- hantera serverfel och återställning av UI;
|
||||
- hantera en otilldelad uppgift som flyttas till Pågående.
|
||||
|
||||
Drag-and-drop kommer efter det enklare statusflödet för att återanvända
|
||||
verifierade backendregler.
|
||||
|
||||
Drag-and-drop använder en kontrollerad optimistisk flytt. Vid fel återställs
|
||||
hela den tidigare task-versionen. En otilldelad uppgift som dras till Pågående
|
||||
använder Feature 5:s befintliga automatiska tilldelning till aktiv användare.
|
||||
Serverns fullständiga task-respons ersätter alltid det optimistiska värdet.
|
||||
Statusknapparna förblir tills vidare serverbekräftade. Implementation och
|
||||
automatisk samt manuell verifiering är genomförda, och featuren är mergad till
|
||||
`main`.
|
||||
|
||||
## Fas 2 – Hantering av uppgifter
|
||||
|
||||
Fasen kompletterar livscykeln för enskilda uppgifter efter att den centrala
|
||||
modellen och statusreglerna finns.
|
||||
|
||||
### Feature 7 – Radera uppgift
|
||||
|
||||
**Status:** Klar
|
||||
|
||||
**Beroenden:** Feature 2
|
||||
|
||||
**Mål:**
|
||||
|
||||
- införa backend-API för radering;
|
||||
- radera en uppgift från brädan;
|
||||
- kräva bekräftelse före radering.
|
||||
|
||||
Radering hålls separat från redigering så att databorttagning och dess
|
||||
konsekvenser kan verifieras isolerat.
|
||||
|
||||
Feature 7 använder permanent fysisk radering genom
|
||||
`DELETE /api/tasks/{taskId}`. En bekräftelsemodal visas före anropet och
|
||||
frontend behåller kortet tills backend har bekräftat raderingen. Operationen
|
||||
använder samma låsning per task-id som status, tilldelning och drag-and-drop.
|
||||
Ett `404 TASK_NOT_FOUND` tar bort ett känt inaktuellt lokalt kort.
|
||||
Implementation samt automatisk och manuell verifiering är genomförda, och
|
||||
featuren är mergad till `main`.
|
||||
|
||||
### Feature 8 – Redigera uppgift
|
||||
|
||||
**Status:** Planerad
|
||||
|
||||
**Beroenden:** Feature 3
|
||||
|
||||
**Mål:**
|
||||
|
||||
- ändra titel;
|
||||
- ändra beskrivning;
|
||||
- ändra poäng.
|
||||
|
||||
Featuren ligger efter poäng för att redigeringsflödet ska omfatta den då
|
||||
aktuella uppgiftsmodellen. Ansvarig ska fortsatt ändras genom
|
||||
tilldelningsflödet från Feature 4 och status genom statusflödet från Feature 5.
|
||||
|
||||
### Feature 9 – Deadline
|
||||
|
||||
**Status:** Planerad
|
||||
|
||||
**Beroenden:** Feature 2
|
||||
|
||||
**Mål:**
|
||||
|
||||
- lägga till en valfri deadline;
|
||||
- stödja beslutad representation av datum och eventuell tid;
|
||||
- visa deadline på uppgiftskort;
|
||||
- markera försenade uppgifter.
|
||||
|
||||
Deadline införs före återkommande uppgifter eftersom framtida förekomster måste
|
||||
kunna ärva eller beräkna deadlines.
|
||||
|
||||
**Öppna frågor:**
|
||||
|
||||
- datum utan tid eller datum och tid;
|
||||
- tidszonshantering;
|
||||
- definition av en försenad uppgift.
|
||||
|
||||
### Feature 10 – Sökning och filtrering
|
||||
|
||||
**Status:** Planerad
|
||||
|
||||
**Beroenden:** Feature 2; Feature 4 för ansvarigfilter; Feature 9 om
|
||||
deadlinefilter ska ingå
|
||||
|
||||
**Mål:**
|
||||
|
||||
- söka på titel och beskrivning;
|
||||
- filtrera på ansvarig och status;
|
||||
- eventuellt filtrera på deadline.
|
||||
|
||||
Datamängden i ett familjehushåll är sannolikt liten. Klientbaserad sökning kan
|
||||
därför vara tillräcklig initialt, men valet ska göras i feature-specifikationen.
|
||||
Sökning på titel och beskrivning samt statusfiltrering kan byggas från Feature
|
||||
2. Filtrering på ansvarig kräver Feature 4, och deadlinefilter kräver Feature 9
|
||||
om det ska ingå.
|
||||
|
||||
**Öppen fråga:**
|
||||
|
||||
- klientbaserad eller serverbaserad sökning.
|
||||
|
||||
## Fas 3 – Återkommande arbete och historik
|
||||
|
||||
Fasen kräver först ett uttryckligt modellbeslut, eftersom återkommande arbete
|
||||
påverkar status, deadline, ansvarig och poäng.
|
||||
|
||||
### Feature 11 – Design av återkommande uppgifter
|
||||
|
||||
**Status:** Planerad
|
||||
|
||||
**Typ:** Designfeature
|
||||
|
||||
**Beroenden:** Beslutade modeller från Feature 3–5 och Feature 9
|
||||
|
||||
**Ingen produktionskod ska implementeras i denna feature.**
|
||||
|
||||
**Mål:**
|
||||
|
||||
- besluta skillnaden mellan uppgiftsmall och konkret förekomst;
|
||||
- definiera hur nästa förekomst skapas;
|
||||
- definiera vad slutförande betyder;
|
||||
- definiera hur en förekomst hoppas över;
|
||||
- definiera hur ändringar påverkar framtida förekomster;
|
||||
- definiera hur ansvarig, poäng och deadline ärvs.
|
||||
|
||||
Resultatet ska vara ett beslutsdokument och en avgränsad implementationsplan för
|
||||
Feature 12. Designsteget ligger före implementationen för att undvika att
|
||||
domänbeslut byggs in implicit.
|
||||
|
||||
### Feature 12 – Implementera återkommande uppgifter
|
||||
|
||||
**Status:** Planerad
|
||||
|
||||
**Beroenden:** Feature 5, Feature 9 och Feature 11
|
||||
|
||||
**Mål:**
|
||||
|
||||
- implementera modellen som beslutades i Feature 11.
|
||||
|
||||
### Feature 13 – Poänghistorik och summering
|
||||
|
||||
**Status:** Planerad
|
||||
|
||||
**Beroenden:** Feature 3, Feature 4, Feature 5 och Feature 12
|
||||
|
||||
**Mål:**
|
||||
|
||||
- registrera vem som slutförde en uppgift;
|
||||
- registrera när uppgiften slutfördes;
|
||||
- summera poäng per användare och period;
|
||||
- visa enkel historik.
|
||||
|
||||
Historik ligger efter status, poäng och återkommande uppgifter eftersom
|
||||
slutförandet måste vara en backendvaliderad händelse med ett definierat
|
||||
poängvärde och en definierad konkret förekomst.
|
||||
|
||||
**Öppna frågor:**
|
||||
|
||||
- om tilldelad och slutförande användare kan vara olika;
|
||||
- om poäng delas ut vid varje återkommande förekomst;
|
||||
- hur återöppnade uppgifter påverkar historik.
|
||||
|
||||
## Fas 4 – Produktion
|
||||
|
||||
Produktionsfasen realiserar den beslutade riktningen i
|
||||
[`005-production-deployment-direction.md`](decisions/005-production-deployment-direction.md).
|
||||
Inget i denna fas är implementerat i nuläget.
|
||||
|
||||
### Feature 14 – PostgreSQL och produktionsdatabas
|
||||
|
||||
**Status:** Planerad
|
||||
|
||||
**Beroenden:** Föregående produktfeatures vars persistens ska produktionssättas
|
||||
|
||||
**Mål:**
|
||||
|
||||
- lägga till produktionskonfiguration för PostgreSQL;
|
||||
- verifiera Flyway-migreringar mot PostgreSQL;
|
||||
- införa PostgreSQL-baserade integrationstester, exempelvis med Testcontainers;
|
||||
- behålla en enkel lokal utvecklingsupplevelse.
|
||||
|
||||
PostgreSQL införs före paketering för att databasdrivrutin, migreringar och
|
||||
konfiguration ska vara verifierade innan en produktionslik stack byggs.
|
||||
|
||||
**Öppen fråga:**
|
||||
|
||||
- om lokal utveckling fortsatt ska kunna använda H2.
|
||||
|
||||
### Feature 15 – Dockerpaketering
|
||||
|
||||
**Status:** Planerad
|
||||
|
||||
**Beroenden:** Feature 14
|
||||
|
||||
**Mål:**
|
||||
|
||||
- skapa en backend-image;
|
||||
- skapa en frontend-image;
|
||||
- stödja produktionslik lokal körning;
|
||||
- ge same-origin `/api` via Nginx.
|
||||
|
||||
Paketeringen kommer före pipelinearbetet så att images kan byggas och verifieras
|
||||
lokalt.
|
||||
|
||||
### Feature 16 – Pipeline och deployment
|
||||
|
||||
**Status:** Planerad
|
||||
|
||||
**Beroenden:** Feature 15
|
||||
|
||||
**Mål:**
|
||||
|
||||
- skapa en Drone-pipeline;
|
||||
- publicera images till ett privat registry;
|
||||
- driftsätta på Ubuntu-servern Biff;
|
||||
- uppdatera tjänster med Watchtower;
|
||||
- införa nödvändig Nginx-konfiguration;
|
||||
- hantera secrets utanför Git.
|
||||
|
||||
Exakta miljödetaljer ska beslutas inom featuren och känsliga värden ska inte
|
||||
committas.
|
||||
|
||||
## Fas 5 – Eventuell extern åtkomst
|
||||
|
||||
### Feature 17 – Autentisering och internetexponering
|
||||
|
||||
**Status:** Villkorad
|
||||
|
||||
**Beroenden:** Beslut att exponera HemHub mot internet och en säker
|
||||
produktionsgrund, normalt Feature 16
|
||||
|
||||
Featuren ska endast genomföras om HemHub ska göras åtkomlig från internet.
|
||||
Nuvarande aktiva användarval är uttryckligen inte autentisering.
|
||||
|
||||
**Mål:**
|
||||
|
||||
- införa riktig autentisering;
|
||||
- införa behörighetsregler;
|
||||
- använda säker sessions- eller tokenhantering;
|
||||
- konfigurera TLS och extern exponering;
|
||||
- säkerhetsgranska API och deployment.
|
||||
|
||||
## Öppna tvärgående frågor
|
||||
|
||||
- Hur ska datum, tider och tidszoner representeras?
|
||||
- Ska H2 behållas för lokal utveckling efter PostgreSQL-införandet?
|
||||
- Hur ska användare senare kunna redigeras eller raderas, särskilt när de är
|
||||
ansvariga för eller har slutfört uppgifter?
|
||||
- Vid vilken typ av extern åtkomst krävs riktig autentisering?
|
||||
- Hur ska UI-designen från befintliga skisser införas inkrementellt utan att
|
||||
blanda in framtida funktionalitet?
|
||||
|
||||
## Ändringshistorik
|
||||
|
||||
- 2026-07-27: Feature 7 verifierades och mergades. Permanent,
|
||||
serverbekräftad radering infördes, och Feature 8 blev nästa planerade
|
||||
produktfeature.
|
||||
- 2026-07-27: Feature 6 verifierades och mergades. Optimistisk drag-and-drop
|
||||
med full rollback infördes, och Feature 7 blev nästa planerade
|
||||
produktfeature.
|
||||
- 2026-07-27: Feature 5 verifierades och mergades. Backendstyrda
|
||||
statusövergångar, automatisk tilldelning vid påbörjande och statusberoende
|
||||
tilldelningsregler infördes. Feature 6 blev nästa planerade produktfeature.
|
||||
- 2026-07-26: Feature 3 och Feature 4 markerades som klara efter verifiering och
|
||||
merge. Feature 5 blev nästa planerade produktfeature.
|
||||
- 2026-07-26: Roadmapen etablerades. Feature 0–2 markerades som klara, Feature
|
||||
3–16 planerades och Feature 17 markerades som villkorad.
|
||||
@ -9,6 +9,8 @@
|
||||
"test": "vitest run"
|
||||
},
|
||||
"dependencies": {
|
||||
"@dnd-kit/dom": "0.5.0",
|
||||
"@dnd-kit/react": "0.5.0",
|
||||
"react": "19.2.8",
|
||||
"react-dom": "19.2.8"
|
||||
},
|
||||
|
||||
74
frontend/pnpm-lock.yaml
generated
74
frontend/pnpm-lock.yaml
generated
@ -8,6 +8,12 @@ importers:
|
||||
|
||||
.:
|
||||
dependencies:
|
||||
'@dnd-kit/dom':
|
||||
specifier: 0.5.0
|
||||
version: 0.5.0
|
||||
'@dnd-kit/react':
|
||||
specifier: 0.5.0
|
||||
version: 0.5.0(react-dom@19.2.8(react@19.2.8))(react@19.2.8)
|
||||
react:
|
||||
specifier: 19.2.8
|
||||
version: 19.2.8
|
||||
@ -115,6 +121,27 @@ packages:
|
||||
resolution: {integrity: sha512-QxULHAm7cNu72w97JUNCBFODFaXpbDg+dP8b/oWFAZ2MTRppA3U00Y2L1HqaS4J6yBqxwa/Y3nMBaxVKbB/NsA==}
|
||||
engines: {node: '>=20.19.0'}
|
||||
|
||||
'@dnd-kit/abstract@0.5.0':
|
||||
resolution: {integrity: sha512-hi13iMJgjPX/KDYVKg5VeDIhmYiV6buc9bAX+tCLYf4QdyYjPbsXjn2sPo6m7fQ6SGJBEFgHJ2PemeKDUbwBaA==}
|
||||
|
||||
'@dnd-kit/collision@0.5.0':
|
||||
resolution: {integrity: sha512-xUqRn3lS7oqLkT0AnnHS/STh/Czvwe1UapZFYiLbsUGxopMsQd4teaPCzPouOThoMdGEe+dHWjfqJl6t9iG4mQ==}
|
||||
|
||||
'@dnd-kit/dom@0.5.0':
|
||||
resolution: {integrity: sha512-f2xFJp5SYQ8EW/Fbtaa8iBb66hpkWc7qa8vU826KW11/tb44sH+AisZnGtwOOTWTQ0GraqBDr5ixTErww+eKXw==}
|
||||
|
||||
'@dnd-kit/geometry@0.5.0':
|
||||
resolution: {integrity: sha512-ubHQS1CiSDH8ssYH2xG5BnpwPSFP1tStXXjug7/Ba6qnQdu/EUH47l6QXKIksQnnanfVfDf0aGeevRxgZlj28A==}
|
||||
|
||||
'@dnd-kit/react@0.5.0':
|
||||
resolution: {integrity: sha512-abQPLI8lmfVE+v/n+pqy5WFxrw6T2Yg0UQZsL78dp5DKci7dKTVDjvLWqvass+XTFtzJmsZEjk1NdqE6xG8Jiw==}
|
||||
peerDependencies:
|
||||
react: ^18.0.0 || ^19.0.0
|
||||
react-dom: ^18.0.0 || ^19.0.0
|
||||
|
||||
'@dnd-kit/state@0.5.0':
|
||||
resolution: {integrity: sha512-y7XbabQqjF58Lk8YmDQuR8l6QjN+Kh4qlGEjUvHuIeasLk1QP+9L5diXS98VMxQIivyMmUtX2//f+3N7qPJX4w==}
|
||||
|
||||
'@emnapi/core@1.11.1':
|
||||
resolution: {integrity: sha512-RSvbQmHzdKzNsLYa/wHrbc3KN4sYLKAdPZxqiM2HATqv/SBk2/ENSHpvXGaLOMcsAyz0poEGqkmmKYG3OWiJEQ==}
|
||||
|
||||
@ -145,6 +172,9 @@ packages:
|
||||
'@oxc-project/types@0.139.0':
|
||||
resolution: {integrity: sha512-r9gHphtCs+1M7J0pw6Sn/hh/Wpa/iQrOOkrNAlVLF/gHq+/CJmHIWKKUUhdWjcD6CIa8idarspCsASiXCXvFUw==}
|
||||
|
||||
'@preact/signals-core@1.14.4':
|
||||
resolution: {integrity: sha512-HNB6HYeYKhQbJ1aKl+YRjrS4+QWHLKX6qKoUsfS/m0vqzsVaEBiZiaKbG/e+NKk2ch5ALQr/ihWaMHxiCuuWHA==}
|
||||
|
||||
'@rolldown/binding-android-arm64@1.1.5':
|
||||
resolution: {integrity: sha512-lZg8fqIv2v7FF237bwMgzGZEJvGL79/s5knJ/i6FmsGF4XXlzccZ4jb+TrFIxtSSxFtIpdsgrPZeMk1I9AFcyQ==}
|
||||
engines: {node: ^20.19.0 || >=22.12.0}
|
||||
@ -961,6 +991,45 @@ snapshots:
|
||||
|
||||
'@csstools/css-tokenizer@4.0.0': {}
|
||||
|
||||
'@dnd-kit/abstract@0.5.0':
|
||||
dependencies:
|
||||
'@dnd-kit/geometry': 0.5.0
|
||||
'@dnd-kit/state': 0.5.0
|
||||
tslib: 2.8.1
|
||||
|
||||
'@dnd-kit/collision@0.5.0':
|
||||
dependencies:
|
||||
'@dnd-kit/abstract': 0.5.0
|
||||
'@dnd-kit/geometry': 0.5.0
|
||||
tslib: 2.8.1
|
||||
|
||||
'@dnd-kit/dom@0.5.0':
|
||||
dependencies:
|
||||
'@dnd-kit/abstract': 0.5.0
|
||||
'@dnd-kit/collision': 0.5.0
|
||||
'@dnd-kit/geometry': 0.5.0
|
||||
'@dnd-kit/state': 0.5.0
|
||||
tslib: 2.8.1
|
||||
|
||||
'@dnd-kit/geometry@0.5.0':
|
||||
dependencies:
|
||||
'@dnd-kit/state': 0.5.0
|
||||
tslib: 2.8.1
|
||||
|
||||
'@dnd-kit/react@0.5.0(react-dom@19.2.8(react@19.2.8))(react@19.2.8)':
|
||||
dependencies:
|
||||
'@dnd-kit/abstract': 0.5.0
|
||||
'@dnd-kit/dom': 0.5.0
|
||||
'@dnd-kit/state': 0.5.0
|
||||
react: 19.2.8
|
||||
react-dom: 19.2.8(react@19.2.8)
|
||||
tslib: 2.8.1
|
||||
|
||||
'@dnd-kit/state@0.5.0':
|
||||
dependencies:
|
||||
'@preact/signals-core': 1.14.4
|
||||
tslib: 2.8.1
|
||||
|
||||
'@emnapi/core@1.11.1':
|
||||
dependencies:
|
||||
'@emnapi/wasi-threads': 1.2.2
|
||||
@ -990,6 +1059,8 @@ snapshots:
|
||||
|
||||
'@oxc-project/types@0.139.0': {}
|
||||
|
||||
'@preact/signals-core@1.14.4': {}
|
||||
|
||||
'@rolldown/binding-android-arm64@1.1.5':
|
||||
optional: true
|
||||
|
||||
@ -1476,8 +1547,7 @@ snapshots:
|
||||
dependencies:
|
||||
punycode: 2.3.1
|
||||
|
||||
tslib@2.8.1:
|
||||
optional: true
|
||||
tslib@2.8.1: {}
|
||||
|
||||
typescript@7.0.2:
|
||||
optionalDependencies:
|
||||
|
||||
File diff suppressed because it is too large
Load Diff
@ -1,48 +1,214 @@
|
||||
import { useEffect, useState } from 'react'
|
||||
import { FormEvent, useEffect, useRef, useState } from 'react'
|
||||
import TaskBoard from './TaskBoard'
|
||||
|
||||
type HealthResponse = {
|
||||
status: string
|
||||
type User = {
|
||||
id: string
|
||||
name: string
|
||||
createdAt: string
|
||||
}
|
||||
|
||||
type ApiError = {
|
||||
code?: string
|
||||
message?: string
|
||||
}
|
||||
|
||||
const ACTIVE_USER_KEY = 'hemhub.activeUserId'
|
||||
|
||||
function App() {
|
||||
const [backendStatus, setBackendStatus] = useState<string | null>(null)
|
||||
const [hasError, setHasError] = useState(false)
|
||||
const [users, setUsers] = useState<User[]>([])
|
||||
const [activeUser, setActiveUser] = useState<User | null>(null)
|
||||
const [loadState, setLoadState] = useState<'loading' | 'ready' | 'error'>('loading')
|
||||
const [showCreateUser, setShowCreateUser] = useState(false)
|
||||
|
||||
const loadUsers = async () => {
|
||||
setLoadState('loading')
|
||||
|
||||
useEffect(() => {
|
||||
const loadHealth = async () => {
|
||||
try {
|
||||
const response = await fetch('/api/health')
|
||||
const response = await fetch('/api/users')
|
||||
|
||||
if (!response.ok) {
|
||||
throw new Error(`Backend svarade med status ${response.status}`)
|
||||
throw new Error('Kunde inte hämta användare')
|
||||
}
|
||||
|
||||
const health = (await response.json()) as HealthResponse
|
||||
setBackendStatus(health.status)
|
||||
const loadedUsers = (await response.json()) as User[]
|
||||
const storedUserId = window.localStorage.getItem(ACTIVE_USER_KEY)
|
||||
const storedUser = loadedUsers.find((user) => user.id === storedUserId) ?? null
|
||||
|
||||
if (storedUserId && !storedUser) {
|
||||
window.localStorage.removeItem(ACTIVE_USER_KEY)
|
||||
}
|
||||
|
||||
setUsers(loadedUsers)
|
||||
setActiveUser(storedUser)
|
||||
setShowCreateUser(loadedUsers.length === 0)
|
||||
setLoadState('ready')
|
||||
} catch {
|
||||
setHasError(true)
|
||||
setLoadState('error')
|
||||
}
|
||||
}
|
||||
|
||||
void loadHealth()
|
||||
useEffect(() => {
|
||||
void loadUsers()
|
||||
}, [])
|
||||
|
||||
let statusMessage = 'Kontrollerar backend…'
|
||||
const selectUser = (user: User) => {
|
||||
window.localStorage.setItem(ACTIVE_USER_KEY, user.id)
|
||||
setActiveUser(user)
|
||||
}
|
||||
|
||||
if (hasError) {
|
||||
statusMessage = 'Backend kunde inte nås'
|
||||
} else if (backendStatus) {
|
||||
statusMessage = `Backend: ${backendStatus}`
|
||||
const logOut = () => {
|
||||
window.localStorage.removeItem(ACTIVE_USER_KEY)
|
||||
setActiveUser(null)
|
||||
setShowCreateUser(false)
|
||||
}
|
||||
|
||||
if (loadState === 'loading') {
|
||||
return <main className="panel">Laddar HemHub…</main>
|
||||
}
|
||||
|
||||
if (loadState === 'error') {
|
||||
return (
|
||||
<main className="panel">
|
||||
<h1>HemHub</h1>
|
||||
<p>Kunde inte ansluta till HemHub.</p>
|
||||
<button type="button" onClick={() => void loadUsers()}>
|
||||
Försök igen
|
||||
</button>
|
||||
</main>
|
||||
)
|
||||
}
|
||||
|
||||
if (activeUser) {
|
||||
return (
|
||||
<TaskBoard
|
||||
activeUserId={activeUser.id}
|
||||
activeUserName={activeUser.name}
|
||||
users={users}
|
||||
onLogOut={logOut}
|
||||
/>
|
||||
)
|
||||
}
|
||||
|
||||
if (showCreateUser) {
|
||||
return (
|
||||
<CreateUserForm
|
||||
hasExistingUsers={users.length > 0}
|
||||
onCancel={() => setShowCreateUser(false)}
|
||||
onCreated={(user) => {
|
||||
setUsers((currentUsers) => [...currentUsers, user])
|
||||
selectUser(user)
|
||||
}}
|
||||
/>
|
||||
)
|
||||
}
|
||||
|
||||
return (
|
||||
<main>
|
||||
<main className="panel">
|
||||
<h1>HemHub</h1>
|
||||
<p>Frontend har startat.</p>
|
||||
<p aria-live="polite">{statusMessage}</p>
|
||||
<h2>Vem är du?</h2>
|
||||
<div className="user-list">
|
||||
{users.map((user) => (
|
||||
<button type="button" key={user.id} onClick={() => selectUser(user)}>
|
||||
{user.name}
|
||||
</button>
|
||||
))}
|
||||
</div>
|
||||
<button type="button" className="secondary" onClick={() => setShowCreateUser(true)}>
|
||||
Skapa användare
|
||||
</button>
|
||||
</main>
|
||||
)
|
||||
}
|
||||
|
||||
type CreateUserFormProps = {
|
||||
hasExistingUsers: boolean
|
||||
onCancel: () => void
|
||||
onCreated: (user: User) => void
|
||||
}
|
||||
|
||||
function CreateUserForm({ hasExistingUsers, onCancel, onCreated }: CreateUserFormProps) {
|
||||
const [name, setName] = useState('')
|
||||
const [error, setError] = useState('')
|
||||
const [isSubmitting, setIsSubmitting] = useState(false)
|
||||
const isSubmittingRef = useRef(false)
|
||||
|
||||
const submit = async (event: FormEvent<HTMLFormElement>) => {
|
||||
event.preventDefault()
|
||||
|
||||
if (isSubmittingRef.current) {
|
||||
return
|
||||
}
|
||||
|
||||
const trimmedName = name.trim()
|
||||
|
||||
if (!trimmedName || [...trimmedName].length > 50) {
|
||||
setError('Namnet måste innehålla mellan 1 och 50 tecken.')
|
||||
return
|
||||
}
|
||||
|
||||
setError('')
|
||||
isSubmittingRef.current = true
|
||||
setIsSubmitting(true)
|
||||
|
||||
try {
|
||||
const response = await fetch('/api/users', {
|
||||
method: 'POST',
|
||||
headers: { 'Content-Type': 'application/json' },
|
||||
body: JSON.stringify({ name: trimmedName }),
|
||||
})
|
||||
|
||||
if (!response.ok) {
|
||||
const apiError = (await response.json().catch(() => ({}))) as ApiError
|
||||
|
||||
if (
|
||||
apiError.code === 'INVALID_USER_NAME' ||
|
||||
apiError.code === 'USER_NAME_ALREADY_EXISTS'
|
||||
) {
|
||||
setError(apiError.message ?? 'Det gick inte att skapa användaren.')
|
||||
} else {
|
||||
setError('Det gick inte att skapa användaren. Försök igen.')
|
||||
}
|
||||
return
|
||||
}
|
||||
|
||||
const createdUser = (await response.json()) as User
|
||||
onCreated(createdUser)
|
||||
} catch {
|
||||
setError('Det gick inte att skapa användaren. Försök igen.')
|
||||
} finally {
|
||||
isSubmittingRef.current = false
|
||||
setIsSubmitting(false)
|
||||
}
|
||||
}
|
||||
|
||||
return (
|
||||
<main className="panel">
|
||||
<h1>HemHub</h1>
|
||||
<h2>Skapa användare</h2>
|
||||
<form onSubmit={(event) => void submit(event)}>
|
||||
<label htmlFor="user-name">Namn</label>
|
||||
<input
|
||||
id="user-name"
|
||||
value={name}
|
||||
disabled={isSubmitting}
|
||||
onChange={(event) => setName(event.target.value)}
|
||||
/>
|
||||
{error && (
|
||||
<p className="error" role="alert">
|
||||
{error}
|
||||
</p>
|
||||
)}
|
||||
<button type="submit" disabled={isSubmitting}>
|
||||
{isSubmitting ? 'Skapar…' : 'Skapa användare'}
|
||||
</button>
|
||||
{hasExistingUsers && (
|
||||
<button type="button" className="secondary" disabled={isSubmitting} onClick={onCancel}>
|
||||
Tillbaka
|
||||
</button>
|
||||
)}
|
||||
</form>
|
||||
</main>
|
||||
)
|
||||
}
|
||||
|
||||
export default App
|
||||
|
||||
|
||||
844
frontend/src/TaskBoard.tsx
Normal file
844
frontend/src/TaskBoard.tsx
Normal file
@ -0,0 +1,844 @@
|
||||
import { FormEvent, MouseEvent, useEffect, useRef, useState } from 'react'
|
||||
import {
|
||||
TaskDragDropProvider,
|
||||
TaskStatus,
|
||||
useTaskColumnDropTarget,
|
||||
useTaskDraggable,
|
||||
} from './TaskDragAndDrop'
|
||||
|
||||
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
|
||||
}
|
||||
|
||||
type ApiError = {
|
||||
code?: string
|
||||
message?: string
|
||||
}
|
||||
|
||||
type TaskBoardProps = {
|
||||
activeUserId: string
|
||||
activeUserName: string
|
||||
users: UserSummary[]
|
||||
onLogOut: () => void
|
||||
}
|
||||
|
||||
const columns: { status: TaskStatus; title: string }[] = [
|
||||
{ status: 'WAITING', title: 'Väntande' },
|
||||
{ status: 'IN_PROGRESS', title: 'Pågående' },
|
||||
{ status: 'COMPLETED', title: 'Klart' },
|
||||
]
|
||||
|
||||
function TaskBoard({ activeUserId, activeUserName, users, onLogOut }: TaskBoardProps) {
|
||||
const [tasks, setTasks] = useState<Task[]>([])
|
||||
const [loadState, setLoadState] = useState<'loading' | 'ready' | 'error'>('loading')
|
||||
const [showCreateTask, setShowCreateTask] = useState(false)
|
||||
const [deletingTask, setDeletingTask] = useState<Task | null>(null)
|
||||
const [deleteError, setDeleteError] = useState('')
|
||||
const [editingAssigneeTaskId, setEditingAssigneeTaskId] = useState<string | null>(null)
|
||||
const [pendingTaskIds, setPendingTaskIds] = useState<Set<string>>(new Set())
|
||||
const [taskErrors, setTaskErrors] = useState<Record<string, string>>({})
|
||||
const pendingTaskIdsRef = useRef(new Set<string>())
|
||||
|
||||
const loadTasks = async () => {
|
||||
setLoadState('loading')
|
||||
|
||||
try {
|
||||
const response = await fetch('/api/tasks')
|
||||
|
||||
if (!response.ok) {
|
||||
throw new Error('Kunde inte hämta uppgifter')
|
||||
}
|
||||
|
||||
setTasks((await response.json()) as Task[])
|
||||
setLoadState('ready')
|
||||
} catch {
|
||||
setLoadState('error')
|
||||
}
|
||||
}
|
||||
|
||||
useEffect(() => {
|
||||
void loadTasks()
|
||||
}, [])
|
||||
|
||||
const beginTaskRequest = (taskId: string) => {
|
||||
if (pendingTaskIdsRef.current.has(taskId)) {
|
||||
return false
|
||||
}
|
||||
|
||||
pendingTaskIdsRef.current.add(taskId)
|
||||
setPendingTaskIds(new Set(pendingTaskIdsRef.current))
|
||||
setTaskErrors((current) => ({ ...current, [taskId]: '' }))
|
||||
return true
|
||||
}
|
||||
|
||||
const finishTaskRequest = (taskId: string) => {
|
||||
pendingTaskIdsRef.current.delete(taskId)
|
||||
setPendingTaskIds(new Set(pendingTaskIdsRef.current))
|
||||
}
|
||||
|
||||
const replaceTask = (updatedTask: Task) => {
|
||||
setTasks((current) =>
|
||||
current.map((task) => (task.id === updatedTask.id ? updatedTask : task)),
|
||||
)
|
||||
}
|
||||
|
||||
const updateAssignee = async (task: Task, assigneeId: string) => {
|
||||
if (!beginTaskRequest(task.id)) {
|
||||
return
|
||||
}
|
||||
|
||||
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
|
||||
setTaskErrors((current) => ({
|
||||
...current,
|
||||
[task.id]: apiError.message ?? 'Det gick inte att ändra ansvarig. Försök igen.',
|
||||
}))
|
||||
return
|
||||
}
|
||||
|
||||
const updatedTask = (await response.json()) as Task
|
||||
replaceTask(updatedTask)
|
||||
setEditingAssigneeTaskId(null)
|
||||
} catch {
|
||||
setTaskErrors((current) => ({
|
||||
...current,
|
||||
[task.id]: 'Det gick inte att ändra ansvarig. Försök igen.',
|
||||
}))
|
||||
} finally {
|
||||
finishTaskRequest(task.id)
|
||||
}
|
||||
}
|
||||
|
||||
const updateStatus = async (
|
||||
task: Task,
|
||||
status: TaskStatus,
|
||||
presentation: 'server-confirmed' | 'optimistic',
|
||||
) => {
|
||||
if (!beginTaskRequest(task.id)) {
|
||||
return
|
||||
}
|
||||
|
||||
const previousTask = task
|
||||
if (presentation === 'optimistic') {
|
||||
replaceTask({
|
||||
...task,
|
||||
status,
|
||||
assignee:
|
||||
status === 'IN_PROGRESS' && !task.assignee
|
||||
? { id: activeUserId, name: activeUserName }
|
||||
: task.assignee,
|
||||
})
|
||||
}
|
||||
|
||||
try {
|
||||
const response = await fetch(`/api/tasks/${task.id}/status`, {
|
||||
method: 'PUT',
|
||||
headers: { 'Content-Type': 'application/json' },
|
||||
body: JSON.stringify({
|
||||
status,
|
||||
...(status === 'IN_PROGRESS' ? { activeUserId } : {}),
|
||||
}),
|
||||
})
|
||||
|
||||
if (!response.ok) {
|
||||
const apiError = (await response.json().catch(() => ({}))) as ApiError
|
||||
if (presentation === 'optimistic') {
|
||||
replaceTask(previousTask)
|
||||
}
|
||||
setTaskErrors((current) => ({
|
||||
...current,
|
||||
[task.id]: apiError.message ?? 'Det gick inte att ändra status. Försök igen.',
|
||||
}))
|
||||
return
|
||||
}
|
||||
|
||||
replaceTask((await response.json()) as Task)
|
||||
setEditingAssigneeTaskId(null)
|
||||
} catch {
|
||||
if (presentation === 'optimistic') {
|
||||
replaceTask(previousTask)
|
||||
}
|
||||
setTaskErrors((current) => ({
|
||||
...current,
|
||||
[task.id]: 'Det gick inte att ändra status. Försök igen.',
|
||||
}))
|
||||
} finally {
|
||||
finishTaskRequest(task.id)
|
||||
}
|
||||
}
|
||||
|
||||
const dropTask = (taskId: string, status: TaskStatus) => {
|
||||
const task = tasks.find((candidate) => candidate.id === taskId)
|
||||
|
||||
if (!task || task.status === status) {
|
||||
return
|
||||
}
|
||||
|
||||
void updateStatus(task, status, 'optimistic')
|
||||
}
|
||||
|
||||
const openDeleteTask = (task: Task) => {
|
||||
if (pendingTaskIdsRef.current.has(task.id)) {
|
||||
return
|
||||
}
|
||||
|
||||
setDeleteError('')
|
||||
setDeletingTask(task)
|
||||
}
|
||||
|
||||
const closeDeleteTask = () => {
|
||||
if (deletingTask && pendingTaskIdsRef.current.has(deletingTask.id)) {
|
||||
return
|
||||
}
|
||||
|
||||
setDeleteError('')
|
||||
setDeletingTask(null)
|
||||
}
|
||||
|
||||
const deleteTask = async (task: Task) => {
|
||||
if (!beginTaskRequest(task.id)) {
|
||||
return
|
||||
}
|
||||
|
||||
setDeleteError('')
|
||||
|
||||
try {
|
||||
const response = await fetch(`/api/tasks/${task.id}`, { method: 'DELETE' })
|
||||
|
||||
if (response.status === 204) {
|
||||
setTasks((current) => current.filter((candidate) => candidate.id !== task.id))
|
||||
setDeletingTask(null)
|
||||
return
|
||||
}
|
||||
|
||||
const apiError = (await response.json().catch(() => ({}))) as ApiError
|
||||
if (response.status === 404 && apiError.code === 'TASK_NOT_FOUND') {
|
||||
setTasks((current) => current.filter((candidate) => candidate.id !== task.id))
|
||||
setDeletingTask(null)
|
||||
return
|
||||
}
|
||||
|
||||
setDeleteError('Det gick inte att radera uppgiften. Försök igen.')
|
||||
} catch {
|
||||
setDeleteError('Det gick inte att radera uppgiften. Försök igen.')
|
||||
} finally {
|
||||
finishTaskRequest(task.id)
|
||||
}
|
||||
}
|
||||
|
||||
return (
|
||||
<main className="task-app">
|
||||
<header className="app-header">
|
||||
<div>
|
||||
<p className="eyebrow">HemHub</p>
|
||||
<h1>Uppgifter</h1>
|
||||
</div>
|
||||
<div className="user-controls">
|
||||
<span>{activeUserName}</span>
|
||||
<button type="button" className="secondary compact" onClick={onLogOut}>
|
||||
Logga ut
|
||||
</button>
|
||||
</div>
|
||||
</header>
|
||||
|
||||
<div className="board-toolbar">
|
||||
<div aria-live="polite">
|
||||
{loadState === 'loading' && 'Laddar uppgifter…'}
|
||||
{loadState === 'error' && (
|
||||
<>
|
||||
Kunde inte hämta uppgifter.
|
||||
<button type="button" className="link-button" onClick={() => void loadTasks()}>
|
||||
Försök igen
|
||||
</button>
|
||||
</>
|
||||
)}
|
||||
</div>
|
||||
<button type="button" onClick={() => setShowCreateTask(true)}>
|
||||
Ny uppgift
|
||||
</button>
|
||||
</div>
|
||||
|
||||
<TaskDragDropProvider onTaskDrop={dropTask}>
|
||||
<section className="board" aria-label="Uppgiftsbräda">
|
||||
{columns.map((column) => (
|
||||
<TaskColumn
|
||||
column={column}
|
||||
tasks={tasks.filter((task) => task.status === column.status)}
|
||||
users={users}
|
||||
editingAssigneeTaskId={editingAssigneeTaskId}
|
||||
pendingTaskIds={pendingTaskIds}
|
||||
taskErrors={taskErrors}
|
||||
onEditAssignee={setEditingAssigneeTaskId}
|
||||
onChangeAssignee={(task, assigneeId) => void updateAssignee(task, assigneeId)}
|
||||
onChangeStatus={(task, status) =>
|
||||
void updateStatus(task, status, 'server-confirmed')
|
||||
}
|
||||
onDelete={openDeleteTask}
|
||||
/>
|
||||
))}
|
||||
</section>
|
||||
</TaskDragDropProvider>
|
||||
|
||||
{showCreateTask && (
|
||||
<CreateTaskModal
|
||||
users={users}
|
||||
onClose={() => setShowCreateTask(false)}
|
||||
onCreated={(task) => {
|
||||
setTasks((currentTasks) => [...currentTasks, task])
|
||||
setShowCreateTask(false)
|
||||
}}
|
||||
/>
|
||||
)}
|
||||
|
||||
{deletingTask && (
|
||||
<DeleteTaskModal
|
||||
task={deletingTask}
|
||||
pending={pendingTaskIds.has(deletingTask.id)}
|
||||
error={deleteError}
|
||||
onClose={closeDeleteTask}
|
||||
onConfirm={() => void deleteTask(deletingTask)}
|
||||
/>
|
||||
)}
|
||||
</main>
|
||||
)
|
||||
}
|
||||
|
||||
type TaskColumnProps = {
|
||||
column: { status: TaskStatus; title: string }
|
||||
tasks: Task[]
|
||||
users: UserSummary[]
|
||||
editingAssigneeTaskId: string | null
|
||||
pendingTaskIds: Set<string>
|
||||
taskErrors: Record<string, string>
|
||||
onEditAssignee: (taskId: string) => void
|
||||
onChangeAssignee: (task: Task, assigneeId: string) => void
|
||||
onChangeStatus: (task: Task, status: TaskStatus) => void
|
||||
onDelete: (task: Task) => void
|
||||
}
|
||||
|
||||
function TaskColumn({
|
||||
column,
|
||||
tasks,
|
||||
users,
|
||||
editingAssigneeTaskId,
|
||||
pendingTaskIds,
|
||||
taskErrors,
|
||||
onEditAssignee,
|
||||
onChangeAssignee,
|
||||
onChangeStatus,
|
||||
onDelete,
|
||||
}: TaskColumnProps) {
|
||||
const { ref, isDropTarget } = useTaskColumnDropTarget(column.status)
|
||||
|
||||
return (
|
||||
<section
|
||||
ref={ref}
|
||||
className={`board-column${isDropTarget ? ' board-column-drop-target' : ''}`}
|
||||
aria-labelledby={column.status}
|
||||
>
|
||||
<h2 id={column.status}>{column.title}</h2>
|
||||
<div className="task-list">
|
||||
{tasks.map((task) => (
|
||||
<TaskCard
|
||||
key={task.id}
|
||||
task={task}
|
||||
users={users}
|
||||
editingAssignee={editingAssigneeTaskId === task.id}
|
||||
pending={pendingTaskIds.has(task.id)}
|
||||
error={taskErrors[task.id]}
|
||||
onEditAssignee={() => onEditAssignee(task.id)}
|
||||
onChangeAssignee={(assigneeId) => onChangeAssignee(task, assigneeId)}
|
||||
onChangeStatus={(status) => onChangeStatus(task, status)}
|
||||
onDelete={() => onDelete(task)}
|
||||
/>
|
||||
))}
|
||||
</div>
|
||||
</section>
|
||||
)
|
||||
}
|
||||
|
||||
type TaskCardProps = {
|
||||
task: Task
|
||||
users: UserSummary[]
|
||||
editingAssignee: boolean
|
||||
pending: boolean
|
||||
error?: string
|
||||
onEditAssignee: () => void
|
||||
onChangeAssignee: (assigneeId: string) => void
|
||||
onChangeStatus: (status: TaskStatus) => void
|
||||
onDelete: () => void
|
||||
}
|
||||
|
||||
function TaskCard({
|
||||
task,
|
||||
users,
|
||||
editingAssignee,
|
||||
pending,
|
||||
error,
|
||||
onEditAssignee,
|
||||
onChangeAssignee,
|
||||
onChangeStatus,
|
||||
onDelete,
|
||||
}: TaskCardProps) {
|
||||
const { ref, isDragging } = useTaskDraggable(task.id, pending)
|
||||
|
||||
return (
|
||||
<article
|
||||
ref={ref}
|
||||
role="article"
|
||||
className={`task-card${pending ? ' task-card-pending' : ''}${
|
||||
isDragging ? ' task-card-dragging' : ''
|
||||
}`}
|
||||
aria-busy={pending || undefined}
|
||||
>
|
||||
<div className="task-card-header">
|
||||
<h3>{task.title}</h3>
|
||||
<div className="task-card-actions">
|
||||
<span className="points-badge">{task.points} p</span>
|
||||
<button
|
||||
type="button"
|
||||
className="task-delete-button"
|
||||
aria-label={`Radera ${task.title}`}
|
||||
disabled={pending}
|
||||
onPointerDown={(event) => event.stopPropagation()}
|
||||
onClick={onDelete}
|
||||
>
|
||||
<TrashIcon />
|
||||
</button>
|
||||
</div>
|
||||
</div>
|
||||
{task.description && <p>{task.description}</p>}
|
||||
<AssigneeControl
|
||||
task={task}
|
||||
users={users}
|
||||
editing={editingAssignee}
|
||||
pending={pending}
|
||||
onEdit={onEditAssignee}
|
||||
onChange={onChangeAssignee}
|
||||
/>
|
||||
<TaskStatusControls task={task} disabled={pending} onChange={onChangeStatus} />
|
||||
{error && (
|
||||
<p className="task-error error" role="alert">
|
||||
{error}
|
||||
</p>
|
||||
)}
|
||||
</article>
|
||||
)
|
||||
}
|
||||
|
||||
function TrashIcon() {
|
||||
return (
|
||||
<svg
|
||||
viewBox="0 0 24 24"
|
||||
width="19"
|
||||
height="19"
|
||||
aria-hidden="true"
|
||||
focusable="false"
|
||||
>
|
||||
<path
|
||||
d="M4 7h16M9 7V4h6v3m-8 0 1 13h8l1-13M10 11v5m4-5v5"
|
||||
fill="none"
|
||||
stroke="currentColor"
|
||||
strokeWidth="1.8"
|
||||
strokeLinecap="round"
|
||||
strokeLinejoin="round"
|
||||
/>
|
||||
</svg>
|
||||
)
|
||||
}
|
||||
|
||||
type DeleteTaskModalProps = {
|
||||
task: Task
|
||||
pending: boolean
|
||||
error: string
|
||||
onClose: () => void
|
||||
onConfirm: () => void
|
||||
}
|
||||
|
||||
function DeleteTaskModal({
|
||||
task,
|
||||
pending,
|
||||
error,
|
||||
onClose,
|
||||
onConfirm,
|
||||
}: DeleteTaskModalProps) {
|
||||
useEffect(() => {
|
||||
const closeOnEscape = (event: KeyboardEvent) => {
|
||||
if (event.key === 'Escape' && !pending) {
|
||||
onClose()
|
||||
}
|
||||
}
|
||||
|
||||
window.addEventListener('keydown', closeOnEscape)
|
||||
return () => window.removeEventListener('keydown', closeOnEscape)
|
||||
}, [onClose, pending])
|
||||
|
||||
const closeFromBackdrop = (event: MouseEvent<HTMLDivElement>) => {
|
||||
if (event.target === event.currentTarget && !pending) {
|
||||
onClose()
|
||||
}
|
||||
}
|
||||
|
||||
return (
|
||||
<div className="modal-backdrop" onMouseDown={closeFromBackdrop}>
|
||||
<section
|
||||
className="modal delete-task-modal"
|
||||
role="dialog"
|
||||
aria-modal="true"
|
||||
aria-labelledby="delete-task-title"
|
||||
>
|
||||
<div className="modal-header">
|
||||
<h2 id="delete-task-title">Radera uppgift?</h2>
|
||||
</div>
|
||||
<p>
|
||||
Är du säker på att du vill radera <strong>{task.title}</strong>? Uppgiften
|
||||
raderas permanent och kan inte återställas.
|
||||
</p>
|
||||
{error && (
|
||||
<p className="error" role="alert">
|
||||
{error}
|
||||
</p>
|
||||
)}
|
||||
<div className="delete-task-actions">
|
||||
<button
|
||||
type="button"
|
||||
className="secondary compact"
|
||||
autoFocus
|
||||
disabled={pending}
|
||||
onClick={onClose}
|
||||
>
|
||||
Avbryt
|
||||
</button>
|
||||
<button
|
||||
type="button"
|
||||
className="danger"
|
||||
disabled={pending}
|
||||
onClick={onConfirm}
|
||||
>
|
||||
Radera
|
||||
</button>
|
||||
</div>
|
||||
</section>
|
||||
</div>
|
||||
)
|
||||
}
|
||||
|
||||
type AssigneeControlProps = {
|
||||
task: Task
|
||||
users: UserSummary[]
|
||||
editing: boolean
|
||||
pending: boolean
|
||||
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,
|
||||
pending,
|
||||
onEdit,
|
||||
onChange,
|
||||
}: AssigneeControlProps) {
|
||||
const displayName =
|
||||
task.assignee?.name ?? (task.status === 'WAITING' ? 'Ta uppgift' : 'Otilldelad')
|
||||
|
||||
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={pending}
|
||||
autoFocus
|
||||
onChange={(event) => onChange(event.target.value)}
|
||||
>
|
||||
{task.status !== 'IN_PROGRESS' && <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={pending}
|
||||
onClick={onEdit}
|
||||
aria-label={`Ändra ansvarig för ${task.title}`}
|
||||
>
|
||||
<UserIcon />
|
||||
<span>{displayName}</span>
|
||||
</button>
|
||||
)}
|
||||
</div>
|
||||
)
|
||||
}
|
||||
|
||||
type TaskStatusControlsProps = {
|
||||
task: Task
|
||||
disabled: boolean
|
||||
onChange: (status: TaskStatus) => void
|
||||
}
|
||||
|
||||
const statusActions: Record<TaskStatus, { label: string; target: TaskStatus }[]> = {
|
||||
WAITING: [
|
||||
{ label: 'Påbörja', target: 'IN_PROGRESS' },
|
||||
{ label: 'Markera klar', target: 'COMPLETED' },
|
||||
],
|
||||
IN_PROGRESS: [
|
||||
{ label: 'Till Väntande', target: 'WAITING' },
|
||||
{ label: 'Markera klar', target: 'COMPLETED' },
|
||||
],
|
||||
COMPLETED: [
|
||||
{ label: 'Till Väntande', target: 'WAITING' },
|
||||
{ label: 'Påbörja igen', target: 'IN_PROGRESS' },
|
||||
],
|
||||
}
|
||||
|
||||
function TaskStatusControls({ task, disabled, onChange }: TaskStatusControlsProps) {
|
||||
return (
|
||||
<div className="task-status-actions" aria-label={`Ändra status för ${task.title}`}>
|
||||
{statusActions[task.status].map((action) => (
|
||||
<button
|
||||
type="button"
|
||||
className="status-button"
|
||||
key={action.target}
|
||||
disabled={disabled}
|
||||
onClick={() => onChange(action.target)}
|
||||
>
|
||||
{action.label}
|
||||
</button>
|
||||
))}
|
||||
</div>
|
||||
)
|
||||
}
|
||||
|
||||
type CreateTaskModalProps = {
|
||||
users: UserSummary[]
|
||||
onClose: () => void
|
||||
onCreated: (task: Task) => void
|
||||
}
|
||||
|
||||
function CreateTaskModal({ users, 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)
|
||||
|
||||
useEffect(() => {
|
||||
const closeOnEscape = (event: KeyboardEvent) => {
|
||||
if (event.key === 'Escape' && !isSubmittingRef.current) {
|
||||
onClose()
|
||||
}
|
||||
}
|
||||
|
||||
window.addEventListener('keydown', closeOnEscape)
|
||||
return () => window.removeEventListener('keydown', closeOnEscape)
|
||||
}, [onClose])
|
||||
|
||||
const closeFromBackdrop = (event: MouseEvent<HTMLDivElement>) => {
|
||||
if (event.target === event.currentTarget && !isSubmittingRef.current) {
|
||||
onClose()
|
||||
}
|
||||
}
|
||||
|
||||
const submit = async (event: FormEvent<HTMLFormElement>) => {
|
||||
event.preventDefault()
|
||||
|
||||
if (isSubmittingRef.current) {
|
||||
return
|
||||
}
|
||||
|
||||
const trimmedTitle = title.trim()
|
||||
const trimmedDescription = description.trim()
|
||||
const numericPoints = Number(points)
|
||||
|
||||
if (!trimmedTitle || [...trimmedTitle].length > 100) {
|
||||
setError('Titeln måste innehålla mellan 1 och 100 tecken.')
|
||||
return
|
||||
}
|
||||
|
||||
if ([...trimmedDescription].length > 500) {
|
||||
setError('Beskrivningen får innehålla högst 500 tecken.')
|
||||
return
|
||||
}
|
||||
|
||||
if (
|
||||
!points.trim() ||
|
||||
!Number.isInteger(numericPoints) ||
|
||||
numericPoints < 1 ||
|
||||
numericPoints > 99
|
||||
) {
|
||||
setError('Poäng måste vara ett heltal mellan 1 och 99.')
|
||||
return
|
||||
}
|
||||
|
||||
setError('')
|
||||
isSubmittingRef.current = true
|
||||
setIsSubmitting(true)
|
||||
|
||||
try {
|
||||
const response = await fetch('/api/tasks', {
|
||||
method: 'POST',
|
||||
headers: { 'Content-Type': 'application/json' },
|
||||
body: JSON.stringify({
|
||||
title: trimmedTitle,
|
||||
description: trimmedDescription || null,
|
||||
points: numericPoints,
|
||||
assigneeId: assigneeId || null,
|
||||
}),
|
||||
})
|
||||
|
||||
if (!response.ok) {
|
||||
const apiError = (await response.json().catch(() => ({}))) as ApiError
|
||||
setError(apiError.message ?? 'Det gick inte att skapa uppgiften. Försök igen.')
|
||||
return
|
||||
}
|
||||
|
||||
onCreated((await response.json()) as Task)
|
||||
} catch {
|
||||
setError('Det gick inte att skapa uppgiften. Försök igen.')
|
||||
} finally {
|
||||
isSubmittingRef.current = false
|
||||
setIsSubmitting(false)
|
||||
}
|
||||
}
|
||||
|
||||
return (
|
||||
<div className="modal-backdrop" onMouseDown={closeFromBackdrop}>
|
||||
<section
|
||||
className="modal"
|
||||
role="dialog"
|
||||
aria-modal="true"
|
||||
aria-labelledby="create-task-title"
|
||||
>
|
||||
<div className="modal-header">
|
||||
<h2 id="create-task-title">Skapa ny uppgift</h2>
|
||||
<button
|
||||
type="button"
|
||||
className="close-button"
|
||||
aria-label="Stäng"
|
||||
disabled={isSubmitting}
|
||||
onClick={onClose}
|
||||
>
|
||||
×
|
||||
</button>
|
||||
</div>
|
||||
<form noValidate onSubmit={(event) => void submit(event)}>
|
||||
<label htmlFor="task-title">Titel</label>
|
||||
<input
|
||||
id="task-title"
|
||||
autoFocus
|
||||
value={title}
|
||||
disabled={isSubmitting}
|
||||
onChange={(event) => setTitle(event.target.value)}
|
||||
/>
|
||||
|
||||
<label htmlFor="task-description">Beskrivning (valfri)</label>
|
||||
<textarea
|
||||
id="task-description"
|
||||
rows={5}
|
||||
value={description}
|
||||
disabled={isSubmitting}
|
||||
onChange={(event) => setDescription(event.target.value)}
|
||||
/>
|
||||
|
||||
<label htmlFor="task-points">Poäng</label>
|
||||
<input
|
||||
id="task-points"
|
||||
type="number"
|
||||
min="1"
|
||||
max="99"
|
||||
step="1"
|
||||
value={points}
|
||||
disabled={isSubmitting}
|
||||
aria-describedby="task-points-help"
|
||||
onChange={(event) => setPoints(event.target.value)}
|
||||
/>
|
||||
<p id="task-points-help" className="field-help">
|
||||
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}
|
||||
</p>
|
||||
)}
|
||||
|
||||
<button type="submit" disabled={isSubmitting}>
|
||||
{isSubmitting ? 'Skapar…' : 'Skapa uppgift'}
|
||||
</button>
|
||||
</form>
|
||||
</section>
|
||||
</div>
|
||||
)
|
||||
}
|
||||
|
||||
export default TaskBoard
|
||||
18
frontend/src/TaskDragAndDrop.test.ts
Normal file
18
frontend/src/TaskDragAndDrop.test.ts
Normal file
@ -0,0 +1,18 @@
|
||||
import { expect, test } from 'vitest'
|
||||
import { resolveTaskDrop } from './TaskDragAndDrop'
|
||||
|
||||
test.each(['WAITING', 'IN_PROGRESS', 'COMPLETED'] as const)(
|
||||
'mappar målkolumnen %s till motsvarande status',
|
||||
(status) => {
|
||||
expect(resolveTaskDrop('task-1', status, false)).toEqual({
|
||||
taskId: 'task-1',
|
||||
targetStatus: status,
|
||||
})
|
||||
},
|
||||
)
|
||||
|
||||
test('avbruten dragning och ogiltig målkolumn är no-op', () => {
|
||||
expect(resolveTaskDrop('task-1', 'WAITING', true)).toBeNull()
|
||||
expect(resolveTaskDrop('task-1', undefined, false)).toBeNull()
|
||||
expect(resolveTaskDrop('task-1', 'UNKNOWN', false)).toBeNull()
|
||||
})
|
||||
90
frontend/src/TaskDragAndDrop.tsx
Normal file
90
frontend/src/TaskDragAndDrop.tsx
Normal file
@ -0,0 +1,90 @@
|
||||
import { ReactNode } from 'react'
|
||||
import { DragDropProvider, useDraggable, useDroppable } from '@dnd-kit/react'
|
||||
import { PointerActivationConstraints, PointerSensor } from '@dnd-kit/dom'
|
||||
|
||||
export type TaskStatus = 'WAITING' | 'IN_PROGRESS' | 'COMPLETED'
|
||||
|
||||
type TaskDragDropProviderProps = {
|
||||
children: ReactNode
|
||||
onTaskDrop: (taskId: string, targetStatus: TaskStatus) => void
|
||||
}
|
||||
|
||||
const taskStatuses = new Set<TaskStatus>(['WAITING', 'IN_PROGRESS', 'COMPLETED'])
|
||||
|
||||
const pointerSensor = PointerSensor.configure({
|
||||
activationConstraints(event) {
|
||||
if (event.pointerType === 'touch') {
|
||||
return [new PointerActivationConstraints.Delay({ value: 250, tolerance: 8 })]
|
||||
}
|
||||
|
||||
return [new PointerActivationConstraints.Distance({ value: 6 })]
|
||||
},
|
||||
})
|
||||
|
||||
export function resolveTaskDrop(
|
||||
sourceId: string | number | undefined,
|
||||
targetId: string | number | undefined,
|
||||
canceled: boolean,
|
||||
) {
|
||||
if (
|
||||
canceled ||
|
||||
sourceId === undefined ||
|
||||
typeof targetId !== 'string' ||
|
||||
!taskStatuses.has(targetId as TaskStatus)
|
||||
) {
|
||||
return null
|
||||
}
|
||||
|
||||
return {
|
||||
taskId: String(sourceId),
|
||||
targetStatus: targetId as TaskStatus,
|
||||
}
|
||||
}
|
||||
|
||||
export function TaskDragDropProvider({
|
||||
children,
|
||||
onTaskDrop,
|
||||
}: TaskDragDropProviderProps) {
|
||||
return (
|
||||
<DragDropProvider
|
||||
sensors={(defaults) => [
|
||||
...defaults.filter((sensor) => sensor !== PointerSensor),
|
||||
pointerSensor,
|
||||
]}
|
||||
onDragEnd={(event) => {
|
||||
const drop = resolveTaskDrop(
|
||||
event.operation.source?.id,
|
||||
event.operation.target?.id,
|
||||
event.canceled,
|
||||
)
|
||||
|
||||
if (!drop) {
|
||||
return
|
||||
}
|
||||
|
||||
onTaskDrop(drop.taskId, drop.targetStatus)
|
||||
}}
|
||||
>
|
||||
{children}
|
||||
</DragDropProvider>
|
||||
)
|
||||
}
|
||||
|
||||
export function useTaskDraggable(taskId: string, disabled: boolean) {
|
||||
const { ref, isDragging } = useDraggable({
|
||||
id: taskId,
|
||||
type: 'task',
|
||||
disabled,
|
||||
})
|
||||
|
||||
return { ref, isDragging }
|
||||
}
|
||||
|
||||
export function useTaskColumnDropTarget(status: TaskStatus) {
|
||||
const { ref, isDropTarget } = useDroppable({
|
||||
id: status,
|
||||
accept: 'task',
|
||||
})
|
||||
|
||||
return { ref, isDropTarget }
|
||||
}
|
||||
@ -8,7 +8,7 @@ body {
|
||||
margin: 0;
|
||||
}
|
||||
|
||||
main {
|
||||
.panel {
|
||||
max-width: 40rem;
|
||||
margin: 6rem auto;
|
||||
padding: 2rem;
|
||||
@ -21,3 +21,390 @@ h1 {
|
||||
margin-top: 0;
|
||||
}
|
||||
|
||||
button,
|
||||
input,
|
||||
select,
|
||||
textarea {
|
||||
font: inherit;
|
||||
}
|
||||
|
||||
button {
|
||||
padding: 0.65rem 1rem;
|
||||
border: 0;
|
||||
border-radius: 0.4rem;
|
||||
color: white;
|
||||
background: #2563eb;
|
||||
cursor: pointer;
|
||||
}
|
||||
|
||||
button:disabled,
|
||||
input:disabled,
|
||||
select:disabled,
|
||||
textarea:disabled {
|
||||
cursor: not-allowed;
|
||||
opacity: 0.65;
|
||||
}
|
||||
|
||||
.secondary {
|
||||
margin-top: 1rem;
|
||||
color: #1f2937;
|
||||
background: #e5e7eb;
|
||||
}
|
||||
|
||||
.user-list {
|
||||
display: flex;
|
||||
flex-wrap: wrap;
|
||||
gap: 0.75rem;
|
||||
}
|
||||
|
||||
form {
|
||||
display: flex;
|
||||
flex-direction: column;
|
||||
align-items: flex-start;
|
||||
gap: 0.75rem;
|
||||
}
|
||||
|
||||
input {
|
||||
box-sizing: border-box;
|
||||
width: 100%;
|
||||
padding: 0.6rem;
|
||||
border: 1px solid #9ca3af;
|
||||
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%;
|
||||
padding: 0.6rem;
|
||||
border: 1px solid #9ca3af;
|
||||
border-radius: 0.4rem;
|
||||
resize: vertical;
|
||||
}
|
||||
|
||||
.error {
|
||||
margin: 0;
|
||||
color: #b91c1c;
|
||||
}
|
||||
|
||||
.task-app {
|
||||
min-height: 100vh;
|
||||
padding: 2rem clamp(1rem, 4vw, 4rem);
|
||||
background: #f3f4f6;
|
||||
}
|
||||
|
||||
.app-header,
|
||||
.board-toolbar,
|
||||
.user-controls,
|
||||
.modal-header {
|
||||
display: flex;
|
||||
align-items: center;
|
||||
justify-content: space-between;
|
||||
gap: 1rem;
|
||||
}
|
||||
|
||||
.app-header {
|
||||
margin: 0 auto 2rem;
|
||||
max-width: 90rem;
|
||||
}
|
||||
|
||||
.app-header h1,
|
||||
.eyebrow {
|
||||
margin: 0;
|
||||
}
|
||||
|
||||
.eyebrow {
|
||||
color: #64748b;
|
||||
font-weight: 600;
|
||||
}
|
||||
|
||||
.user-controls {
|
||||
justify-content: flex-end;
|
||||
}
|
||||
|
||||
.compact {
|
||||
margin-top: 0;
|
||||
}
|
||||
|
||||
.board-toolbar {
|
||||
min-height: 2.75rem;
|
||||
margin: 0 auto 1rem;
|
||||
max-width: 90rem;
|
||||
}
|
||||
|
||||
.link-button {
|
||||
padding: 0.25rem 0.5rem;
|
||||
color: #2563eb;
|
||||
background: transparent;
|
||||
text-decoration: underline;
|
||||
}
|
||||
|
||||
.board {
|
||||
display: grid;
|
||||
grid-template-columns: repeat(3, minmax(0, 1fr));
|
||||
gap: 1rem;
|
||||
margin: 0 auto;
|
||||
max-width: 90rem;
|
||||
}
|
||||
|
||||
.board-column {
|
||||
min-height: 20rem;
|
||||
padding: 1rem;
|
||||
border: 1px solid transparent;
|
||||
border-radius: 0.75rem;
|
||||
background: #e5e7eb;
|
||||
transition: border-color 120ms ease, background-color 120ms ease;
|
||||
}
|
||||
|
||||
.board-column-drop-target {
|
||||
border-color: #93c5fd;
|
||||
background: #e0e7ff;
|
||||
}
|
||||
|
||||
.board-column h2 {
|
||||
margin: 0 0 1rem;
|
||||
font-size: 1.1rem;
|
||||
}
|
||||
|
||||
.task-list {
|
||||
display: flex;
|
||||
flex-direction: column;
|
||||
gap: 0.75rem;
|
||||
}
|
||||
|
||||
.task-card {
|
||||
padding: 1rem;
|
||||
border-radius: 0.5rem;
|
||||
background: white;
|
||||
box-shadow: 0 0.125rem 0.4rem rgb(0 0 0 / 8%);
|
||||
}
|
||||
|
||||
.task-card-pending {
|
||||
opacity: 0.65;
|
||||
}
|
||||
|
||||
.task-card-dragging {
|
||||
cursor: grabbing;
|
||||
}
|
||||
|
||||
.task-card h3,
|
||||
.task-card p {
|
||||
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;
|
||||
}
|
||||
|
||||
.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;
|
||||
}
|
||||
|
||||
.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;
|
||||
justify-content: space-between;
|
||||
gap: 0.75rem;
|
||||
}
|
||||
|
||||
.task-card-actions {
|
||||
display: flex;
|
||||
flex: 0 0 auto;
|
||||
align-items: center;
|
||||
gap: 0.35rem;
|
||||
}
|
||||
|
||||
.task-delete-button {
|
||||
display: inline-grid;
|
||||
width: 2.5rem;
|
||||
height: 2.5rem;
|
||||
padding: 0;
|
||||
place-items: center;
|
||||
color: #64748b;
|
||||
background: transparent;
|
||||
}
|
||||
|
||||
.task-delete-button:hover,
|
||||
.task-delete-button:focus-visible {
|
||||
color: #991b1b;
|
||||
background: #fee2e2;
|
||||
}
|
||||
|
||||
.task-delete-button:focus-visible {
|
||||
outline: 2px solid #dc2626;
|
||||
outline-offset: 2px;
|
||||
}
|
||||
|
||||
.points-badge {
|
||||
flex: 0 0 auto;
|
||||
padding: 0.2rem 0.5rem;
|
||||
border-radius: 999px;
|
||||
color: #1e3a8a;
|
||||
background: #dbeafe;
|
||||
font-size: 0.8rem;
|
||||
font-weight: 700;
|
||||
line-height: 1.25;
|
||||
}
|
||||
|
||||
.task-card p {
|
||||
margin-top: 0.5rem;
|
||||
color: #475569;
|
||||
white-space: pre-wrap;
|
||||
}
|
||||
|
||||
.task-status-actions {
|
||||
display: flex;
|
||||
flex-wrap: wrap;
|
||||
gap: 0.5rem;
|
||||
margin-top: 0.9rem;
|
||||
}
|
||||
|
||||
.status-button {
|
||||
padding: 0.4rem 0.65rem;
|
||||
color: #1e3a8a;
|
||||
background: #dbeafe;
|
||||
font-size: 0.85rem;
|
||||
}
|
||||
|
||||
.task-error {
|
||||
margin-top: 0.6rem;
|
||||
font-size: 0.85rem;
|
||||
}
|
||||
|
||||
.field-help {
|
||||
margin: -0.25rem 0 0;
|
||||
color: #64748b;
|
||||
font-size: 0.85rem;
|
||||
}
|
||||
|
||||
.modal-backdrop {
|
||||
position: fixed;
|
||||
inset: 0;
|
||||
display: grid;
|
||||
place-items: center;
|
||||
padding: 1rem;
|
||||
background: rgb(15 23 42 / 55%);
|
||||
}
|
||||
|
||||
.modal {
|
||||
width: min(100%, 34rem);
|
||||
padding: 1.5rem;
|
||||
border-radius: 0.75rem;
|
||||
background: white;
|
||||
box-shadow: 0 1rem 3rem rgb(0 0 0 / 25%);
|
||||
}
|
||||
|
||||
.modal-header {
|
||||
margin-bottom: 1.25rem;
|
||||
}
|
||||
|
||||
.modal-header h2 {
|
||||
margin: 0;
|
||||
}
|
||||
|
||||
.delete-task-modal p {
|
||||
margin: 0 0 1rem;
|
||||
}
|
||||
|
||||
.delete-task-actions {
|
||||
display: flex;
|
||||
justify-content: flex-end;
|
||||
gap: 0.75rem;
|
||||
}
|
||||
|
||||
.delete-task-actions .secondary {
|
||||
margin-top: 0;
|
||||
}
|
||||
|
||||
.danger {
|
||||
background: #b91c1c;
|
||||
}
|
||||
|
||||
.danger:hover,
|
||||
.danger:focus-visible {
|
||||
background: #991b1b;
|
||||
}
|
||||
|
||||
.close-button {
|
||||
padding: 0.2rem 0.55rem;
|
||||
color: #475569;
|
||||
background: transparent;
|
||||
font-size: 1.75rem;
|
||||
line-height: 1;
|
||||
}
|
||||
|
||||
@media (max-width: 48rem) {
|
||||
.panel {
|
||||
margin: 2rem 1rem;
|
||||
}
|
||||
|
||||
.app-header {
|
||||
align-items: flex-start;
|
||||
}
|
||||
|
||||
.user-controls {
|
||||
flex-direction: column;
|
||||
align-items: flex-end;
|
||||
}
|
||||
|
||||
.board {
|
||||
grid-template-columns: 1fr;
|
||||
}
|
||||
|
||||
.board-column {
|
||||
min-height: 8rem;
|
||||
}
|
||||
}
|
||||
|
||||
@ -1,2 +1,37 @@
|
||||
import '@testing-library/jest-dom/vitest'
|
||||
|
||||
class ResizeObserverStub implements ResizeObserver {
|
||||
observe() {}
|
||||
|
||||
unobserve() {}
|
||||
|
||||
disconnect() {}
|
||||
}
|
||||
|
||||
globalThis.ResizeObserver = ResizeObserverStub
|
||||
|
||||
const storedValues = new Map<string, string>()
|
||||
|
||||
Object.defineProperty(window, 'localStorage', {
|
||||
configurable: true,
|
||||
value: {
|
||||
get length() {
|
||||
return storedValues.size
|
||||
},
|
||||
clear() {
|
||||
storedValues.clear()
|
||||
},
|
||||
getItem(key: string) {
|
||||
return storedValues.get(key) ?? null
|
||||
},
|
||||
key(index: number) {
|
||||
return [...storedValues.keys()][index] ?? null
|
||||
},
|
||||
removeItem(key: string) {
|
||||
storedValues.delete(key)
|
||||
},
|
||||
setItem(key: string, value: string) {
|
||||
storedValues.set(key, String(value))
|
||||
},
|
||||
} satisfies Storage,
|
||||
})
|
||||
|
||||
@ -11,6 +11,11 @@ export default defineConfig({
|
||||
},
|
||||
test: {
|
||||
environment: 'jsdom',
|
||||
environmentOptions: {
|
||||
jsdom: {
|
||||
url: 'http://localhost',
|
||||
},
|
||||
},
|
||||
setupFiles: './src/test/setup.ts',
|
||||
},
|
||||
})
|
||||
|
||||
Reference in New Issue
Block a user