-
Notifications
You must be signed in to change notification settings - Fork 810
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
Pincode validation, test card & cors fixes. #201
Conversation
<ng-template [ngSwitchCase]="'Check'"> | ||
<app-cash-on-delivery (payOnDelivery)="makePayment()"></app-cash-on-delivery> | ||
<ng-template [ngSwitchCase]="'COD'"> | ||
<app-cash-on-delivery (payOnDelivery)="makePaymentCod()"></app-cash-on-delivery> | ||
</ng-template> | ||
<ng-template [ngSwitchCase]="'Payubiz'"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why string comparison. Also if payment mode name is changed, then what!! redeployment ???
import { Address } from '../../../core/models/address'; | ||
import { environment } from '../../../../environments/environment'; | ||
import { ToastrService } from '../../../../../node_modules/ngx-toastr'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why absolute path for ngx-toastr
|
||
paymentModes: PaymentMode[]; | ||
selectedMode: PaymentMode = new PaymentMode; | ||
isAuthenticated: boolean; | ||
showOrderSuccess = false; | ||
free_shipping_order_amount = environment.config.free_shipping_order_amount; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why underscore naming
@@ -5,7 +5,7 @@ import { Store } from '@ngrx/store'; | |||
import { Observable, Subscription } from 'rxjs'; | |||
import { Component, OnInit, OnDestroy } from '@angular/core'; | |||
import { environment } from '../../../environments/environment'; | |||
import { CheckoutService } from '../../core/services/checkout.service'; | |||
import { Router } from '../../../../node_modules/@angular/router'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
check the import path before auto applying
@@ -6,7 +6,7 @@ export class PaymentService { | |||
setCODAsSelectedMode(modes) { | |||
let selectedMode; | |||
modes.forEach((mode) => { | |||
if (mode.name === 'Check') { | |||
if (mode.name === 'Payubiz') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use constant only
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
check comments
Why? Features Added random transaction id for every payubiz transaction. Added test card when user clicks on payubiz payment method for testing payment process. Added pincode check for COD orders. Fixes Eliminated Cors: Instead of Ajax the form-data post request has been made via backend api. This change addresses the need by: Random transaction ID instead of Order-Id. Test card will helps user to test payubiz payment Process. Now Orders with available pincodes are eligible for free shipping with specified free shipping value. [delivers #159134594, #159209334]
Why? Features Added random transaction id for every payubiz transaction. Added test card when user clicks on payubiz payment method for testing payment process. Added pincode check for COD orders. Fixes Eliminated Cors: Instead of Ajax the form-data post request has been made via backend api. This change addresses the need by: Random transaction ID instead of Order-Id. Test card will helps user to test payubiz payment Process. Now Orders with available pincodes are eligible for free shipping with specified free shipping value. [delivers #159134594, #159209334]
Why? Features Added random transaction id for every payubiz transaction. Added test card when user clicks on payubiz payment method for testing payment process. Added pincode check for COD orders. Fixes Eliminated Cors: Instead of Ajax the form-data post request has been made via backend api. This change addresses the need by: Random transaction ID instead of Order-Id. Test card will helps user to test payubiz payment Process. Now Orders with available pincodes are eligible for free shipping with specified free shipping value. [delivers #159134594, #159209334]
Why?
Features
Fixes
This change addresses the need by:
[delivers #159134594, #159209334]
Story Test Card.
Story Pincode check.