From 2c86cd02690a24d8c123ccf07724e4feebd4cac9 Mon Sep 17 00:00:00 2001
From: Adrian Hopek <adrian.hopek@blumilk.pl>
Date: Wed, 20 Apr 2022 11:17:21 +0200
Subject: [PATCH] #120 - wip

---
 app/Domain/Actions/VacationRequest/ApproveAction.php | 8 ++++++--
 resources/js/Pages/Users/Create.vue                  | 4 ++--
 resources/js/Pages/VacationRequest/Create.vue        | 4 ++--
 3 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/app/Domain/Actions/VacationRequest/ApproveAction.php b/app/Domain/Actions/VacationRequest/ApproveAction.php
index 3d55542..a6d5896 100644
--- a/app/Domain/Actions/VacationRequest/ApproveAction.php
+++ b/app/Domain/Actions/VacationRequest/ApproveAction.php
@@ -7,6 +7,7 @@ namespace Toby\Domain\Actions\VacationRequest;
 use Toby\Domain\Enums\Role;
 use Toby\Domain\Notifications\VacationRequestStatusChangedNotification;
 use Toby\Domain\VacationRequestStateManager;
+use Toby\Domain\VacationTypeConfigRetriever;
 use Toby\Eloquent\Models\User;
 use Toby\Eloquent\Models\VacationRequest;
 use Toby\Infrastructure\Jobs\SendVacationRequestDaysToGoogleCalendar;
@@ -15,15 +16,18 @@ class ApproveAction
 {
     public function __construct(
         protected VacationRequestStateManager $stateManager,
+        protected VacationTypeConfigRetriever $configRetriever,
     ) {}
 
     public function execute(VacationRequest $vacationRequest, ?User $user = null): void
     {
         $this->stateManager->approve($vacationRequest, $user);
 
-        SendVacationRequestDaysToGoogleCalendar::dispatch($vacationRequest);
+        if ($this->configRetriever->isVacation($vacationRequest->type)) {
+            SendVacationRequestDaysToGoogleCalendar::dispatch($vacationRequest);
 
-        $this->notify($vacationRequest);
+            $this->notify($vacationRequest);
+        }
     }
 
     protected function notify(VacationRequest $vacationRequest): void
diff --git a/resources/js/Pages/Users/Create.vue b/resources/js/Pages/Users/Create.vue
index d56ca33..8962bda 100644
--- a/resources/js/Pages/Users/Create.vue
+++ b/resources/js/Pages/Users/Create.vue
@@ -133,7 +133,7 @@
                 as="template"
                 :value="role"
               >
-                <li :class="[active ? 'text-white bg-blumilk-600' : 'text-gray-900', 'cursor-default select-none relative py-2 pl-3 pr-9']">
+                <li :class="[active ? 'bg-gray-100' : 'text-gray-900', 'cursor-default select-none relative py-2 pl-3 pr-9']">
                   <span :class="[selected ? 'font-semibold' : 'font-normal', 'block truncate']">
                     {{ role.label }}
                   </span>
@@ -188,7 +188,7 @@
                 as="template"
                 :value="employmentForm"
               >
-                <li :class="[active ? 'text-white bg-blumilk-600' : 'text-gray-900', 'cursor-default select-none relative py-2 pl-3 pr-9']">
+                <li :class="[active ? 'bg-gray-100' : 'text-gray-900', 'cursor-default select-none relative py-2 pl-3 pr-9']">
                   <span :class="[selected ? 'font-semibold' : 'font-normal', 'block truncate']">
                     {{ employmentForm.label }}
                   </span>
diff --git a/resources/js/Pages/VacationRequest/Create.vue b/resources/js/Pages/VacationRequest/Create.vue
index 7489fcf..fa6f8fa 100644
--- a/resources/js/Pages/VacationRequest/Create.vue
+++ b/resources/js/Pages/VacationRequest/Create.vue
@@ -72,7 +72,7 @@
                       as="template"
                       :value="user"
                     >
-                      <li :class="[active ? 'text-white bg-blumilk-600' : 'text-gray-900', 'cursor-default select-none relative py-2 pl-3 pr-9']">
+                      <li :class="[active ? 'bg-gray-100' : 'text-gray-900', 'cursor-default select-none relative py-2 pl-3 pr-9']">
                         <div class="flex items-center">
                           <img
                             :src="user.avatar"
@@ -166,7 +166,7 @@
                       as="template"
                       :value="vacationType"
                     >
-                      <li :class="[active ? 'text-white bg-blumilk-600' : 'text-gray-900', 'cursor-default select-none relative py-2 pl-3 pr-9']">
+                      <li :class="[active ? 'bg-gray-100' : 'text-gray-900', 'cursor-default select-none relative py-2 pl-3 pr-9']">
                         <span :class="[selected ? 'font-semibold' : 'font-normal', 'block truncate']">
                           <VacationType :type="vacationType.value" />
                         </span>