101Loop / 101Loop/drf-user

Remove drf-addons dependency.

Ouverte
#110 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
enhancement good first issue
Langage dominant
Python
Étoiles
158
Forks
69
Métriques de merge des PR
Aucune PR mergée en 30 j

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

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

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.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
django, python
Domaine
api, backend
Type d'issue
Refactorisation
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
55/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.