-
Notifications
You must be signed in to change notification settings - Fork 33
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
Deleting an user should not delete the related items created by that user. #11
base: master
Are you sure you want to change the base?
Conversation
Reset app version.
Reset app version.
…zone in the title. Add div wrapper for ad content. Easier for styling. Remove django from class name. It's better to not give clues of the underlying technology.
…re only the width may be necessary.
Hi @GabrielDumbrava, Sorry for taking too long to Comment on your PR. I was very busy in the past few months, and I could not manage to update django-ads. I already extracted most of the changes in your PR and added them to original Repo. I did not extract changes made to use filer, because I created this package to use it in a project (blog), and I do not use filer in it. I think it is better to create another package to be used on filer, or may be we can add filer as an optional requirement and add an option (lets say the idea is to add different field for filer. if The only problem in the approach is that it will make |
No description provided.