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

How to add the employee's name on the voucher #6377

Closed
lomamech opened this issue Feb 4, 2022 · 3 comments · Fixed by #6455
Closed

How to add the employee's name on the voucher #6377

lomamech opened this issue Feb 4, 2022 · 3 comments · Fixed by #6455

Comments

@lomamech
Copy link
Contributor

lomamech commented Feb 4, 2022

image

It is important to be able to put the employee's name on the documents instead of using their code

@jniles
Copy link
Collaborator

jniles commented Feb 7, 2022

Hmm... this has certain privacy implications. A printed document like this could be given to someone who doesn't have the right to access this information. Was this requested by a production site? If not, we should check if them to see if they agree or not.

@lomamech lomamech self-assigned this Mar 15, 2022
@lomamech
Copy link
Contributor Author

image
at IMCK for example, he had asked for it a very long time ago and for Vanga with the document generated during salary advances it will be very difficult to detect in the event of an error whether the data entry operator had chosen the best person

bors bot added a commit that referenced this issue Mar 15, 2022
6455: improvement(Employee voucher) r=jniles a=lomamech

- Add creditor name in Voucher for Employee

closes #6377

Co-authored-by: lomamech <lomamech@gmail.com>
@jniles
Copy link
Collaborator

jniles commented Mar 15, 2022

I would actually add this to the text of the description as well. Can we make this from "Avance Salaire" repeated to "Avance Salaire Jonathan" for example? It would help us find text later.

@bors bors bot closed this as completed in e8436a7 Mar 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants