search

dmpayton / django-admin-honeypot

:honey_pot: A fake Django admin login screen page.
http://django-admin-honeypot.readthedocs.io/
MIT License
1.03k stars 188 forks source link

Added missing template vars to context data #76

Open GitRon opened 4 years ago

GitRon commented 4 years ago

Fixes https://github.com/dmpayton/django-admin-honeypot/issues/62

denizdogan commented 4 years ago

@dmpayton I'm sorry if you're swamped with other tasks, but is there any chance this will get looked at soon? :) Thanks.

GitRon commented 4 years ago

@dmpayton @9mido Pipeline is fixed and PR is ready to merge.

9mido commented 3 years ago

@GitRon Awesome job!

GitRon commented 3 years ago

@dmpayton @9mido Would be awesome if somebody could review and merge it 😃

blag commented 3 years ago

Hi, I have forked this project, included a few PRs (including this one), and released a package to PyPI. You might have better luck with my fork.

GitRon commented 3 years ago

@blag Thx for your effort!

dmpayton commented 2 years ago

@GitRon I just pushed a commit to develop that I think resolves the issue. The relevant bit is here:

https://github.com/dmpayton/django-admin-honeypot/blob/develop/admin_honeypot/views.py#L40

Basically, just grab the default context for every Django admin view, instead of rebuilding that list manually and having to keep up with Django's own updates. What do you think?

GitRon commented 2 years ago

Hi @dmpayton! Thx for all the effort! Unfortunately, I removed the package from my application due to version conflicts. I think the solution looks good, as far as I can tell.

Will you release a new version soon, after fixing so many issues? 🙂