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

feat: Adicionando rss para as rotas da api #1566

Closed
wants to merge 1 commit into from

Conversation

eletroswing
Copy link
Contributor

Baseado na discussão do pull adding rss to relevant route and changing default path to starts with rss, foi adicionado de forma simples um feed rss para a rota de relevantes.

Os caminhos padrões não foram modificados, para garantir compatibilidade com a api já existente.
As rotas novas são simplificadas:
/rss/recentes
/rss/relevantes
Além da não modificada:
/relevantes/rss

Mais:
essas rotas agora são capazes também de aceitar per_page e page, com padrão definido respectivamente em 30 e 1(default da api)

Como dito na discussão informada anteriormente, foi criada uma consulta única para a página de rss relevantes na forma de uma nova estrategia chamada 'relevantes_rss', podendo ser modificada posteriormente.

Nenhum teste foi escrito para essa implementação, a fim de (se necessário), melhora-lá antes da escrita dos testes.

@aprendendofelipe

adding rss to relevant route and changing default path to starts with rss
Copy link

vercel bot commented Dec 12, 2023

@eletroswing is attempting to deploy a commit to the TabNews Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Collaborator

@aprendendofelipe aprendendofelipe left a comment

Choose a reason for hiding this comment

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

Fala @eletroswing, obrigado pelo PR! 💪

Quais e como os problemas levantados no #1480 foram resolvidos nesse PR?

Pelo que entendi, você está usando apenas a consulta de recentes, e reordenando os itens usando a função que classifica os comentários (como era a primeira versão de relevantes do TabNews), ou seja, os três endpoints retornarão os mesmos conteúdos, apenas alterando a ordenação. Além disso, adicionou paginação no RSS.

Por favor, me corrija se eu deixei escapar algo, e explique se existe fundamento para essas decisões no contexto de feed RSS. E se conseguiu sanar outras dúvidas levantadas no #1480, compartilhe o que você descobriu 🤝

Mas o principal é não fugir do seu objetivo. Qual problema está tentando resolver? Naquele PR você citou a sugestão do @filipedeschamps de criar uma alternativa para só mostrar itens relevantes no Feed RSS, algo que não está sendo feito nesse PR.

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

Successfully merging this pull request may close these issues.

2 participants