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

Feat#59/tela de edit profile #23

Merged
merged 9 commits into from
Dec 7, 2024

Conversation

R1K4S
Copy link
Contributor

@R1K4S R1K4S commented Dec 5, 2024

[FEATURE] Criação da tela EditProfile (#59)

Descrição

Este pull request adiciona classes e pastas que envolvam a parte de Gerenciamento do Usuário,como EditProfile que se encontra a tela de alteração de informações do Usuário,criação do componente CustomFieldText para criar placeholder,criação do componente de ProfileCard que seria apresentação do usuário conforme previsto na issue #59.

Revisão

  • Criação da pasta edit_profile no caminho /home/r/UnB/MDS/2024.2-ARANDU-APP/lib/ui/edit_profile;
  • Criação da pasta edit_profile no caminho /home/r/UnB/MDS/2024.2-ARANDU-APP/lib/ui/edit_profile/view;
  • Criação do arquivo EditProfile no caminho /home/r/UnB/MDS/2024.2-ARANDU-APP/lib/ui/edit_profile/view/;
  • Criação da pasta edit_profile no caminho /home/r/UnB/MDS/2024.2-ARANDU-APP/lib/ui/profile;
  • Criação do arquivo CustomTextField no caminho /home/r/UnB/MDS/2024.2-ARANDU-APP/lib/ui/shared/;
  • Criação do arquivo ProfileCard no caminho /home/r/UnB/MDS/2024.2-ARANDU-APP/lib/ui/shared/;

Checklist

  • O conteúdo adicionado nesse PR segue o código de conduta do projeto;
  • Foram seguidos os padrões estabelecidos no guia de contribuição;
  • O conteúdo adicionado foi devidamente testado;
  • O conteúdo do PR foi revisado por membro alheio ao mesmo;
  • O conteúdo deste PR foi devidamente identado, comentado e documentado;
  • Para conteúdos de documentação, foi adicionada tabela de versionamento aos artefatos.

@R1K4S R1K4S assigned R1K4S and unassigned R1K4S Dec 5, 2024
@R1K4S R1K4S self-assigned this Dec 5, 2024
@GabrielCostaDeOliveira
Copy link
Contributor

@R1K4S, está faltando algumas coisas.

O código não está compilando.

Você não está seguindo o padrão MVVM.

Tome cuidado com métodos muito grandes. O construtor de EditProfile tem quase 200 linhas quebre ele em widgets menores.

import 'package:aranduapp/ui/shared/CustomTextField.dart';
import 'package:aranduapp/ui/shared/TextPassword.dart';

class EditProfile extends StatelessWidget {
Copy link
Contributor

Choose a reason for hiding this comment

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

O MVVM não está implementado nesta classe

EditProfile({Key? key}) : super(key: key);

@override
Widget build(BuildContext context) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Esse construtor está muito grande quebre ele em widgets menores

@R1K4S
Copy link
Contributor Author

R1K4S commented Dec 5, 2024

Beleza

final String label;
final String? placeholder;
final TextEditingController? controller;
final bool obscureText;
Copy link
Contributor

Choose a reason for hiding this comment

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

O MVVM não está implementado nessa classe

Copy link
Contributor

@GabrielCostaDeOliveira GabrielCostaDeOliveira left a comment

Choose a reason for hiding this comment

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

.

@GabrielCostaDeOliveira GabrielCostaDeOliveira merged commit 0d86c82 into dev Dec 7, 2024
1 check failed
GabrielCostaDeOliveira pushed a commit that referenced this pull request Dec 9, 2024
GabrielCostaDeOliveira pushed a commit that referenced this pull request Dec 9, 2024
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.

2 participants