ClockGU / ClockGU/clock-backend

Shifts/Reports: `locked` field migration

Open
#385 0 comments 0 reactions 1 assignee Claimed by @zakriya057 View on GitHub
enhancement low-priority
Dominant language
Python
Stars
0
Forks
4
PR merge metrics
No merged PRs in 30d

Description

So far the `locked` field signals whether a month is already locked and the report is archieved in the time vault.

However i have the impression that we should change that so that

- [ ] Migrate Field `locked` from `Shift` to `Report` (Includes data migration *sigh*)

- [ ] Refactor the validation of `ShiftSerializer` to check whether the months `Report` instance is already locked

- [ ] Refactor `lock_shifts()` Endpoint of `ContractViewSet` to set `locked` status on Report rather than shifts

- [ ] Refactor all Querries that access `locked` field of Shifts to now access `locked` of Report

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.