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.

Sign up for GitHub

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

Jump to bottom

Adiciona a propriedade owner_username no conteúdo e remove username #619

Closed
filipedeschamps opened this issue Aug 5, 2022 · 4 comments
Closed

Adiciona a propriedade owner_username no conteúdo e remove username #619

filipedeschamps opened this issue Aug 5, 2022 · 4 comments
Labels
back Envolve modificações no backend front Envolve modificações no frontend
Milestone
Milestone 5: Lanç...

Comments

@filipedeschamps
Copy link
Owner

filipedeschamps commented Aug 5, 2022 •
edited

Contexto

Nós precisamos adicionar a propriedade owner_username no objeto content para fazer par com o owner_id. Por compatibilidade, ainda iremos manter o username, mas que após notificação pública, iremos remover e fazer uma breaking change na interface pública. Estamos nos dando o luxo de fazer isso dessa forma pela API estar em beta.

O motivo desta implementação é semântico, uma vez que um conteúdo não possui username, ele possui no máximo um dono e esse dono é quem possui um username.

Com a implementação no backend feita, precisamos também alterar os componentes em interface como ContentList e Content. Muito cuidado nessa parte, principalmente nas horas que é feito a edição do conteúdo (e é enviado um PATCH para .../contents/[owner_username]/[slug])

Na publicação abaixo anunciamos que dia 19 de Agosto iremos remover a propriedade username:

https://www.tabnews.com.br/filipedeschamps/breaking-change-na-api-do-tabnews-sobre-o-objeto-content

Execução
@filipedeschamps filipedeschamps added front Envolve modificações no frontend back Envolve modificações no backend labels Aug 5, 2022
@filipedeschamps filipedeschamps mentioned this issue Aug 5, 2022
Closed
@filipedeschamps
Copy link
Owner Author

Fechado por #618

@filipedeschamps filipedeschamps changed the title Adiciona a propriedade owner_username no conteúdo Adiciona a propriedade owner_username no conteúdo e remove username Aug 5, 2022
@filipedeschamps
Copy link
Owner Author

filipedeschamps commented Aug 5, 2022 •
edited

Reabri a issue para contemplar nela também a parte de remover o username no content 🤝

@filipedeschamps filipedeschamps mentioned this issue Aug 19, 2022
Closed
@filipedeschamps
Copy link
Owner Author

Fechado por #618 e #660

@coffeeispower
Copy link
Contributor

aaaaa quebrou meu app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
back Envolve modificações no backend front Envolve modificações no frontend
Projects
None yet
Milestone
Milestone 5: Lançamento
Development

No branches or pull requests
2 participants
@filipedeschamps @coffeeispower