... because I can

Sometimes folks might want to download AdImpression data for further analysis or backups. Might be useful to provide a way to do this.

When new version is releasing???

Hi guys, im having problem with adding new banners, and i just got the error message: Please correct the following error

without any more data, i cant see where and why happen this error

Thanks

See also #22

Allow for users to supply details for external advertising sources in some capacity

the datetime.now function in the templatetag does not have brackets. '()'

Thanks to powellc for letting me know of this issue

Currently, AdImpressions and AdClicks have an fk in their admin list view, which triggers select_related without any limits. That means that all related models and model fields (to an unlimited depth) will be selected - even though all that's being used is the ad name!

I'd suggest the following querysets:

AdImpression.objects.select_related('ad').only('ad__title', 'impression_date', 'source_ip')
AdClick.objects.select_related('ad').only('ad__title', 'click_date', 'source_ip')

......F.....

FAIL: testClickOnAds (adzone.tests.ClickTestCase)

Traceback (most recent call last):
File "/home/fadeev/askbot-devel/adzone/tests.py", line 185, in testClickOnAds
self.assertEquals(len(AdClick.objects.all()), 2)
AssertionError: 1 != 2

Ran 12 tests in 1.745s

The database has been modified, but the migrations to let it be brought back in line are missing.

I needed the template tag to only look up random ads based on their zone, as a bunch of ads were in different categories. So I modified the random_ad manager with a really simple if/else check for the category slug.

It's hackish, and now you have to send a blank string in place of the category slug with the template tag, but it works.

See diff here: http://gist.github.com/582724

In django-adzone/admin.py:

• Class 'AdClickAdmin'
• parameter search_fields = ['ad' , 'source_ip' ] needs to be changed to search_fields = ['ad__title', 'source_ip ] , since you can't directly search on foreign key ( i'm using Django 1.8 )

Having a adzone depend on sites is not really ideal. This situation comes up with projects that don't really require multi site support (which is probably most projects tbh). Authors may not even include the sites package at all, but will be forced to do so, just so that they can run this app.

I feel that if a client/dev wants site support, they should add this relation to the child models that inherits from Adbase.

If a website is redesigned and the zones change, right now, all ads that were in the zone will be deleted - and by association, all ad impressions from ads in that zone will be deleted. Which would make ad impression data irrecoverably deleted.

I think there are a couple of potential solutions here.

1. Set ad category/zone foreign keys to NULL on delete instead of cascading.
2. Record advertisers on AdImpressions instead of/in addition to ads. Slash set the AdImpression -> Ad foreign key to NULL on delete instead of cascading.

Hi, I just installed adzone and followed the documentation. I am running django 1.7 on a python 3.4 virtualenv. I added adzone tomy installed apps and get the following error when starting the development server:

It would be nice to have the option of setting an ad to, say, start being chosen at midnight on a certain day, and stop being chosen on some other day at midnight - without having to actually stay up until midnight either day.

Do you plan to add support for google adsense codes?

The error is occurring here:

File ".../src/django-adzone/src/adzone/models.py", line 19, in Advertiser 
website = models.URLField(verify_exists=True)

Producing the following error:

TypeError: __init__() got an unexpected keyword argument 'verify_exists'

As per Django depreciation timeline:

"django.db.models.fields.URLField.verify_exists will be removed. The feature was deprecated in 1.3.1 due to intractable security and performance issues and will follow a slightly accelerated deprecation timeframe."

https://docs.djangoproject.com/en/1.5/internals/deprecation/

src/adzone/urls.py:Line 1:

• from django.conf.urls.defaults import *
• from django.conf.urls import patterns, include, url

Fix the inclusion tag name in the README

It is possible to run an entire site on {% random_zone_ad %} - but you would still need to specify a Category for every ad you create. Would it be possible to make Categories optional?

It would be awesome to be able to install django-adzone via:

pip install django-adzone

Short guide to registering on PyPI:
http://guide.python-distribute.org/quickstart.html#register-your-package-with-the-python-package-index-pypi

Hi,
i am using your django-adzone
my django version == 1.7.4
and django-adzone == 0.2
python == 2.7.4

i add app in install app
add url in mauin urls
and done migrations after i add category ans zone
after that i add banner and textads and add template tag of django_adzone in template
i follow all your instruction from doc but still not render any add in page

Thanks in advance!!!

Most django projects I've seen have a simple repository file structure, with the source folder at the root of the repository. I'd propose that django-adzone switch to this structure; the src/ folder doesn't seem to serve any purpose.

Right now, it's possible to have duplicate advertisers. For example, I could have two advertisers called BECU, each with the company website http://becu.org/.

It seems like it would make sense to require at least one of those fields to be unique. Maybe even both? Thoughts?