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

Breaking Change: alterar strategy de best para relevant #658

Closed
filipedeschamps opened this issue Aug 19, 2022 · 2 comments
Closed

Breaking Change: alterar strategy de best para relevant #658

filipedeschamps opened this issue Aug 19, 2022 · 2 comments
Labels
back Envolve modificações no backend front Envolve modificações no frontend

Comments

@filipedeschamps
Copy link
Owner

filipedeschamps commented Aug 19, 2022

Contexto

Hoje o endpoint /api/v1/contents aceita 3 tipos de estratégia:

Estratégia Descrição
/api/v1/contents?strategy=best Ordena pelos conteúdos mais relevantes.
/api/v1/contents?strategy=new Ordena dos conteúdos mais novos para os mais antigos.
/api/v1/contents?strategy=old Ordena dos conteúdos mais antigos para os mais novos.

Mas o nome da estratégia best não acredito ser o melhor e deveríamos mudar o nome para relevant, pois não são os melhores conteúdos e sim os mais relevantes para aquele momento. Isto fará um par muito melhor lá no menu principal superior, pois poderíamos colocar os itens Relevantes e Recentes.

Fora que a estratégia best poderia ser usada para ordenar as publicações apenas pelas tabcoins para um dia em específico. Por exemplo, mostrar qual foi a publicação mais bem avaliada X, ou até da semana, do mês, etc.


Anúncio da Breaking Change: https://www.tabnews.com.br/filipedeschamps/breaking-change-na-api-remocao-da-estrategia-best-e-informacoes-adicionais-de-parent-no-objeto-content

Execução

Por hora, copiar para a estratégia relevant o que a estratégia best está fazendo e manter as duas rodando ao mesmo tempo, depois anunciar a breaking change, e por fim remover a best até que ela seja implementada da forma nova proposta no contexto.

@filipedeschamps filipedeschamps added front Envolve modificações no frontend back Envolve modificações no backend labels Aug 19, 2022
This was referenced Aug 19, 2022
@filipedeschamps
Copy link
Owner Author

Primeiro passo dado que foi o merge do PR #662 e que expõe a estratégia relevant. Agora só falta anunciar a breaking change e por enquanto remover a best 👍

@filipedeschamps filipedeschamps changed the title Breaking change: alterar strategy de best para relevant Breaking Change: alterar strategy de best para relevant Aug 24, 2022
@filipedeschamps
Copy link
Owner Author

Fechado por #662 e #681

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back Envolve modificações no backend front Envolve modificações no frontend
Projects
None yet
Development

No branches or pull requests

1 participant