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

djangor-river removes default `objects` manager from the model #33

Closed
javrasya opened this issue Oct 13, 2016 · 3 comments
Closed

djangor-river removes default `objects` manager from the model #33

javrasya opened this issue Oct 13, 2016 · 3 comments
Milestone

Comments

@javrasya
Copy link
Owner

@javrasya javrasya commented Oct 13, 2016

Django river is adding its own manager named workflow to the model. But, django detects it and decide there is no need to add default one which is objects automatically.

This is related with issue #32.

javrasya added a commit that referenced this issue Oct 17, 2016
…ing models not to have default objects one. There is no need for automatic manager injection. If anyone want to use it, this workflow manager can be added explicitly.
javrasya added a commit that referenced this issue Oct 17, 2016
…e is no automatic injection of workflow manager. It should be used explicitly.
@javrasya javrasya added this to the 0.9.0 milestone Oct 17, 2016
@javrasya

This comment has been minimized.

Copy link
Owner Author

@javrasya javrasya commented Oct 17, 2016

@psychok7 , accessing objects now possible but to access workflow manager, you should put it in your model as objects or as another manager to use it. Could you please check it and let me know is there any problem?

@psychok7

This comment has been minimized.

Copy link
Contributor

@psychok7 psychok7 commented Oct 17, 2016

@javrasya yes i will test it tomorrow and let you know.

@psychok7

This comment has been minimized.

Copy link
Contributor

@psychok7 psychok7 commented Oct 18, 2016

@javrasya tested it and it looks fine.

@javrasya javrasya closed this Oct 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.