Restore events link on camp #375
Conversation
Also remove git from the frontend container, because we now have no dependencies from GitHub anymore again.
| eventInstances () { | ||
| return this.periods.items.flatMap(period => period.event_instances()) | ||
| events () { | ||
| return this.camp().events() |
There was a problem hiding this comment.
Nur im v-if="events._meta.loading" auf dem skeleton loader. Aber der Nebeneffekt davon ist, dass der events Endpoint abgerufen wird. In jedem Event in dieser Liste sind die EventInstances embedded, darum werden so auf einen Schlag (mit einem Request) alle EventInstances des Camps im Store verfügbar.
Erst wenn das gemacht ist wird der Picasso dann angezeigt. Der period.event_instances().items lädt zwar noch vorher die Liste mit Links zu EventInstances, aber die Inhalte sind dann alle schon im Store und werden nicht einzeln gefetcht.
Könnte man in Zukunft vielleicht noch verbessern indem man nicht einmal auf die Periods wartet, sondern skeleton-loaders anzeigt während this.periods._meta.loading || this.events._meta.loading.
| @@ -1,5 +1,5 @@ | |||
| <template> | |||
| <v-app style="background: #90A4AE"> | |||
| <v-app style="background: #F4F4F4"> | |||
There was a problem hiding this comment.
Finde ich gut die Material-Farben zu verwenden. Diese Schattierung scheint mir aber immer noch etwas schwer (dunkel). Ich habe das ursprünglich geändert weil mir Leute denen ich die aktuelle Version gezeigt habe das bemängelt haben, und kann es auch selber sehr gut nachvollziehen. Aber kann man ja in Zukunft noch gut dran schrauben.
Fixes #359 and basic frontend improvements