google / loaner

Automated Chromebook loaner management
Apache License 2.0
170 stars 63 forks source link

[Question] How's the notifications/reminder email works? #100

Closed WinsenTjhang closed 4 years ago

WinsenTjhang commented 4 years ago

Expected Behaviour

Received email notification during log in and returning device

Actual Behaviour

Did not receive notifications

Reproduce

Config version 1: I put the SEND_EMAIL_AS constant equal to my admin email and put the authorized email sender using the same email.

Config 2: Put the loaner role accounts

How's the app sending email notifications works? Does it require an account with G Suite license?

helfrichmichael commented 4 years ago

Please include errors from Google Cloud Logging or what steps you've taken to troubleshoot so far.

Additionally, please check your task queues in Google Cloud Platform and ensure they aren't paused/erroring.

As far as I can remember, the account doesn't have to exist, it just must be an authorized sender in GCP. We use a noreply@ address usually.

WinsenTjhang commented 4 years ago

Hi Michael,

I think the account needs to be exist

image

Here what I tried: SEND_EMAIL_AS = 'winsen@geekycloud.com'

image

that account is an active account with a license attached

I have flushed the Memcache and redeployed the web app

Logs

image

image

image

helfrichmichael commented 4 years ago

Did you update the serving version for each App Engine instance (action-system, chrome, default, and endpoints)? Additionally, if you debug the version running via the App Engine view and open the constants.py, is the email the email address you expect?

You're right regarding the email needing to exist. I confirmed it does exist on our end. Apologies for the confusion around that.

WinsenTjhang commented 4 years ago

It works now! Apparently I have been only updating my default system since my first deployment. Thanks a lot for the help!

No worries, also G Suite license is essential for the account.

helfrichmichael commented 4 years ago

That's great to hear and yeah, unfortunately it's not always obvious that you need to update each serving version. I'll close this out.