Adds delivery address to distribution printout/receipt #4164
Conversation
|
@dorner where exactly this is shown (on the pdf?) or inside the app? |
|
If you sign in as verified@example.com, click edit my organization -- there is a Program Address (if different) in that screen, which needs to be changed. Also, if you click "My organization", which is the corresponding view. These are also accessed from the bank side, though the partner agencies, then clicking on the specific partner 'edit" or "view', but I'm pretty sure they use the same partials. |
kannans5
force-pushed
the
4138-adds-delivery-addr-to-distr-printout
branch
from
cd02ffd
to
94d8a15
Compare
|
I changed at these 3 places:
|
Let me know if these are the changes we are looking for. |
|
Looks like it to me. |
|
Looks good to me too. Thanks! |
|
Hrm. When I did the pre-release test of this on staging, on a delivery distribution, I didn't get a delivery address printing out. I'm not going to hold the release on this, but something is not working properly. |
|
@kannans5: Your PR |
Resolves #4138
Description
Adds delivery address to distribution printout/receipt
Type of change
How Has This Been Tested?
Delivery address:added to the left. (if delivery method is shipped or delivered)Note: I did not find any existing unit test or browser test related to distribution pdf. If someone could point me to the relevant tests, I am happy to cover this scenario.
Screenshots
If the distribution delivery method is either shipped or delivered, the print will look like this
If not no change in the receipt.