search

damn-dvlpr / loanApp

Loan Management Application.
9 stars 2 forks source link

Print receipt feature #4

Closed damn-dvlpr closed 4 years ago

damn-dvlpr commented 4 years ago

Print receipt feature to be added which prints the table shown in Customer details.

gaurav219 commented 4 years ago

Hey!! Do you mean a PDF document which'll show the table, right?

karmanya007 commented 4 years ago

I would like to help.Is anyone working on this?

damn-dvlpr commented 4 years ago

@gaurav219 yes, that will suffice. When one clicks on "marked delivered", it shall show the receipt to print in pdf format.

karmanya007 commented 4 years ago

table This table?

damn-dvlpr commented 4 years ago

Yes.

karmanya007 commented 4 years ago

table2 Or this one?

karmanya007 commented 4 years ago

After it has been marked delivered?

damn-dvlpr commented 4 years ago

Both have seldom differences, as the receipt is for the customer, it should contain the date of delivery and the total amount submitted i.e. amount given+interest. Hence first one would do, with a added field of delivery date.

karmanya007 commented 4 years ago

Okay

IndyDevGuy commented 4 years ago

Would it be okay to use wkhtmltopdf? There is a PHP wrapper for it here: https://github.com/mikehaertl/phpwkhtmltopdf, I have used it many times before and it works great. More info here: https://wkhtmltopdf.org

karmanya007 commented 4 years ago

Bro I am nearly done with this.

IndyDevGuy commented 4 years ago

Bro I am nearly done with this.

Okay no problem!

karmanya007 commented 4 years ago

1 2 3 Reciept.pdf

Is this fine??

karmanya007 commented 4 years ago

4

Also removed the metal which doesn't exist.

damn-dvlpr commented 4 years ago

This looks great! btw which library have you used for this? please open a PR so I can view the changes.

karmanya007 commented 4 years ago

No library. Just simple ol' window.print() 😉

damn-dvlpr commented 4 years ago

A sophisticated library would have been preferred, but nevertheless, I would merge this. If some problem persists, would revert it back as we don't have tests, so keep an eye if you're in for hacktoberfest.

damn-dvlpr commented 4 years ago

Solved by PR #6