StarWiki App Concluído #21
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Saved favorites did not have the right icon on startup. Favorite filter used to override search query. Search -> Favorites ON -> Erase query -> Favorites OFF would yield incorrect results. These issues have now been resolved.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Todos os requisitos propostos pelo desafio foram atendidos.
Observações:
Por "erro" em caso de requisição à API também se entende falta de conexão à internet quando favoritando. Essas requisições também são reenviadas em sessão futura.
Requisições reenviadas em nova sessão do app são realizadas automaticamente, utilizando o id selecionado pelo usuário em sessão anterior. Metade dessas requisições automáticas adiciona o header
Prefercomstatus=400.Por
"A busca pelo nome do planeta e da espécie deve ser feita em paralelo."entendeu-se que se espera requisições à API para os objetosplanetespeciesrespectivos ao personagem. Para a versão atual, apenas a url desses objetos é preservada em banco de dados, na tabela dos objetosPeople. Espera-se que em uma extensão desse app os modelosplanetespeciessejam extendidos e integrados ao banco de dados em tabelas próprias.Para clarificar: devido a um erro encontrado e o tempo já tomado para responder o desafio, a exibição dos personagens na homepage não utiliza lazy loading mencionado no segundo commit.