Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug #167827 fix: Showing incomplete vendor profile #126

Open
wants to merge 1 commit into
base: release-1.4.4
Choose a base branch
from

Conversation

deepgawade89
Copy link
Contributor

If "Send payments directly to campaign promoter without charging any commissions (works only with Paypal payment gateway)" as Yes in that case profile should display 100% if vendor has filled remaining all details.

If "Send payments directly to campaign promoter without charging any commissions (works only with Paypal payment gateway)" as No, in that case payment gateway details should be considered to complete vendor profile.

@scrutinizer-notifier
Copy link

The inspection completed: 1 new issues, 1 updated code elements

@@ -660,7 +661,15 @@ public function getVendorProfileStatus($userId, $client)
$total += (!empty($data->vat_number)) ? 5: 0;
$total += (!empty($data->vendor_description)) ? 5: 0;
$total += (!empty($data->vendor_logo))? 5: 0;
$total += (!empty($data->payment_gateway)) ? 5: 0;

if ($sendPaymentsToOwner == '0')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@deepgawade89 you want to check 2 things here. First - send payment directly to the vendor is enabled or not.
OR
Second - Adaptive/split payment gateway is getting used by the client (JT/JGive)

@manojLondhe manojLondhe changed the base branch from release-1.4.3 to release-1.4.4 July 26, 2021 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants