Skip to content

Commit

Permalink
Merge pull request #3744 from ColoredCow/feat/phase-2-prospect-module
Browse files Browse the repository at this point in the history
[Feat]:- Prospect Module Updates
  • Loading branch information
Ayush8923 authored Nov 4, 2024
2 parents 643ff54 + c0555f8 commit ddd1e2b
Show file tree
Hide file tree
Showing 17 changed files with 345 additions and 95 deletions.
11 changes: 11 additions & 0 deletions Modules/Client/Entities/Client.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
use Modules\Invoice\Entities\LedgerAccount;
use Modules\Invoice\Services\InvoiceService;
use Modules\Project\Entities\Project;
use Modules\Prospect\Entities\Prospect;
use Modules\User\Entities\User;

class Client extends Model
Expand Down Expand Up @@ -396,6 +397,16 @@ public function hasCustomInvoiceTemplate()
return false;
}

public function prospect()
{
return $this->belongsTo(Prospect::class);
}

public function getClientsAttribute()
{
return $this->query()->orderBy('name')->get();
}

protected static function booted()
{
static::addGlobalScope(new ClientGlobalScope);
Expand Down
9 changes: 9 additions & 0 deletions Modules/Prospect/Config/config.php
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,13 @@
'existing' => 'Existing',
'dormant' => 'Dormant',
],
'status' => [
'pending' => 'Pending',
'proposal-sent' => 'Proposal Sent',
'discussions-ongoing' => 'Discussions Ongoing',
'converted' => 'Converted',
'rejected' => 'Rejected',
'client-unresponsive' => 'Client Unresponsive',
'final-decision-pending' => 'Final Decision Pending',
],
];
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

class AddedClientIdProjectName extends Migration
{
/**
* Run the migrations.
*
* @return void
*/
public function up()
{
Schema::table('prospects', function (Blueprint $table) {
$table->unsignedBigInteger('client_id')->nullable();
$table->string('project_name')->nullable();
$table->string('organization_name')->nullable()->change();

$table->foreign('client_id')->references('id')->on('clients');
});
}

/**
* Reverse the migrations.
*
* @return void
*/
public function down()
{
Schema::table('prospects', function (Blueprint $table) {
$table->dropForeign(['client_id']);
$table->dropColumn('client_id');
$table->dropColumn('project_name');
$table->string('organization_name')->nullable(false)->change();
});
}
}
19 changes: 19 additions & 0 deletions Modules/Prospect/Entities/Prospect.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

use Carbon\Carbon;
use Illuminate\Database\Eloquent\Model;
use Modules\Client\Entities\Client;
use Modules\User\Entities\User;

class Prospect extends Model
Expand All @@ -26,4 +27,22 @@ public function getFormattedDate($date)
return $date ? Carbon::parse($date)->format('M d, Y')
: '-';
}

public function client()
{
return $this->belongsTo(Client::class, 'client_id');
}

public function getProspectDisplayName()
{
return $this->organization_name ?? optional($this->client)->name ?? 'N/A';
}

public function formattedAmount($amount)
{
$formattedAmount = preg_replace('/\B(?=(\d{2})+(?!\d))/', ',', substr($amount, 0, -3)) .
',' . substr($amount, -3);

return $formattedAmount;
}
}
2 changes: 1 addition & 1 deletion Modules/Prospect/Entities/ProspectComment.php
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,6 @@ public function prospect()

public function user()
{
return $this->belongsTo(User::class);
return $this->belongsTo(User::class, 'user_id');
}
}
25 changes: 13 additions & 12 deletions Modules/Prospect/Http/Controllers/ProspectController.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
use Illuminate\Contracts\Support\Renderable;
use Illuminate\Http\Request;
use Illuminate\Routing\Controller;
use Modules\Client\Entities\Client;
use Modules\Client\Entities\Country;
use Modules\Prospect\Entities\Prospect;
use Modules\Prospect\Http\Requests\ProspectRequest;
Expand All @@ -26,14 +27,10 @@ public function __construct(ProspectService $service)
*/
public function index()
{
$prospects = Prospect::with('pocUser')->get();
$countries = Country::all();
$currencySymbols = $countries->pluck('currency_symbol', 'currency');
$requestData = request()->all();
$data = $this->service->index($requestData);

return view('prospect::index', [
'prospects' => $prospects,
'currencySymbols' => $currencySymbols,
]);
return view('prospect::index', $data);
}

