101Loop / 101Loop/drf-user

Remove drf-addons dependency.

Open
#110 0 comments 0 reactions 0 assignees View on GitHub
enhancement good first issue
Dominant language
Python
Stars
158
Forks
69
PR merge metrics
No merged PRs in 30d

Description

### Describe the feature you'd like

We need to remove [drf-addons](https://github.com/101Loop/drf-addons) dependency from this project as we're only using it's minor features only.

We will rewrite these 2 methods:
- get_client_ip
- send_message

And we'll remove usage of `JsonResponse`. Use DRF's `Response` or something.

Please write tests. 🙏🏼
### Is your feature request related to a problem?

### Your Environment

- `drf-user` version(s) (`drf-user --version`:
- Operating System(s):
- Python version(s):

### Additional context

Contributor guide

Open the contributing guide

Research direction

Find where drf-addons is imported and used, likely in views or utils. Locate the get_client_ip and send_message methods and the JsonResponse usage. Write replacements using Django REST Framework's Response and standard Python libraries for IP detection and messaging. Ensure tests cover the new implementations and remove the dependency from requirements.

Written by the indexing model from the issue text.

Assessment

Tech stack
django, python
Domain
api, backend
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.