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

Rozszerzony opis firmy #1363

Closed
MatPerowicz opened this issue Dec 7, 2021 · 11 comments
Closed

Rozszerzony opis firmy #1363

MatPerowicz opened this issue Dec 7, 2021 · 11 comments

Comments

@MatPerowicz
Copy link
Contributor

MatPerowicz commented Dec 7, 2021

Chcielibyśmy, aby firma miła możliwość rozszerzenia opisu firmy. Oprócz tekstu mógłby się tam znajdować również logotyp, link do strony producenta i logotypy marek firmy.

Należałoby dodać w CMS pole „Rozszerzony opis firmy” i tam możliwość dodawania logotypu oraz linku do strony. Logotyp powinien być hiperłączem. Po kliknięciu w logo przenosi nas na stronę firmy.

Dodatkowo należałoby wprowadzić możliwość dodawania logotypów do marek firmy. Mamy już opcję dodawania marek firmy, trzeba tylko dodać opcję wstawiania logotypów i linków przekierowujących. Tutaj również logo powinno być przekierowaniem do strony konkretnek marki.

Wizualizacja:

image

@MatPerowicz
Copy link
Contributor Author

@mik-laj @rodkiewicz @AdamWalkowski Prośba o nadanie temu priorytetu. Potrzebujemy tego zarówno w aplikacji, jaki i w wyszukiwarce na stronie głównej.

@mik-laj
Copy link
Member

mik-laj commented Aug 18, 2023

Brzmi dla mnie sensownie. Warto pamiętać, że logotypy powinny być przechowywane w S3. Ja mogę utworzyć nowe wiaderko, jak będzie potrzeba.

@rodkiewicz
Copy link
Contributor

rodkiewicz commented Aug 19, 2023

Propozcyja implementacji #3120 @mik-laj daj znac co sądzisz

@MatPerowicz
Copy link
Contributor Author

Mega dzięki @rodkiewicz!

@MatPerowicz
Copy link
Contributor Author

@mik-laj możesz na to spojrzeć 🙂?

@MatPerowicz
Copy link
Contributor Author

@mik-laj próbuję dodać logotyp i link
image

pojawia się taki błąd
image

Prośba o sprawdzenie dlaczego to może nie działać.

@MatPerowicz
Copy link
Contributor Author

Udało się dodać link. Błąd powoduje dodanie grafik.

@MatPerowicz
Copy link
Contributor Author

@mik-laj daj proszę znać czy to już działa :)

@WezSieTato
Copy link
Member

WezSieTato commented Jan 16, 2024

Na moje API nie wygląda tak jak powinno. Z tego co zrozumiałem potrzebujemy listy obiektów, które mają URL loga, URL strony, nazwa (przyda się dla VoiceOver).
Aktualnie zwracamy to na dwa różne sposoby:
Dla kodu https://www.pola-app.pl/a/v4/get_by_code?code=9771644705002&device_id=%220%22 zwracamy official_url i logotype_url zagnieżdzone razem z innymi informacjami o firmie.

Dla kodu https://www.pola-app.pl/a/v4/get_by_code?code=5900820002206&device_id=0 jest cała lista obiektów name i logotype_url na poziomie roota

Według mnie to powinno zawsze przychodzić w liście na poziomie informacji o firmie. A obiekt najlepiej jakby wyglądał tak:

{
 "name" : "Klub Jagielloński",
"website_url": "https://klubjagiellonski.pl/",
"logotype_url": "https://kj-pola-app-company-logotype.s3.amazonaws.com/logo/2023/12/17/KJ_ksiega_05__Logo_podstawowe_4.png"
}

Wszystkie pola wymagane. Jak jakiegoś brakuje to obiekt nie przychodzi. Aktualne pole co przychodzą z tym związane bym wywalił.

@MatPerowicz
Copy link
Contributor Author

@mik-laj prośba o dodanie możliwości załączenia logotypów i linków przekierowujących do marek danej firmy oto przykład:

image
image

@WezSieTato
Copy link
Member

Zadanie zamykam. Rozszerzony opis jest już zaimplementowany. Dodatkowe zmiany, które przydałoby się dodać opisałem tu:
#3820
#3821

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

No branches or pull requests

4 participants