We need to show more context to the user

[djpnewton]

The user needs more information when:

• the user receives an email
• on the final invoice status page

The info we need is:

• utility name
• how much we are paying

[eoliveros]

added the amount in msg.body. Testing and waiting for the email to be sent.

[eoliveros]

not getting the email. will check.

[djpnewton]

is the email still failing?

[eoliveros]

just tested this. and i registered my personal email to see if I'll get the update. So far nothing.

[djpnewton]

I have added you as a collaborator (https://dashboard.heroku.com/apps/zap-bill-payment), check the logs when it sends the email

[eoliveros]

the zap-bll-payment in heroku is sending emails. Is there any way to get the one in caprover to do that as well? I can't remember how to deploy to heroku and with my constant change and test, its better for me to get this environment working.

[djpnewton]

We dont have an instance in caprover yet, you can put one on if you want

[eoliveros]

I have this at the moment:

https://test-zap-bill-payment.caprover.acuerdo.dev

May need to update it though. As well as the config incase something is not matching.

[djpnewton]

Yes, you can deploy it to caprover for testing

[eoliveros]

trying to get this to email but, for some reason its not emailing. cant tell why its not emailing though.

[eoliveros]

checking the docker logs:

--SNIP--
connect sid: 6680d913ac71451191fa4e5d7cee6676
connect sid: 3586a20b3f96455e9983edd2e90395e7
watching address 3MwXqmVwLKF2ZKZ2g2HMcppe667TCLjnmtS for b0159ffc8caf3857
connect sid: 5bfa3949fc0f49db92e9615a3677c91a

--/SNIP--

There's nothing thats triggering the email?

[eoliveros]

ahhh, looks like its not fully uptodate.... pulling updates.

[eoliveros]

deployed and edited to get the amount:

Will need to cleanup the message though.

Also investigating on how to get the name of the utility in to the message as well.

[eoliveros]

Ok, with the changes to models.py and others, i got the msg to include the bank name as well:

so the name is also now included as well as the amount in $

[djpnewton]

Cool, now we need a PR to add that info to the invoice status page too

[eoliveros]

invoice status page? You mean this page?

The amount was already there. To get the utility_name to show up i had to edit the this as well to get the utility_name in the msg as well.

[djpnewton]

no the status page showed to the user, like when they follow the link in the email

On Wed, Sep 9, 2020 at 12:59 PM eoliveros notifications@github.com wrote:

invoice status page? You mean this page? [image: image] https://user-images.githubusercontent.com/47799652/92542112-04f1d900-f29c-11ea-9672-7eecc018e2f5.png

The amount was already there. To get the utility_name to show up i had to edit the this as well to get the utility_name in the msg as well.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/zap-me/zap_bill_payment/issues/17#issuecomment-689233357, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAKLWZPUFPTK5YL2W25WYTSE3HOFANCNFSM4QFTJFHQ .

[eoliveros]

Like this?

[djpnewton]

yeah but maybe in the main part next to the expiry

On Wed, Sep 9, 2020 at 3:36 PM eoliveros notifications@github.com wrote:

Like this? [image: image] https://user-images.githubusercontent.com/47799652/92551476-2b227380-f2b2-11ea-9da1-3a3642f6c39d.png

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/zap-me/zap_bill_payment/issues/17#issuecomment-689279414, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAKLW3TUBRLH6GFOZE7ZG3SE3Z4HANCNFSM4QFTJFHQ .

[eoliveros]

How about this:

[djpnewton]

yep