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

Adiciona página /classificados em recentes e no perfil dos usuários #1749

Merged
merged 2 commits into from
Jul 17, 2024

Conversation

aprendendofelipe
Copy link
Collaborator

Draft da tapa 3e citada em #1491 (comment).

Mudanças realizadas

Por enquanto adicionei as abas de classificados junto com as abas de recentes e dos usuários. Assim não temos problemas com o Header:

image image

Daí na aba "Todos", até pelo nome dela, acho que faz sentido continuar mostrando tudo, inclusive os ads, mas aí preciso de opiniões sobre alguns pontos, e por isso vou deixar o PR como draft.

Não sei se faz sentido mostrar um anúncio destacado nessa página que já pode conter anúncios na listagem. Vejam:

image

E, se os anúncios vão aparecer listados junto com os outros conteúdos, acham que vale a pena diferenciar eles de alguma forma? Por exemplo como no estudo anterior:

Conteúdo patrocinado

Outra questão que já trouxe para cá é sobre os cálculos de ganhos de TabCoins (diários e ao publicar). Os ganhos se baseiam em qualificações recebidas anteriormente. Eu acredito que só devemos considerar as qualificações passadas para os tipos content. Sem considerar as qualificações dos anúncios. Concordam?

Tipo de mudança

  • Nova funcionalidade

Copy link

vercel bot commented Jul 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tabnews ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 17, 2024 11:50pm

@filipedeschamps
Copy link
Owner

E, se os anúncios vão aparecer listados junto com os outros conteúdos, acham que vale a pena diferenciar eles de alguma forma? Por exemplo como no estudo anterior:

Excelente ponto! Acho que deveria sim, assim como o pitch no futuro também vai ser diferenciado 🤝 Agora, fico na dúvida entre Patrocinado (sponsored) e Anúncio (ad).

Eu acredito que só devemos considerar as qualificações passadas para os tipos content. Sem considerar as qualificações dos anúncios. Concordam?

Concordo 100%

Copy link
Collaborator

@Rafatcb Rafatcb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Não sei se faz sentido mostrar um anúncio destacado nessa página que já pode conter anúncios na listagem.

Acho que faz sentido sim, visto que é o anúncio que está recebendo o destaque.

E, se os anúncios vão aparecer listados junto com os outros conteúdos, acham que vale a pena diferenciar eles de alguma forma?

Assim como o Filipe, também acho que faz sentido. Isso ajuda o usuário a entender o que está vendo sem precisar lembrar do contexto maior, isto é, identificar que as publicações são patrocinadas sem precisar ver que está na página "Classificados".

Outra questão que já trouxe para cá é sobre os cálculos de ganhos de TabCoins (diários e ao publicar). Os ganhos se baseiam em qualificações recebidas anteriormente. Eu acredito que só devemos considerar as qualificações passadas para os tipos content. Sem considerar as qualificações dos anúncios. Concordam?

Também concordo. Pode criar testes para garantir isso (imagino que você estava aguardando se nós concordaríamos ou não, antes de criar os testes).


Agora, fico na dúvida entre Patrocinado (sponsored) e Anúncio (ad).

Considero "Patrocinado" melhor por ser fácil de associar que o anúncio criado através da página /publicar (PR #1747), que tem um Checkbox "Criar como publicação patrocinada", é justamente o que o usuário está vendo no momento.

Se não existirem outros tipos de anúncio, isto é, todos serem criados pelo mesmo lugar e terem o mesmo tipo de comportamento (como estamos fazendo agora), então não tem problema chamar de "Anúncio" (e talvez seja até melhor, por ser mais claro para quem não conhece). Nesse caso, acredito que seria necessário mudar o PR #1747 também para usarmos apenas a palavra "Anúncio" ao invés do termo "Publicação patrocinada".

Base automatically changed from feat/filter-content-type to main July 16, 2024 21:07
@aprendendofelipe aprendendofelipe force-pushed the feat/classificados-page branch from fb98e40 to 1e8dc91 Compare July 16, 2024 21:17
@aprendendofelipe
Copy link
Collaborator Author

Ainda faltam testes provando que os anúncios não serão considerados nos ganhos de TabCoins, mas, se acharem que os demais PRs estão prontos para o merge, acho que compensa enviar esse aqui também. Acho que faz sentido fazer merge na seguinte ordem, sendo que os dois últimos é bom fazer o merge um logo em seguida do outro:

Daí só fica pendente os testes dos ganhos de TabCoins e identificar as patrocinadas nas listas de conteúdos que elas forem aparecer.

Vou disponibilizar para revisão

@aprendendofelipe aprendendofelipe marked this pull request as ready for review July 17, 2024 21:06
Copy link
Collaborator

@Rafatcb Rafatcb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Meu único comentário sobre este PR era sobre os testes mesmo (eu já havia revisado tudo).

Ainda preciso revisar o #1751. Sua sugestão faz sentido para mim, e mesclando o #1751 já não teria nenhum impeditivo para mesclar os outros dois também.

@aprendendofelipe aprendendofelipe force-pushed the feat/classificados-page branch from 1e8dc91 to aba3580 Compare July 17, 2024 23:48
@aprendendofelipe aprendendofelipe merged commit d6cc266 into main Jul 17, 2024
7 checks passed
@aprendendofelipe aprendendofelipe deleted the feat/classificados-page branch July 17, 2024 23:52
@Rafatcb Rafatcb added front Envolve modificações no frontend novo recurso Nova funcionalidade/recurso labels Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
front Envolve modificações no frontend novo recurso Nova funcionalidade/recurso
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants