Peter Bittner
Peter Bittner
Your PR needs to be rebased to be eligible for being merged. > Sorry, no time to improve the tests This sounds like an arrogant statement. I'm sure you don't...
Any chance you would rebase your branch for this PR, @rristow? If you need any help for adjusting the tests afterwards, please let us know!
We should explain how to configure multiple databases, caches or search URLs directly in the [URL-based values section](https://django-configurations.readthedocs.io/en/latest/values/#url-based-values) of the documentation. This pops up regularly (see #97, #161, #170). Preferably,...
I have a similar complaint in issue #217.
The documentation on [Setup methods](https://django-configurations.readthedocs.io/en/latest/patterns/?highlight=work#setup-methods) says: > Of course that won’t work for `post_setup` since that’s when the settings setup is already done. Maybe it's the `setup()` method you need...
@ArijitWeird Can you try to figure out in the related GA documentation which settings (in JavaScript code, probably) are required to make this work? django-analytical is really just injecting those...
Also, please make sure you understand the meaning of that flag, and you verified your Django project with the note in the [related section of our documentation](https://django-analytical.readthedocs.io/en/latest/features.html#identifying-visitors). Thanks! :+1:
Is this issue still relevant? Could you figure out whether this was a bug, or rather an unclarity in the docs?
Can you compare the JavaScript code our implementation uses with the one of the current docs of Big Google? Maybe this is out-of-date here and needs to be aligned again?...
In my opinion, using setup methods is clunky. How do we sell `django-configurations`' advantages if the result of code like this is cumbersome (and less readable). I have a similar...