Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dodanie pola website_url do obiektu brand dla endpointu get_by_code #3821

Open
WezSieTato opened this issue May 10, 2024 · 2 comments
Open

Comments

@WezSieTato
Copy link
Member

Opis

W celu wzbogacenia informacji dostarczanych przez nasze API, wymagane jest dodanie nowego pola website_url do każdego obiektu brand w strukturze danych JSON. Pole to będzie zawierało URL do oficjalnej strony internetowej danej marki, co ułatwi użytkownikom uzyskanie dodatkowych informacji o marce.

Szczegóły implementacji:

Pole website_url: Każdy obiekt brand w odpowiedzi JSON będzie rozszerzony o nowe pole website_url, które będzie przechowywało URL jako string.
Przykładowa wartość dla marki Pola:

{
  "name": "Pola. Zabierz ją na zakupy",
  "logotype_url": "https://kj-pola-app-company-logotype.s3.amazonaws.com/brand-logotype/2024/03/25/pola_logo_1.png",
  "website_url": "https://www.pola-app.pl"
}

Pole to powinno być wymagane w obiekcie brand. Opis zmian po stronie CMS dostarczy @MatPerowicz

@cincinek
Copy link

@WezSieTato
Dane dla get_by_code generowane są w metodzie get_result_from_code w pliku logic.py. W tej metodzie dane na temat marki, są tylko tworzone dane dla firm

            if product_company:
                companies.append(product_company)
                result['name'] = product_company.common_name or product_company.official_name or product_company.name
            if brand_company:
                companies.append(brand_company)

Czy w zadaniu chodzi o to, żeby generalnie zmodyfikować get_result_from_code, tak aby zwracała informację na temat marki io rozbudowaniu dodać pole website_urldo modelu brand?
Czy może coś pomieszałem i zmiany są potrzebne gdzieś indziej w kodzie?

@WezSieTato
Copy link
Member Author

Nie wiem jak to jest zaimplementowane. Ja jestem tylko konsumentem tego API :)
Mi chodzi o to, żeby brand miało dodatkowe pole. Jeśli o chodzi o zmodyfikowania struktury company, żeby zwracała informacje o markach to na to też jest zadanie #3820

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants