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

Warning about invalid info in the PDF version of reimbursement #29

Merged
merged 3 commits into from
Apr 25, 2016

Conversation

ancorgs
Copy link
Member

@ancorgs ancorgs commented Apr 25, 2016

This closes #27

@ancorgs
Copy link
Member Author

ancorgs commented Apr 25, 2016

@tsdgeos This pull request (when fixed to work with ruby 1.9) will introduce a warning like this in the PDF if some information is not complete. I hope it would be enough to prevent people from signing the blank paper first and entering the information afterwards.

tsp_warning

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.8%) to 92.498% when pulling 44a4106 on warning_in_pdf into e180da9 on master.

@tsdgeos
Copy link
Contributor

tsdgeos commented Apr 25, 2016

I guess that should be enough yes.

@ancorgs ancorgs merged commit 35fdc41 into master Apr 25, 2016
@ancorgs ancorgs deleted the warning_in_pdf branch January 26, 2017 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Warn if trying to print the "signed acceptance" without authorized amounts
3 participants