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

make django-oscar-accounts really could work without oscar #44

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

wtayyeb
Copy link

@wtayyeb wtayyeb commented Jul 18, 2016

I gather rest of what is imported from oscar in a model (compact_oscar.py) and with a simple import check it could fallback to oscar if installed. But in case one use this interesting package alone, there will some replacements to get the oscar-accounts works alone.

@sutyrin
Copy link

sutyrin commented Jul 30, 2016

Pity it's not merged yet. I had to take entire oscar with my small project. %p

@codecov-io
Copy link

codecov-io commented Jul 30, 2016

Current coverage is 68.87% (diff: 39.28%)

No coverage report found for master at be57d82.

Powered by Codecov. Last update be57d82...3e9359a

@gabn88
Copy link

gabn88 commented Dec 22, 2016

Really a pity this is not merged. I think it would be good for the project as well. This package helps a lot if you want to make your own module based project with recurred billing, where an account is the credit a user has.

@solarissmoke
Copy link
Member

I know this is quite old, but if you're able to bring this up to date with master I'm more than happy to review/merge!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants