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: expor endpoint /parent e remover propriedades de parent do content #680

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

Comments

@filipedeschamps
Copy link
Owner

filipedeschamps commented Aug 24, 2022

Contexto

O objeto content possui informações do seu conteúdo parent, que muitas vezes pode ter sido deletado. O objeto possui a referência parent_id e essa deve permanecer, mas parent_title, parent_slug e parent_username hoje revelam informações de um conteúdo que já foi deletado. Isto foi reportado inclusive nessa issue recente #654 de @CarlosZiegler .

Então devemos remover do content as propriedades parent_title, parent_slug e parent_username e expor em um novo endpoint /parent todo o objeto do conteúdo pai, onde se deletado, deve continuar enviando o objeto, mas com informações sensíveis mascaradas, como:

  • title
  • body
  • slug
  • source_url

O mesmo comportamento deve acontecer para o novo endpoint /root.


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

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

Fechado por #676 e #677

@filipedeschamps
Copy link
Owner Author

Resultado no P50 da latência do /api/v1/contents?strategy=best|relevant após os dois merges relacionados nessa issue:

image

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