Skip to content

show ally logo if exists#426

Merged
sdelquin merged 3 commits intopythoncanarias:mainfrom
CrisKrus:420-logo-is-not-mandatory
Oct 26, 2021
Merged

show ally logo if exists#426
sdelquin merged 3 commits intopythoncanarias:mainfrom
CrisKrus:420-logo-is-not-mandatory

Conversation

@CrisKrus
Copy link
Copy Markdown
Contributor

It shows the ally logo if the ally has one
#420

@sdelquin
Copy link
Copy Markdown
Contributor

Yo creo que se podría usar el filtro default de Django y en el caso de que no hubiera photo de aliado usar esta ally-placeholder.png:

ally-placeholder

@sdelquin sdelquin added hacktoberfest-accepted HacktoberFest Accepted hacktoberfest Aptas para Hacktoberfest labels Oct 21, 2021
@CrisKrus
Copy link
Copy Markdown
Contributor Author

Yo creo que se podría usar el filtro default de Django y en el caso de que no hubiera photo de aliado usar esta ally-placeholder.png:

ally-placeholder

No estoy seguro de que se pueda hacer mediante el default ya que hay que concatenar varios render templates. ¿La solución propuesta vale?

@sdelquin
Copy link
Copy Markdown
Contributor

Genial @CrisKrus. Muchas gracias por tus aportaciones!

@sdelquin sdelquin merged commit ce10c55 into pythoncanarias:main Oct 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest Aptas para Hacktoberfest hacktoberfest-accepted HacktoberFest Accepted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants