Skip to content
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

Update for modern Django! #66

Open
pydanny opened this issue Apr 28, 2020 · 25 comments
Open

Update for modern Django! #66

pydanny opened this issue Apr 28, 2020 · 25 comments

Comments

@pydanny
Copy link

pydanny commented Apr 28, 2020

Modern Django could really use this tool. I want to keep it in Two Scoops of Django 3.x!

@dmpayton
Copy link
Owner

Done. See #67 and #68. I'm going to leave this open until there's an updated release on PyPI.

Thanks @pydanny for the nudge and @harisibrahimkv for doing the bulk of the work! 🎉

@9mido
Copy link

9mido commented May 18, 2020

Any idea when the new version will be available on PyPi?

@dmpayton
Copy link
Owner

@9mido I'll issue a PyPI release when #62 is resolved.

@GitRon
Copy link

GitRon commented Aug 7, 2020

Hi @dmpayton

I just saw some deprecation warnings for django 4.0:

site-packages\admin_honeypot\signals.py:3: RemovedInDjango40Warning: The providing_args argument is deprecated. As it is purely documentational, it has no replacement. If you rely on this argument as documentation, you can move the text to a code comment or docstring.

and

site-packages\admin_honeypot\models.py:7: RemovedInDjango40Warning: django.utils.translation.ugettext_lazy() is deprecated in favor of django.utils.translation.gettext_lazy().

Might be a quick win to fix this as well?

Update: Here is another one, but this might impact the django versions we would be able to support...?

site-packages\admin_honeypot\urls.py:7: RemovedInDjango40Warning: django.conf.urls.url() is deprecated in favor of django.urls.re_path().

Best regards
Ronny

@dmpayton
Copy link
Owner

dmpayton commented Aug 7, 2020

Thanks @GitRon, that's good to know! I'm under a big work deadline atm, and likely won't have a chance to work on admin_honeypot for another week or so, but I'd happily merge a PR for this before then. :)

@SuperMasterBlasterLaser

@dmpayton Looks like support for Django 3.0 had been merged. When new version will be released to pip?

@GitRon
Copy link

GitRon commented Nov 30, 2020

My merge requests have failed in the pipeline... Never realised that it didn't went through... I'll see that I'll have a look at them.

@9mido
Copy link

9mido commented Dec 2, 2020

@GitRon Yes please fix your PR failures

@GitRon
Copy link

GitRon commented Dec 2, 2020

@9mido @dmpayton The develop pipeline is broken, too. The django admin and the honeypot html code are not the same anymore. But when I start adjusting it for the latest django, I'll break older tests. I need help on this one 😔

@GitRon
Copy link

GitRon commented Dec 2, 2020

I think I'll go with commenting out the test for now. The problem lives in the dev branch and has nothing to do with my two branches. Ok for you?

@GitRon
Copy link

GitRon commented Dec 4, 2020

@9mido @dmpayton All fixed 😃

@SuperMasterBlasterLaser

@dmpayton @9mido Any progress on new version?

@dimadk24
Copy link

Any updates on a new release? I see that many fixes were done in the dev branch, but it's not released to pypi

@9mido
Copy link

9mido commented Feb 4, 2021

I have no idea. I don't maintain this project. I am like all of you just patiently waiting.

Just a suggestion but maybe this should become a jazzband project so that we could get more frequent updates?

@GitRon
Copy link

GitRon commented Mar 8, 2021

@9mido Might be an idea to move it to Jazzband. But can you do anything in this direction?

@9mido
Copy link

9mido commented Mar 8, 2021

I don't think I can do anything @GitRon. I think @dmpayton can only do something since he is the owner of this github library. Not sure exactly.

https://docs.github.com/en/github/administering-a-repository/transferring-a-repository

https://jazzband.co/about/guidelines

Someone may be able to fork the github library / download the files to their desktop and upload them to their own repository and do it, but we should at least give @dmpayton a say in what direction to go in first. Personally, I would love if all github libraries I used for django were on jazzband. It makes things more open source and depends less on 1 person calling the shots or disappearing. The django community becomes the entity to maintain the github library instead. @dmpayton can still be an adviser if he wants to.

I think it would be in @dmpayton's best interest since he is so busy and all of our best interests to port this library over to jazzband. If @dmpayton does not reply soon then we may have no choice and someone needs to fork it / upload it to their own github repository but then we still run into the same problem of 1 person calling the shots or disappearing. With jazzband, you do not have these problems. Otherwise if jazzband is not what @dmpayton wants, please step up your game. It has been almost 1 year now.

According to #jazzband on IRC they say we need to submit a proposal for this project to move to jazzband. https://github.com/jazzband-roadies/help/issues

@VentGrey
Copy link

Any updates on this?

@GitRon
Copy link

GitRon commented Jun 1, 2021

@dmpayton Any chance that you could find some time for handling this project? Either yourself or maybe we pass the project to jazzband like @9mido suggested?

@GitRon
Copy link

GitRon commented Sep 23, 2021

I will stop using this package now. Really sad but it will break with the next major release of django 🤷

@9mido
Copy link

9mido commented Sep 23, 2021

@GitRon I made a proposal in jazzband. Hopefully somebody there could do something. I agree it is sad that this project seems to be no longer maintained.

@blag
Copy link

blag commented Nov 26, 2021

Hi, I have forked this project, included a few PRs (including @GitRon's Django 3 updates), and released a package to PyPI. You might have better luck with my fork.

@shawnngtq
Copy link

shawnngtq commented Dec 11, 2021

It's broken as of Django 4.0. What should we do folks?

Love this project, so helpful

#86

@blag
Copy link

blag commented Dec 12, 2021

@shawnngtq You can use my fork which is available from PyPI as django-admin-honeypot-updated-2021. My fork should work with Django 4.0, and if it doesn't I'd be happy to accept a PR to fix it.

@IrfanAkram5
Copy link

Hi,
Is there any update on the project status - will there be an update on pypi? I am using Django 4.0. It appears most(all?) issues are fixed via the develop branch - I installed via pip and simply went through the errors as I hit them comparing the broken modules to the develop branch.
Only issue is no ip address( not even 127.0.0.1 ) in the db but I haven't looked into it further yet or tried the suggestion in the docs yet(it may be my nginx settings).

This is a good project filling a practical need. Hopefully there will be a updated release on pypi at some point.

@hanckmann
Copy link

Just checking (and bumping) this thread.
Has there been an update from the developer and/or jazzband?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests