Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(subscription): Remove property decorators #17395

Merged
merged 2 commits into from May 27, 2019

Conversation

Projects
None yet
3 participants
@joeirimpan
Copy link
Contributor

commented Apr 26, 2019

Fixes #17394

joeirimpan and others added some commits Apr 26, 2019

@nabinhait nabinhait merged commit a0be413 into frappe:hotfix May 27, 2019

5 checks passed

Codacy/PR Quality Review Up to standards. A positive pull request.
Details
Semantic Pull Request ready to be squashed
Details
Travis CI - Pull Request Build Passed
Details
coverage/coveralls Coverage remained the same at 52.224%
Details
security/snyk - requirements.txt (frappe) No manifest changes detected

nabinhait added a commit to nabinhait/erpnext that referenced this pull request May 27, 2019

nabinhait added a commit that referenced this pull request May 27, 2019

Merge pull request #17756 from nabinhait/property-decorator
fix(subscription): Remove property decorator (#17395)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.