Skip to content

feat: Add a process document for Subscription by deepeshgarg007 [frappe] PR#37126

Bot Dokos requested to merge ft-pr-37126 into develop

From: https://github.com/frappe/erpnext/pull/37126
Date: 2023-09-19 18:39:44+05:30

  • feat: Add a process document for Subscription (#37126)


Diagnostics

pre-commit failed for source commit: e19e04b050484dabb2185b5a2a174facc5c483d0
Check python ast.........................................................Failed
- hook id: check-ast
- exit code: 1

erpnext/accounts/doctype/subscription/subscription.py: failed parsing with CPython 3.10.13:

    Traceback (most recent call last):
      File "/root/.cache/pre-commit/repoah03545j/py_env-python3.10/lib/python3.10/site-packages/pre_commit_hooks/check_ast.py", line 20, in main
        ast.parse(f.read(), filename=filename)
      File "/usr/local/lib/python3.10/ast.py", line 50, in parse
        return compile(source, filename, mode, flags,
      File "erpnext/accounts/doctype/subscription/subscription.py", line 523
        			]
        			^
    SyntaxError: unmatched ']'

flake8...................................................................Failed
- hook id: flake8
- exit code: 1

erpnext/accounts/doctype/subscription/subscription.py:523:5: E999 SyntaxError: unmatched ']'
erpnext/accounts/doctype/subscription/test_subscription.py:24:24: F821 undefined name 'unittest'
erpnext/accounts/doctype/subscription/test_subscription.py:239:74: F821 undefined name 'current_date'
erpnext/accounts/doctype/subscription/test_subscription.py:240:72: F821 undefined name 'current_date'
erpnext/accounts/doctype/subscription/test_subscription.py:246:45: F821 undefined name 'current_date'
erpnext/accounts/doctype/subscription/test_subscription.py:250:50: F821 undefined name 'current_date'
erpnext/accounts/doctype/subscription/test_subscription.py:250:69: F821 undefined name 'current_date'
erpnext/accounts/doctype/subscription/test_subscription.py:272:14: F821 undefined name 'today'
erpnext/accounts/doctype/subscription/test_subscription.py:278:24: F821 undefined name 'diff'
erpnext/accounts/doctype/subscription/test_subscription.py:278:31: F821 undefined name 'plan_days'
erpnext/accounts/doctype/subscription/test_subscription.py:280:24: F821 undefined name 'invoice'
erpnext/accounts/doctype/subscription/test_subscription.py:282:3: F821 undefined name 'settings'
erpnext/accounts/doctype/subscription/test_subscription.py:282:22: F821 undefined name 'to_prorate'
erpnext/accounts/doctype/subscription/test_subscription.py:283:3: F821 undefined name 'settings'
erpnext/accounts/doctype/subscription/test_subscription.py:364:14: F821 undefined name 'today'
erpnext/accounts/doctype/subscription/test_subscription.py:389:3: F821 undefined name 'settings'
erpnext/accounts/doctype/subscription/test_subscription.py:389:33: F821 undefined name 'default_grace_period_action'
erpnext/accounts/doctype/subscription/test_subscription.py:390:3: F821 undefined name 'settings'
erpnext/accounts/doctype/subscription/test_subscription.py:402:3: F821 undefined name 'check_sales_documents_unicity'
erpnext/accounts/doctype/subscription/test_subscription.py:403:3: F821 undefined name 'check_si_creation'
erpnext/accounts/doctype/subscription/test_subscription.py:409:3: F821 undefined name 'payment_request'
erpnext/accounts/doctype/subscription/test_subscription.py:410:3: F821 undefined name 'payment_request'
erpnext/accounts/doctype/subscription/test_subscription.py:411:20: F821 undefined name 'payment_request'
erpnext/accounts/doctype/subscription/test_subscription.py:413:3: F821 undefined name 'check_si_creation'
erpnext/accounts/doctype/subscription/test_subscription.py:448:43: F821 undefined name 'from_date'
erpnext/accounts/doctype/subscription/test_subscription.py:449:41: F821 undefined name 'to_date'


Checkout instructions 
# Checkout locally
git fetch upstream
git switch ft-pr-37126

# Make changes then rebase
git rebase -i upstream/develop --reapply-cherry-picks

# Fix or ignore conflicts
git checkout --theirs .
git rebase --continue

# Force-push changes
git push --force-with-lease

Merge request reports