ecamp / ecamp/ecamp3

Picasso: Reload all schedule entries has no dirty handling

Open
#2,457 2 comments 0 reactions 0 assignees View on GitHub
agents:ignore
Dominant language
PHP
Stars
156
Forks
72
Avg merge
12h 43m
Merged PRs (30d)
203

Description

Currently, the schedule entries have wrong numbers after dragging them around. When dragging around a schedule entry on the picasso (or changing the date / time in a form), we need to reload all schedule entries on both affected days: The day that the schedule entry was before, and the day the schedule entry is now.

Also, the `` component has a local copy of the schedule entries and currently only updates it based on changes to the whole collection of schedule entries. So even when the vuex store receives an update to at least the dragged schedule entry, the `` component never tells the picasso component about that change.

So probably a single reload of `period().scheduleEntries()` after dragging a schedule entry would solve this issue completely. The question is, can we do better?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.