Packet 3.3 - Fall Packet MVP #162

devinmatte · 2019-08-20T16:28:31Z

New Features:

Push Notifications
Automated Emails
Speed Improvements

Send an email when the packet is generated

Updating Gulp and npm

Fixing css minify

Minifying JS

Gzip requests

Report an issue to Evals

Notifications Phase 1: Subscription

Phase 2: Notifications sent on Events

Fixing filter to map to allow signing

Fixing Notifications and adding logging

Notifications Phase 3: Using two clients

Fixing hardcoded references to instances

Phase 4: Starting notifications

mxmeinhold

LGTM! 🚢 🙌

config.env.py

packet/mail.py

packet/notifications.py

packet/routes/api.py

packet/routes/shared.py

JoelEager

Minor/style comments only. Overall, it looks great to me. 🚢

devinmatte added 30 commits August 10, 2019 00:08

Send an email when the packet is generated

a358a20

Fixing package versions

4a7d8cf

Fixing email formatting and variables

0076d38

Merge pull request #145 from devinmatte/mail

057714d

Send an email when the packet is generated

Fixing bool parse

2a55a59

Updating Gulp and npm

5a7cbdc

Merge pull request #146 from devinmatte/gulp_update

3130348

Updating Gulp and npm

Fixing css minify

1d6241d

Merge pull request #147 from devinmatte/gulp_update

444d824

Fixing css minify

Minifying JS

99911e6

Gzip and compressing

11401cd

Merge pull request #148 from devinmatte/minify_js

67edcc3

Minifying JS

Merge pull request #149 from devinmatte/gzip

af22d6e

Gzip requests

Report an issue to Evals

fc12145

Fixing newlines

3ccb94a

Merge pull request #151 from devinmatte/report

fe8ef31

Report an issue to Evals

Push Notifications Phase 1

23e7c20

Adding model to collect subscriptions

ee28245

Merge pull request #153 from devinmatte/onesignal_notifications

f6c2506

Notifications Phase 1: Subscription

Phase 2: Notification on signing

8f5f64a

Adding 100 notification

7ea68b0

Completed Notifications

fbb5482

Send 100% notification

6cdf8be

Removing prints

ebb19c6

Merge pull request #155 from devinmatte/onesignal_notifications

ead921d

Phase 2: Notifications sent on Events

Fixing filter to map to allow signing

5e0a5dd

Merge pull request #158 from devinmatte/onesignal_notifications

e9f0284

Fixing filter to map to allow signing

Fixing Notifications and adding logging

ec16f11

Merge pull request #159 from devinmatte/onesignal_notifications

24b4df8

Fixing Notifications and adding logging

Splitting into two onesignal clients

65109f8

devinmatte added 3 commits August 19, 2019 17:16

Dual Client setup

b30322e

Using send_notification in both

7147d85

Merge pull request #161 from devinmatte/onesignal_notifications

dcaf83b

Notifications Phase 3: Using two clients

devinmatte added this to the Version 3.3 milestone Aug 20, 2019

devinmatte added 7 commits August 20, 2019 23:18

Starting notifications

5f2361f

Fixing datetime formating

22da9ac

Single notification for starting

eed9e79

Fixing hardcoded references to instances

97d2932

Merge pull request #164 from devinmatte/hardcode_fix

6c5f713

Fixing hardcoded references to instances

Merge pull request #163 from devinmatte/onesignal_notifications

2a4bd8a

Phase 4: Starting notifications

Version Bump to 3.3.0

e0d3fae

devinmatte marked this pull request as ready for review August 23, 2019 17:23

devinmatte requested review from JoelEager and mxmeinhold August 23, 2019 17:27

Using the correct APP_ID

7b7374c

mxmeinhold approved these changes Aug 23, 2019

View reviewed changes