101Loop / 101Loop/drf-user

Remove drf-addons dependency.

Đang mở
#110 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
enhancement good first issue
Ngôn ngữ chính
Python
Star
158
Fork
69
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

### 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

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

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.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
django, python
Lĩnh vực
api, backend
Loại issue
Tái cấu trúc
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Đặc tả rõ ràng
Mức phù hợp với người mới
55/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.