Added price and creator to description #248
Conversation
|
Eu fiz alguns testes e tenho algumas coisas pra mostrar @juramos-2020, pra complementar a conversa que tivemos no #182.
Fica aqui uma sugestão de como pode ficar (Ainda vamos dar uma olhada nas imagens na cambiatus/lambda-functions#2). O que acham @juramos-2020 e @lucca65?
|
|
Oi Matheus achei ótima a sugestão. |
|
Oi @juramos-2020 , tem sim, só que estamos deixando pra ver essa parte das imagens na issue cambiatus/lambda-functions#2, porque daí vamos ter que investigar porque algumas imagens não aparecem e como fazer pra deixar todas elas padronizadas no mesmo tamanho. |
|
Beleza então 😉👍 |
| get_currency = fn community_subdomain -> | ||
| case Commune.find_community(%{}, %{subdomain: community_subdomain}, %{}) do | ||
| {:ok, community} -> | ||
| String.slice(community.symbol, 2, 7) | ||
|
|
||
| {:error, reason} -> | ||
| {:error, reason} | ||
| end | ||
| end |
There was a problem hiding this comment.
Why not get the value directly instead of making a function?
currency = case Commune.find_community(%{}, %{subdomain: community_subdomain}, %{}) do
...
Also doesn't work :/
|
Co-authored-by: Julien Lucca <julienlucca@gmail.com>
There was a problem hiding this comment.
Legal bro, essa issue não para de crescer e acho que podemos resolver essa última parte nesse PR mesmo. Tá faltando traduções adequadas para os textos que são exibidos. Podemos ter duas fontes pro idioma:
o header accept-language se tiver presente ou o idioma configurado do usuário em user.language. Se nenhum dos dois estiver preenchido aí sim vamos pro default que é en-US
| {:error, _} -> | ||
| "" | ||
| end | ||
| end | ||
|
|
||
| def create_description(%{description: description, creator: creator}) do | ||
| "Vendido por #{creator} - #{md_to_txt(description)}" |
There was a problem hiding this comment.
Não tenho certeza se o padrão deve ser português. Temos um mecanismo de tradução colocado que você pode ver nos nossos emails. Arquivos dentro de /priv/gettext/ e no lib/cambiatus_web/email.ex podem servir de referência para como fazer!
What issue does this PR close
Closes #245
Changes Proposed ( a list of new changes introduced by this PR)
Atualizar a funcionalidade do rich link preview para mostrar o preço dos produtos no título da preview
Link do Figma