-
-
Notifications
You must be signed in to change notification settings - Fork 81
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
[change!] Changed the command to delete_old_radiusbatch_users #294 #295
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @praptisharma28 👍
e52b3c1
to
da4ecd5
Compare
@nemesifier what corrections are needed to be done here now? |
Nothing here, just the build on master seems failing, we'll need to resolve that before merging. |
@praptisharma28 From the initial impression it looks like a python dependency issue. Look a this line in the logs https://github.com/openwisp/docker-openwisp/pull/295/checks#step:6:1391 |
Yeah, u r right the error message indicates that the need for netaddr~=0.8 or netaddr~=0.8.0 is not fulfilled, but idk how to resolve it. @pandafy |
Updated the command. Related to openwisp/openwisp-radius#502 .This change ensures proper functionality and maintains consistency.
Closes #294