django-qrcode icon indicating copy to clipboard operation
django-qrcode copied to clipboard

quick fix for #11

Open olivierdalang opened this issue 9 years ago • 1 comments

I didn't put lots of thoughts there ;)

IMO we could think of removing qr_from_object altogether, since not every user defines the absolute urls in the same way...

Bests,

Olivier

olivierdalang avatar Apr 15 '16 01:04 olivierdalang

I agree that's broken, and using django.contrib.site is quite a bad hack (apologies, this is a very old project), but moving the import doesn't fix the issue.

Maybe using context['request'] is a better fix? That will mean to enforce the 'django.template.context_processors.request' context processor, but surely that's better.

pablorecio avatar Aug 24 '16 21:08 pablorecio