-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add rich link preview to the shop #182
Comments
Should we use this issue to show previews on more areas of the app? I can imagine some areas where we can use specific previews:
We should also probably have a "default" preview, that's shown on all other pages (e.g. the dashboard). In that case, I see two possibilities:
I think this is a really interesting issue, and a great improvement, with lots of possibilities! It is pretty restrained though, we pretty much just get an image, the title and a quick description. Here is a good resource to get started on implementing this. |
backend requirements will be implemented on #177 |
@juramos-2020 @rafachadud With this, we now have the possibility to display the product page for guests as well! I was thinking we could show the same thing, but when the user clicks "Buy", they're redirected to the Login page. What do you think? Is that ok? |
@lucca65 I had all the structure ready for this, and I couldn't get it to work... upon further research I found out that the Facebook, Twitter and Whatsapp crawlers don't execute JS. From there, I found two options:
Number 1 is pretty unrealistic for us at this point IMO, it would complicate our stack by a lot, and I'm not really sure what would happen with pages that are for logged-in users only. So that leaves us with option 2... do you think that's doable, or should we put this on the icebox and maybe revisit it at a later date? What we can do right now is just add some generic Cambiatus-related stuff (title, image, description) to the |
I think this is great. It would be more attractive for the user to enter and be part of the 😉👍 |
@lucca65 did you get a chance to read my comment above? How should we proceed? |
I think we should go for the second solution, with a preview on Elixir and a redirect on Nginx! Maybe we should transfer this to the backend? looks like the frontend will do nothing |
+1 on transferring to the backend! |
Olá pessoal! vocês já tem tudo o que precisam de Design para essa implementação? Vocês vão precisar das telas? |
Ju acho que essa funcionalidade não tem nenhuma tela. você tinha pensado em alguma coisa? |
Tinha pensado em algo assim, mas tranquilo se já fizeram 😉👍 Se precisarem de alguma ajuda para as telas de compartilhamento de perfis é só falar 😉👍 |
Ju não tenho permissão pra ver esse link |
Ops, tranquilo. Acabei de mandar um convite para vocês e expliquei ainda pouco no Slack (aqui nessa conversa) sobre a migração de projetos que estamos fazendo no Figma. 😉👍 |
Ju tem um elemento que não colocamos na loja, de ter o preço depois da descrição. @MatheusBuss pode criar uma issue pra colocarmos isso? |
Acho que isso é algo que vamos tratar na cambiatus/lambda-functions#2. Lá já tem alguma descrição do que pensamos em fazer. É legal já pensar em como vão ficar as imagens sim 👍. |
não vamos tratar da imagem? pq aqui acho que é mais colocar o preço após a descrição do produto mesmo |
Podemos fazer isso sim, só teria que ver se o preço não precisaria vir antes da descrição, por que se for no final pode ficar cortado. |
Criado #245 |
@juramos-2020 oq acha? Realmente é bem comum os celulares cortarem por falta de espaço |
Poderia ter uma quebra de linha? |
Não dá Ju, infelizmente só podemos ter uma linha que é gerenciada pelo app que for mostrar (Instagram, zap, etc) Uma outra ideia seria colocar o preço na frente da descrição, acho que inclusive podemos colocá-la em negrito, certo @MatheusBuss? |
Imaginava que funcionaria bem o negrito, mas pelo jeito nem sempre. O negrito pode fucionar, mas depende mais da plataforma que vai mostrar a preview pro usuário. Pelo menos é o que falaram nessa thread. Podemos dar uma testada nisso. |
entendi obrigado pela pesquisa @MatheusBuss! O que acha @juramos-2020? Com ou sem negrito, a proposta seria colocar o preço antes da descrição |
Tudo bem sim se o preço vir antes. Também fica interessante nesse formato 😉👍 |
Is your feature request related to a problem? Please describe.
When users share Cambiatus links it is shown plain, without a preview
Describe the solution you'd like
Implement a preview with the product image and the community logo. Maybe with some text like "${user} is selling ${product} on Cambiatus, check it out"
Describe alternatives you've considered
N/A
Additional context
N/A
Screen shots

The text was updated successfully, but these errors were encountered: