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

Pierre HERMAN test technique #13

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Conversation

HermanPierre
Copy link

@HermanPierre HermanPierre commented Nov 28, 2024

User:

  • Bugfix: Bouton Update non fonctionnel

Projects:

  • Bugfix: Amélioration de la création de projet : Redirection sur le détail d’un projet après création
  • Récupération du projet depuis l’api, modification de la route pour que le get project par id ne renvoie qu’un seul élément et non pas un tableau
  • Nom du projet -> Project name (app en anglais) (différentes traduction dans l'app pour la cohérence)
    • Gap après « Project name »

Edit Projets:

  • Ajouts de liens : input sur toute la largeur à l’ajout
  • Bugfixs: Amélioration globale du formulaire d’édition de project

Activities:

  • Bugfix: Nom du projet dans le placeholder
  • Bugfix: Dans la page activities, récupération des activities avec le userId (on utilisait userName)

Features:

  • Utilisation de l’email pour la récupération de l’image gravatar (si existante)
  • Ajout d’un export excel sur le mois en cours
  • Dockerisation

Feedback à propos du code:
J’ai remarqué que Redux était installé sur le front, mais qu’il n’était quasiment pas exploité, j’améliorerais le code pour l’utiliser et réduire le nombre de requête faite à l’API.
J’ai aussi créé des service dédié à chaque model sur le front même si « api.js » existait déjà, je trouve que cela rends le code plus lisible.

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.

1 participant