101Loop / 101Loop/drf-user

Remove drf-addons dependency.

Aperta
#110 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
enhancement good first issue
Lingua principale
Python
Stelle
158
Fork
69
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

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.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
django, python
Ambito
api, backend
Tipo di issue
Refactoring
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
55/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.