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.

Sign up for GitHub

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

Jump to bottom

Multiple payment methods while lending #166

Open
shahns opened this issue Feb 23, 2024 · 0 comments · May be fixed by #194
Open

Multiple payment methods while lending #166

shahns opened this issue Feb 23, 2024 · 0 comments · May be fixed by #194

Comments

@shahns
Copy link

shahns commented Feb 23, 2024

Is your feature request related to a problem? Please describe.

Currently, it looks like it is not possible to lend using multiple payment methods, i.e. partially by cash and remaining by bank transfer, UPI etc.

As per Registrar General of Money Lenders, Maharashtra State this limit is 20,000.
(https://sahakarayukta.maharashtra.gov.in/site/upload/documents/Circular%2028%20Aug%202018.pdf)

Describe the solution you'd like

From my pov solution looks like:

  1. Single doctype which defines this limit.
  2. A payment screen/section where multiple mode of payments (mop) are available.
  3. While making payment with mode of payment as cash check if this limit is obeyed, if not, error screen is shown and entered lending value for cash mop is auto-updated to 20,000 (as defined in single doctype) and remaining amount is taken for bank transfer mop.
  4. Such accounting effects are made against cash a/c and bank a/c.

Describe the alternatives you've considered

No response

Additional context

No response
nirzaf added a commit to nirzaf/lending that referenced this issue Aug 14, 2024
@nirzaf
Add multiple payment methods while lending
55c316b 
Fixes frappe#166

Add support for multiple payment methods in loan applications and loans.

* Add a new doctype `Payment Method Limit` to define the cash limit.
* Update `loan_application.js` to include a payment section with multiple modes of payment.
  * Add validation to check if the cash limit is obeyed.
  * Show an error screen if the cash limit is exceeded and auto-update the entered lending value for cash to 20,000.
* Update `loan_application.py` to handle the logic for multiple payment methods.
  * Add logic to distribute the remaining amount to other payment methods like bank transfer or UPI.
* Update `loan.js` to support multiple payment methods.
  * Add a payment section with multiple modes of payment.
  * Add validation to check if the cash limit is obeyed.
  * Show an error screen if the cash limit is exceeded and auto-update the entered lending value for cash to 20,000.
* Update `loan.py` to handle the logic for multiple payment methods.
  * Add logic to distribute the remaining amount to other payment methods like bank transfer or UPI.
@nirzaf nirzaf linked a pull request Aug 14, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add multiple payment methods while lending nirzaf/lending
1 participant
@shahns