search

hkorzeniewski / url-shortener

URL Shortener is a Django web application that provides two main functions: encoding a long URL to a short URL and decoding a short URL back to its original long URL. This is achieved using the pyshorteners library for URL shortening and caching with Django's cache framework.
0 stars 0 forks source link

2nd round #3

Closed grski closed 1 year ago

grski commented 1 year ago

Dalej zacommitowany plik sqlite3 z bazą danych DEBUG = True na sztywno zamiast użycia os.environ.get i pobrania ze zmiennych środowiskowych REDIS_URL a potem w CACHES redefinicja na sztywno _pycache__ zacomitowane redis_instance = redis.StrictRedis(settings.REDIS_URL, db=0) -> zamiast tego django.cache po prostu type hinty, poczytaj i używaj brak komentarzy, docstringów etc zakomentowane brudne printy czy obecnie wyprodukowany przez ciebie kod zwraca https://github.com/hkorzeniewski/url-shortener/blob/main/urlshortener/views.py -> jako krótki url np. shortener.url/XDFASF? Poczytaj o JsonResponse

  1. masz djangorestframework a nawet go chyba nie używasz
  2. pymemcache - potrzebne?

spróbuj samodzielnie zaimplementować generację sluga/id dla skróconego urla bez wykorzystania pyshorteners

grski commented 1 year ago

image trzymaj się struktury commitów i pracuj na branczach

[feature/docker] krótki opis [feature/api-view] krótki opis

grski commented 1 year ago

Mostly solved by: https://github.com/hkorzeniewski/url-shortener/pull/4 @hkorzeniewski except for the issue with commit message