/**
Expand All @@ -44,11 +41,13 @@ public function create()
{
$countries = Country::all();
$user = new User();
$client = new Client();
$activeUsers = $user->active_users;

return view('prospect::create', [
'users' => $activeUsers,
'countries' => $countries,
'clients' => $client->clients,
]);
}

Expand All @@ -70,7 +69,7 @@ public function store(ProspectRequest $request)
*/
public function show($id)
{
$prospect = Prospect::with(['pocUser', 'comments', 'comments.user'])->find($id);
$prospect = Prospect::with(['comments'])->find($id);
$countries = Country::all();
$currencySymbols = $countries->pluck('currency_symbol', 'currency');

Expand All @@ -87,26 +86,28 @@ public function show($id)
*/
public function edit($id)
{
$prospect = Prospect::with(['pocUser', 'comments'])->find($id);
$prospect = Prospect::with(['comments'])->find($id);
$countries = Country::all();
$user = new User();
$activeUsers = $user->active_users;
$client = new Client();

return view('prospect::edit', [
'prospect' => $prospect,
'users' => $activeUsers,
'countries' => $countries,
'clients' => $client->clients,
]);
}

/**
* Update the specified resource in storage.
* @param Request $request
* @param int $id
* @param Prospect $prospect
*/
public function update(Request $request, $id)
public function update(Request $request, Prospect $prospect)
{
$data = $this->service->update($request, $id);
$data = $this->service->update($request, $prospect);

return $data;
}
Expand Down
26 changes: 24 additions & 2 deletions Modules/Prospect/Http/Requests/ProspectRequest.php
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ class ProspectRequest extends FormRequest
public function rules()
{
return [
'org_name' => 'required',
'org_name' => 'nullable',
'poc_user_id' => 'required',
'proposal_sent_date' => 'nullable|date',
'domain' => 'nullable',
Expand All @@ -26,9 +26,30 @@ public function rules()
'rfp_link' => 'nullable|url',
'proposal_link' => 'nullable|url',
'currency' => 'nullable',
'client_id' => 'nullable|exists:clients,id',
'project_name' => 'nullable',
];
}

/**
* Configure the validator instance.
*
* @param \Illuminate\Validation\Validator $validator
* @return void
*/
public function withValidator($validator)
{
if ($this->customer_type === 'new') {
$validator->addRules([
'org_name' => 'required',
]);
} elseif ($this->customer_type === 'existing') {
$validator->addRules([
'client_id' => 'required',
]);
}
}

/**
* Determine if the user is authorized to make this request.
*
Expand All @@ -47,7 +68,8 @@ public function authorize()
public function messages()
{
return [
'org_name.required' => 'Organization name is required',
'org_name.required' => 'Organization name is required when customer type is new.',
'client_id.required' => 'Please select an organization when customer type is existing.',
'poc_user_id.required' => 'Point of contact user ID is required',
];
}
Expand Down
38 changes: 38 additions & 0 deletions Modules/Prospect/Resources/assets/js/app.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,38 @@
const CUSTOMER_TYPES = {
NEW: 'new',
EXISTING: 'existing',
DORMANT: 'dormant'
};
document.addEventListener('DOMContentLoaded', function () {
const customerTypeField = document.getElementById('customer_type');
const orgNameTextField = document.getElementById('org_name_text_field');
const orgNameSelectField = document.getElementById('org_name_select_field');
const orgNameTextInput = document.getElementById('org_name');
const orgNameSelectInput = document.getElementById('org_name_select');

function toggleOrgNameField() {
if (customerTypeField.value === CUSTOMER_TYPES.NEW) {
orgNameTextField.classList.remove('d-none');
orgNameSelectField.classList.add('d-none');
orgNameTextInput.required = true;
orgNameSelectInput.value = '';
orgNameSelectInput.required = false;
} else if (customerTypeField.value === CUSTOMER_TYPES.EXISTING) {
orgNameTextField.classList.add('d-none');
orgNameSelectField.classList.remove('d-none');
orgNameSelectInput.required = true;
orgNameTextInput.required = false;
orgNameTextInput.value = null;
} else {
orgNameTextField.classList.remove('d-none');
orgNameSelectField.classList.add('d-none');
orgNameTextInput.required = true;
orgNameSelectInput.value = '';
orgNameSelectInput.required = false;
}
}

toggleOrgNameField();

customerTypeField.addEventListener('change', toggleOrgNameField);
});
Loading

0 comments on commit ddd1e2b

Please sign in to comment.