Conversation
… modes, prevent duplicate EMI processing, and ensure webhook log persistence.
status field fix in student applicant
…y payment requests in webhooks with fallback logic.
…e error reporting for API responses.
…abase commit after requests, and remove integration request logging from webhook handling.
PG webhook: extract bank_reference_id from payment_details and store as reference_no EMI webhook: extract UTR from disbursement_details and store as reference_no
…oks when `udf_details` are missing.
…le fallbacks and error handling.
Safety net for cases where late fee was wrongly added after EMI initiation. On emi.disbursed webhook, if the disbursed amount differs from PR grand_total, removes late fee for that specific payment term only using fees.adjust_late_fee(0, payment_term) and updates PR grand_total to match the actual disbursed amount.
fix: adjust late fee on EMI disbursement when amount mismatches
For dt.payment.captured, use payment_details.paid_on date. For emi.disbursed, use disbursement_details.date. Falls back to today if date is null or unparseable.
fix: use webhook date as Payment Entry posting_date instead of today
…on (#22) * fix: use correct Ticket field names in fee header calculation - Replace custom_selected_payment_gateway with payment_gateway_type - Replace custom_selected_payment_gateway_name with payment_gateway_account - Replace custom_seat_pricing_total with seat_pricing_total - Replace custom_free_seats_count with free_seats_count - Replace custom_chargeable_seats_count with chargeable_seats_count The incorrect custom_ prefix caused _get_ticket_fee_headers and _calculate_grade_breakdown to read None from the Ticket doc, making the fee_headers payload always fall back to {total_payable, current_payable} instead of using the grade-specific fee header from Event Listing grade_details. * test: add unit tests for Ticket fee header field name fix - Test grade-specific fee header returned when grade + gateway match - Test fallback to total_payable/current_payable when no match - Test empty seats fallback - Test multi-grade per-grade fee header breakdown - Test multi-grade fallback when no grade_details configured
- Add _get_ticket_student_details helper to collect unique allocated students from ticket seats and return the first student's details - GrayQuest API expects a dict for student_details, not a list; sending a list caused a 400 validation error - Update _get_event_ticket_payload to accept controller as first arg and call the new helper to populate student_details - Update get_payload to pass controller through to _get_event_ticket_payload
…etails Fix GrayQuest Ticket payment to include student_details
…etails fix: accumulate fee header amounts in _get_ticket_fee_headers function
feat: redirect fee payments to walsh SPA payment-status page for deep…
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.