From ae081ddddf1b1ba629cd36be353e66c30486b3ac Mon Sep 17 00:00:00 2001 From: Jack Goh Date: Wed, 8 Aug 2018 10:41:29 +0800 Subject: [PATCH 1/4] fix booking internal server error due to db not accepting null value fixed copy right on booking form update notification seeder --- ...103115_make_nullable_in_bookings_table.php | 37 +++++++++++++++++++ database/seeds/NotificationSeeder.php | 4 +- resources/assets/js/pages/home.vue | 2 +- 3 files changed, 40 insertions(+), 3 deletions(-) create mode 100644 database/migrations/2018_08_08_103115_make_nullable_in_bookings_table.php diff --git a/database/migrations/2018_08_08_103115_make_nullable_in_bookings_table.php b/database/migrations/2018_08_08_103115_make_nullable_in_bookings_table.php new file mode 100644 index 00000000..d8524e66 --- /dev/null +++ b/database/migrations/2018_08_08_103115_make_nullable_in_bookings_table.php @@ -0,0 +1,37 @@ +string('company_name')->nullable()->change(); + $table->string('company_address')->nullable()->change(); + $table->string('bank_address')->nullable()->change(); + $table->string('swift_code')->nullable()->change(); + $table->string('cnap')->nullable()->change(); + $table->string('term')->nullable()->change(); + }); + } + + /** + * Reverse the migrations. + * + * @return void + */ + public function down() + { + Schema::table('bookings', function (Blueprint $table) { + // + }); + } +} diff --git a/database/seeds/NotificationSeeder.php b/database/seeds/NotificationSeeder.php index ac238997..316b11ad 100644 --- a/database/seeds/NotificationSeeder.php +++ b/database/seeds/NotificationSeeder.php @@ -19,7 +19,7 @@ class NotificationSeeder extends Seeder DB::table('notifications')->insert([ 'detail' => 'This is an user seeder notification', 'user_id' => $user->id, - 'link' => 'http://www.google.com', + 'link' => '/', 'created_at' => Carbon::now()->format('Y-m-d H:i:s'), 'updated_at' => Carbon::now()->format('Y-m-d H:i:s'), ]); @@ -27,7 +27,7 @@ class NotificationSeeder extends Seeder DB::table('notifications')->insert([ 'detail' => 'This is an admin seeder notification', 'user_id' => $admin->id, - 'link' => 'http://www.google.com', + 'link' => '/', 'created_at' => Carbon::now()->format('Y-m-d H:i:s'), 'updated_at' => Carbon::now()->format('Y-m-d H:i:s'), ]); diff --git a/resources/assets/js/pages/home.vue b/resources/assets/js/pages/home.vue index 748e8fb6..cd6abbc6 100644 --- a/resources/assets/js/pages/home.vue +++ b/resources/assets/js/pages/home.vue @@ -149,7 +149,7 @@ - + From 526486f95aeb96dff43f61f9808c5cf818d86311 Mon Sep 17 00:00:00 2001 From: Jack Goh Date: Wed, 8 Aug 2018 10:55:41 +0800 Subject: [PATCH 2/4] fix supplier booking nullable fix booking seeder wrong status --- ...ke_nullable_in_supplier_bookings_table.php | 33 +++++++++++++++++++ database/seeds/BookTableSeeder.php | 2 +- 2 files changed, 34 insertions(+), 1 deletion(-) create mode 100644 database/migrations/2018_08_08_104726_make_nullable_in_supplier_bookings_table.php diff --git a/database/migrations/2018_08_08_104726_make_nullable_in_supplier_bookings_table.php b/database/migrations/2018_08_08_104726_make_nullable_in_supplier_bookings_table.php new file mode 100644 index 00000000..6c91e5f7 --- /dev/null +++ b/database/migrations/2018_08_08_104726_make_nullable_in_supplier_bookings_table.php @@ -0,0 +1,33 @@ +decimal('transfer_amount')->nullable()->change(); + $table->decimal('rmbBankAmount')->nullable()->change(); + }); + } + + /** + * Reverse the migrations. + * + * @return void + */ + public function down() + { + Schema::table('supplier_bookings', function (Blueprint $table) { + // + }); + } +} diff --git a/database/seeds/BookTableSeeder.php b/database/seeds/BookTableSeeder.php index 9bbc0d9a..e27ff224 100644 --- a/database/seeds/BookTableSeeder.php +++ b/database/seeds/BookTableSeeder.php @@ -35,7 +35,7 @@ class BookTableSeeder extends Seeder 'rate_id' => $rate->id, 'user_id' => $user->id, 'status' => 2, - 'admin_status' => 2, + 'admin_status' => 1, 'order_no' => '001', 'payment_for' => 'FULL PAYMENT', 'created_at' => Carbon::now()->format('Y-m-d H:i:s'), From 06bacfcaf9a97506039e329c1057a80b4f08d0a3 Mon Sep 17 00:00:00 2001 From: Jack Goh Date: Wed, 8 Aug 2018 16:31:35 +0800 Subject: [PATCH 3/4] fixed x2 status for user side --- app/Http/Controllers/BookingController.php | 21 +- .../assets/js/components/ProgressTrack.vue | 28 +-- .../assets/js/pages/booking/completed.vue | 8 +- .../assets/js/pages/booking/confirmation.vue | 8 +- resources/assets/js/pages/booking/po.vue | 6 +- .../assets/js/pages/booking/poComplete.vue | 8 +- .../assets/js/pages/booking/transfer.vue | 8 +- .../assets/js/pages/booking/uploadPo.vue | 8 +- .../js/pages/booking/uploadUserBankSlip.vue | 65 +++--- .../assets/js/pages/booking/x2Completed.vue | 186 ++++++++++++++++++ resources/assets/js/pages/home.vue | 14 +- resources/assets/js/router/routes.js | 4 +- 12 files changed, 288 insertions(+), 76 deletions(-) create mode 100644 resources/assets/js/pages/booking/x2Completed.vue diff --git a/app/Http/Controllers/BookingController.php b/app/Http/Controllers/BookingController.php index 7d8c6bf2..1df1aba5 100644 --- a/app/Http/Controllers/BookingController.php +++ b/app/Http/Controllers/BookingController.php @@ -46,6 +46,12 @@ class BookingController extends Controller $booking->transfer_amount = $booking->bia; $booking->track_status = "(". $booking->status ."/7)"; + + // all x2 until step 5 only + if ($booking->term == "x2_cash" || $booking->term == "x2_cheque" || $booking->term == "x2_ba") { + $booking->track_status = "(". $booking->status ."/5)"; + } + switch ($booking->status) { case 1: $status_desc = "Order in progress"; @@ -61,6 +67,9 @@ class BookingController extends Controller break; case 5: $status_desc = "Transfer Complete, please upload your PO"; + if ($booking->term == "x2_cash" || $booking->term == "x2_cheque" || $booking->term == "x2_ba"){ + $status_desc = "Order Completed"; + } break; case 6: $status_desc = "Processing invoice"; @@ -118,15 +127,11 @@ class BookingController extends Controller $status_desc = "Verify user bank slip"; break; case 3: - if($term !== "x1_ba" || $term !== "x2_cheque"){ - $status_desc = "Generate supplier booking report"; - break; - } + $status_desc = "Generate supplier booking report"; + break; case 4: - if($term !== "x1_ba" || $term !== "x2_cheque"){ - $status_desc = "Supplier booking report. Please confirm with your booking."; - break; - } + $status_desc = "Supplier booking report. Please confirm with your booking."; + break; case 5: $status_desc = "Please, upload China Bankslip"; break; diff --git a/resources/assets/js/components/ProgressTrack.vue b/resources/assets/js/components/ProgressTrack.vue index 21241dfb..f7238a85 100644 --- a/resources/assets/js/components/ProgressTrack.vue +++ b/resources/assets/js/components/ProgressTrack.vue @@ -1,12 +1,12 @@