Fixed up a few things as per pull request review:

- Moved email domain and css bucket into dev config, not default

- deleted duplicated property

- removed unused property
This commit is contained in:
Martyn Inglis
2016-09-08 09:07:43 +01:00
parent d8bfad1ce7
commit bdc1d464bf

View File

@@ -49,15 +49,11 @@ class Config(object):
ADMIN_CLIENT_USER_NAME = 'notify-admin' ADMIN_CLIENT_USER_NAME = 'notify-admin'
AWS_REGION = 'eu-west-1' AWS_REGION = 'eu-west-1'
INVITATION_EXPIRATION_DAYS = 2 INVITATION_EXPIRATION_DAYS = 2
INVITATION_EMAIL_FROM = 'no-reply'
NOTIFY_APP_NAME = 'api' NOTIFY_APP_NAME = 'api'
NOTIFY_LOG_PATH = '/var/log/notify/application.log' NOTIFY_LOG_PATH = '/var/log/notify/application.log'
# Notification Queue names are a combination of a prefix plus a name
NOTIFICATION_QUEUE_PREFIX = 'development'
SQLALCHEMY_COMMIT_ON_TEARDOWN = False SQLALCHEMY_COMMIT_ON_TEARDOWN = False
SQLALCHEMY_RECORD_QUERIES = True SQLALCHEMY_RECORD_QUERIES = True
SQLALCHEMY_TRACK_MODIFICATIONS = True SQLALCHEMY_TRACK_MODIFICATIONS = True
NOTIFY_EMAIL_DOMAIN = 'notify.tools'
PAGE_SIZE = 50 PAGE_SIZE = 50
SMS_CHAR_COUNT_LIMIT = 495 SMS_CHAR_COUNT_LIMIT = 495
BRANDING_PATH = '/static/images/email-template/crests/' BRANDING_PATH = '/static/images/email-template/crests/'
@@ -132,7 +128,6 @@ class Config(object):
] ]
API_HOST_NAME = "http://localhost:6011" API_HOST_NAME = "http://localhost:6011"
CSV_UPLOAD_BUCKET_NAME = 'development-notifications-csv-upload'
NOTIFICATIONS_ALERT = 5 # five mins NOTIFICATIONS_ALERT = 5 # five mins
FROM_NUMBER = 'development' FROM_NUMBER = 'development'
@@ -148,6 +143,8 @@ class Config(object):
###################### ######################
class Development(Config): class Development(Config):
NOTIFY_EMAIL_DOMAIN = 'notify.tools'
CSV_UPLOAD_BUCKET_NAME = 'development-notifications-csv-upload'
NOTIFY_ENVIRONMENT = 'development' NOTIFY_ENVIRONMENT = 'development'
NOTIFICATION_QUEUE_PREFIX = 'development' NOTIFICATION_QUEUE_PREFIX = 'development'
DEBUG = True DEBUG = True