Configurable response messages

[surajraktate]

I make changes in view.py to make responses configurable please review.

[anx-ckreuzberger]

Thanks for your PR.

Your change is api-breaking, e.g.:
Instead of

return Response({'status': 'OK'})

you use

response_dict = dict({"status_code": None, "message": None})
message = get_response_message("PASSWORD_CHANGED")
response_dict.update({"status_code": 200, "message": message})

return Response(response_dict)

which results in the response {"status_code": 200, "message": "..."} instead of {"status": "OK"}.

Please re-introduce the "status" field.

[anx-ckreuzberger]

Furthermore, please rename the PR accordingly, and add the "documentation" that is currently in Contributors.md into README.md within the configuration / settings section - e.g., with a heading Configurable Response.

[surajraktate]

Thanks for your PR.

Your change is api-breaking, e.g.:
Instead of

return Response({'status': 'OK'})

you use

response_dict = dict({"status_code": None, "message": None})
message = get_response_message("PASSWORD_CHANGED")
response_dict.update({"status_code": 200, "message": message})

return Response(response_dict)

which results in the response {"status_code": 200, "message": "..."} instead of {"status": "OK"}.

Please re-introduce the "status" field.

re-introduces the "status" field in #d7429b6 commit
d7429b6

[surajraktate]

I have done all requested change

[anx-cbenke]

Sorry the answer to your PR took so long! There are a few further change-requests, mainly about language. If these are adressed and all new conflicts are resolved, we can finally merge your PR.

[anx-cbenke]

Perfect! Thank you for the changes! There's one last merge-conflict that needs to be resolved - could you fix that too so we can finalize this?

[surajraktate]

sorry for commit without testing, now I can see all checks passed, Thanks