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

[ISSUE-72] - Mantendo paginas já carregadas após recomposição #73

Merged

Conversation

gabrielbmoro
Copy link
Collaborator

Descrição

Passos para reproduzir o problema:

  • Rolar alguma seção (stream carousel) várias vezes;
  • Selecionar o stream para acessar a tela de detalhes;
  • Voltar da tela de detalhes para a tela anterior.

Resultado Atual

  • Posição do scroll do carousel fica no início da lista, pois a lista foi recarregada.

Resultado Esperado

  • Posição do scroll do carousel tem que estar aonde o usuário deixou;

Checklist

  • Os testes foram executados e passaram com sucesso.
  • As alterações de código seguem as diretrizes de estilo do projeto.
  • Foram adicionados testes, se aplicável.
  • Se inscreveu no canal?😛

Issues Relacionadas

@gabrielbmoro gabrielbmoro requested a review from a team as a code owner July 26, 2023 01:00
@gabrielbmoro gabrielbmoro changed the title [ISSUE-72] - Keep pagination data after recomposition [ISSUE-72] - Mantendo paginas já carregadas após recomposição Jul 26, 2023
highlightBanner = getHighlightBanner(latest)
)
}
}

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Removido por causa que era uma request duplicada (já realizada anteriormente no combine)

override fun onCreate(owner: LifecycleOwner) {
super.onCreate(owner)

init {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

onCreate estava sendo chamado mais de uma vez, trigando ao retornar da tela. Para realizar a chamada uma única vez, eu movi para o init block

@gabrielbmoro gabrielbmoro merged commit 1f0c969 into master Aug 14, 2023
@gabrielbmoro gabrielbmoro deleted the feature/72-dados-de-paginao-esto-sendo-perdidos- branch August 14, 2023 12:19
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.

None yet

2 participants