search

SEED-platform / seed

Standard Energy Efficiency Data (SEED) Platform™ is a web-based application that helps organizations easily manage data on the energy performance of large groups of buildings.
Other
106 stars 55 forks source link

Fix email #4642

Closed haneslinger closed 1 month ago

haneslinger commented 2 months ago

[

Any background context you want to provide?

What's this PR do?

How should this be manually tested?

What are the relevant tickets?

Screenshots (if appropriate)](https://github.com/SEED-platform/seed/issues/4639)

github-actions[bot] commented 2 months ago

Label error. Requires at least 1 of: Feature, Bug, Enhancement, Maintenance, Documentation, Performance, Do not publish. Found:

axelstudios commented 2 months ago
nllong commented 2 months ago

This seems close, but when I select only a single property, then no data are sent. If I select two items in the inventory, then the first is selected. It feels like we have an off by one issue.

To test, by default the emailer is your console. So if you had an email address to a property's owner_email field and add the following email template in the http://127.0.0.1:8000/app/#/accounts/10/email_templates config, then when you select and run, it will show up in your console.

Hello owner of {{ubid}}!

You have a square footage of {{gross_floor_area}}.

Enjoy!
haneslinger commented 2 months ago

@nllong, I don't see the emails in my console, but I do see log that imply both selected properties were sent emails

seed_web       | [INFO]2024-04-23 09:38:45 PID 15: Process finished, 2 attempted, 2 sent, 0 failed, 0 requeued
seed_web       | 2024-04-23 09:38:45 INFO     Process finished, 2 attempted, 2 sent, 0 failed, 0 requeued
seed_web       | [INFO]2024-04-23 09:38:45 PID 15: 2 emails attempted, 2 sent, 0 failed, 0 requeued
seed_web       | 2024-04-23 09:38:45 INFO     2 emails attempted, 2 sent, 0 failed, 0 requeued

are you sure both of your selected properties have emails?