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

Feedback da avaliação #1

Open
19 of 20 tasks
paaatcha opened this issue Feb 9, 2023 · 11 comments
Open
19 of 20 tasks

Feedback da avaliação #1

paaatcha opened this issue Feb 9, 2023 · 11 comments

Comments

@paaatcha
Copy link
Member

paaatcha commented Feb 9, 2023

Avaliação do projeto

  • Integrantes:
    • Sophie Dilhon

Neste documento você encontra a avaliação geral do seu projeto. Primeiro avalio os requisitos que foram cumpridos. Na sequência, faço comentários sobre
o tema, usabilidade, código, etc.

Qualquer dúvida ou problema, sinta-se livre para entrar em contato.

Requisitos

Primeiramente, segue a avaliação de cada um dos requisitos técnicos obrigatórios que constam na descrição do trabalho. Os que foram cumpridos estão demarcados com um checkbox.

  • Conter pelo menos duas Activities
  • Conter pelo menos 4 telas diferentes
  • Consumir dados de uma API e/ou usar Banco de Dados
    • Se API é usada para salvar dados, você não precisa ter o banco local
    • Se você não usar uma API, você é obrigado a usar o banco localmente
  • Utilizar Fragments
  • Utilizar Navigation
  • Utilizar pelo menos uma Recycler View
  • Utilizar um padrão de projeto de livre escolha (Recomendável o MVVM)
  • Utilizar boas práticas de codificação
  • Usar arquivos strings.xml e colors.xml
    • Tem texto injetado direto no código
  • Seguir padrões de nomenclaturas dentro do projeto
    • Tanto no Kotlin quanto no XML
  • Construir a aplicação em pelo menos duas línguas através da ferramenta de internacionalização

Outros requisitos eram desejáveis (mas não são obrigatórios):

  • Utilizar Arquitetura MVVM
  • Utilizar o padrão de projeto Singleton
  • Criar cobertura de testes para o projeto (haverá pontuação extra para este item)
  • Utilizar Firebase
    • Principalmente para gerenciamento de contas de usuários (se o seu app permitir)

Requisitos não técnicos, mas obrigatórios:

  • Documentação de código
    • parcial
  • Documentação de Funcionalidades
    • parcial
  • Documentação em vídeo

Comentários gerais

Importante: infelizmente eu não consegui logar no app para usar. Quando clico para usar a conta Google ele falha e não entra. Todas as observações estão sendo feitas em relação ao vídeo apenas.
- Pelo o que eu entendi do código você está pegando o token direto do values.xml, é esse o problema?

De maneira geral, esta é uma aplicação simples, mas que serve para cumprir os pré-requisitos do projeto. Eu apenas discordo de ser um Kaban. Acho que o seu projeto web sim é um Kanban, mas o estado atual do app ele é apenas uma app de notas em formato de card.

A identidade visual é interessante 😸 e já vem do seu projeto maior, que parece ser bem legal. As funcionalidade do app poderiam ser melhor descritdas e a documentação de código falha em algumas partes.

Não foi feita cobertura de testes (que era extra)

Comentários técnicos

  • Gostei que você personalizou o launch icon da aplicação

  • Faltou documentar os modelos. O que cada atributo significa o que pode assumir

  • Poderia ter dividido o pacote de fragments e activities

  • Possui strings sendo injetado diretamente no código, o que não é uma boa prática

    • Exemplo: o collectionPath de nome users poderia ter sido armazenado em uma contante
    • Exemplo 2: as mensagens de log
    • Exemplo 3: as chaves dos parametros do bundle (kanban_id, note, etc)
    • Inclusive você já possui um arquivo de contantes, poderia ter usado

Comentários sobre usabilidade

  • Poderia ter opção de cadastro por email sem ser apenas via Google
    • Como disse, não consegui usar seu app 😩
    • Como nao consegui usar, não consigo colocar mais recomendações/feedbacks

Nota final

Pontuação de cada parte do trabalho:

  • Nota da implementação: 7.5
    • Observação: se eu consegui usar o app você aumenta essa nota para 8
  • Nota da documentação: 8.5
  • Nota dos testes: 0
@AHalic
Copy link
Collaborator

AHalic commented Feb 9, 2023

@paaatcha Estou pegando o token do values.xml sim, isso é um problema? Quando eu tinha testado, ao rodar o build esse arquivo era gerado, então achei que não fosse um problema...

@paaatcha
Copy link
Member Author

Eu realmente não sei, foi de fato uma pergunta. Será que é esse o problema?
Se quiser pode vim na minha sala e tentamos logar.

@paaatcha
Copy link
Member Author

Tentei rodar aqui novamente, e percebi que ele está retornando esse erro quando eu tento logar (o que aparentemente não é problema na conta Google)

Error while executing: am start -n "com.example.katify/com.example.katify.view.SplashActivity" -a android.intent.action.MAIN -c android.intent.category.LAUNCHER
Starting: Intent { act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] cmp=com.example.katify/.view.SplashActivity }
Error type 3
Error: Activity class {com.example.katify/com.example.katify.view.SplashActivity} does not exist.

Error while Launching activity
Failed to launch an application on all devices

@AHalic
Copy link
Collaborator

AHalic commented Feb 10, 2023

@paaatcha Que estranho. Aqui não tá dando esse erro de jeito nenhum. Pesquisando dizem para reiniciar a IDE link. Outra coisa é que eu tinha mudado o nome do rootProject, não sei se poderia ser isso também, mas já voltei para o que estava antes (rootProject.name = "katify").
Enfim, posso ir na sua sala no horário da aula umas 15h?

@paaatcha
Copy link
Member Author

Tenho reunião de departamento as 15h. As 14h estou disponivel.

@AHalic
Copy link
Collaborator

AHalic commented Feb 10, 2023

Tudo bem, então 14h estou ai

@AHalic
Copy link
Collaborator

AHalic commented Feb 10, 2023

@paaatcha Eu instalei o APK no celular e consegui logar com meu edu.ufes, e ai eu lembrei que existem domínios autorizados para logar no sistema. Você pode me dizer com qual dominio você está tentando logar?

@paaatcha
Copy link
Member Author

Testei com da UFES e gmail

@paaatcha Eu instalei o APK no celular e consegui logar com meu edu.ufes, e ai eu lembrei que existem domínios autorizados para logar no sistema. Você pode me dizer com qual dominio você está tentando logar?

@AHalic
Copy link
Collaborator

AHalic commented Feb 13, 2023

@paaatcha Não consegui descobrir qual o problema com o google 🫠
Então eu fiz um botãozinho de login anônimo

@paaatcha
Copy link
Member Author

@AHalic certo. Consigo testar hoje mais tarde apenas.

@paaatcha
Copy link
Member Author

@paaatcha Não consegui descobrir qual o problema com o google 🫠 Então eu fiz um botãozinho de login anônimo

Consegui usar e atualizei sua nota.

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

No branches or pull requests

2 participants