101Loop / 101Loop/drf-user

Remove drf-addons dependency.

Offen
#110 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement good first issue
Vorherrschende Sprache
Python
Sterne
158
Forks
69
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

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.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
django, python
Bereich
api, backend
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.