darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-04 10:21:14 -05:00
Code Issues Packages Projects Releases Wiki Activity

The admin app is looking for reply_to_text not reply_to. This will fix the admin app and not affect the public api calls for template versions, they don't use the marshmallow schema

Browse Source
This commit is contained in:
Rebecca Law
2018-02-13 17:12:00 +00:00
parent 16c78a8924
commit c833a76070
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/schemas.py
View File

@@ -346,7 +346,7 @@ class TemplateSchema(BaseTemplateSchema):
class TemplateHistorySchema(BaseSchema): class TemplateHistorySchema(BaseSchema):
reply_to = fields.Method("get_reply_to", allow_none=True) reply_to_text = fields.Method("get_reply_to", allow_none=True)
created_by = fields.Nested(UserSchema, only=['id', 'name', 'email_address'], dump_only=True) created_by = fields.Nested(UserSchema, only=['id', 'name', 'email_address'], dump_only=True)
created_at = field_for(models.Template, 'created_at', format='%Y-%m-%d %H:%M:%S.%f') created_at = field_for(models.Template, 'created_at', format='%Y-%m-%d %H:%M:%S.%f')

2
tests/app/template/test_rest_history.py
View File

@@ -111,4 +111,4 @@ def test_update_template_reply_to_updates_history(client, sample_letter_template
hist_json_resp = json.loads(resp.get_data(as_text=True)) hist_json_resp = json.loads(resp.get_data(as_text=True))
assert 'service_letter_contact_id' not in hist_json_resp['data'] assert 'service_letter_contact_id' not in hist_json_resp['data']
assert hist_json_resp['data']['reply_to'] == str(letter_contact.id) assert hist_json_resp['data']['reply_to_text'] == str(letter_contact.id)