No description
* fix: sender is not set to current user I am not sure if I did it right: 1. import frappe 2. check if the sender is None: using frappe.session.user to fetch email address for current user * fix: update the correct field name for S3-backup-setting The field variable name should be updated to 'notify_email' instead of 'notification_email'. * style: remove a trailing whitespace * fix: Assign notify_email instead of calling frappe.db.get_value twice * chore: remove default sender and notify_email check * default sender already gets set in queue.py, so there's no need for a check inside __init__.py * notify_email seems to be a mandatory field, so there's no need to check if the field has a value (assuming that it always will). Signed-off-by: Chinmay D. Pai <chinmaydpai@gmail.com> Co-authored-by: Chinmay Pai <chinmaydpai@gmail.com> |
||
|---|---|---|
| .github | ||
| .travis | ||
| ci | ||
| cypress | ||
| frappe | ||
| rollup | ||
| .codacy.yml | ||
| .deepsource.toml | ||
| .eslintignore | ||
| .eslintrc | ||
| .gitignore | ||
| .mergify.yml | ||
| .pylintrc | ||
| .snyk | ||
| .travis.yml | ||
| attributions.md | ||
| bandit.yml | ||
| CODE_OF_CONDUCT.md | ||
| cypress.json | ||
| generate_bootstrap_theme.js | ||
| hooks.md | ||
| LICENSE | ||
| Makefile | ||
| node_utils.js | ||
| package.json | ||
| README.md | ||
| requirements.txt | ||
| SECURITY.md | ||
| setup.py | ||
| socketio.js | ||
| yarn.lock | ||
Full-stack web application framework that uses Python and MariaDB on the server side and a tightly integrated client side library. Built for ERPNext
Table of Contents
Installation
Contributing
Website
For details and documentation, see the website https://frappe.io
License
This repository has been released under the MIT License.