Skip to content
This repository has been archived by the owner on Oct 15, 2023. It is now read-only.

Prise en charge des ENT basés sur le CMS Skolengo #91

Merged
merged 16 commits into from
Apr 6, 2023
Merged

Prise en charge des ENT basés sur le CMS Skolengo #91

merged 16 commits into from
Apr 6, 2023

Conversation

maelgangloff
Copy link
Contributor

@maelgangloff maelgangloff commented Apr 5, 2023

Pull request checklist

Please check if your PR fulfills the following requirements:

  • You can build successfully the whole project with your changes locally
  • You use the coding conventions and the naming conventions of the project
  • You use tabs for indentation
  • You make sure that your PR is not a duplicate
  • This PR is ready to be reviewed and merged
  • This PR merges into the development branch
  • There is no TODO in the code
  • There is no spelling or grammatical errors in the code
  • Details of the issue or feature are documented below
  • This PR is not a breaking change (e.g. changes that would cause existing functionality to change)

Proposed changelog

Français fr
Support de certaines fonctionnalités des ENT régionaux basés sur le CMS Skolengo 🥳

Informations supplémentaires

  • Page de login: pouvoir se connecter avec le mot de passe temporaire ou avec un jeton
  • Emploi du temps: l'emploi du temps est fonctionnel. Il pourrait être envisageable de calculer un hash en utilisant le nom de la matière pour afficher une couleur différente pour chaque matière dans l'UI.
  • Travail à faire: afficher les devoirs dans la section correspondante et dans le "Récap" sur la page d'accueil.
  • Actualités: dernières actualités de l'établissement.

Quelques screenshots:

Login avec un ENT Skolengo

Sélectionner le service Sélectionner l'instance Skolengo Page de login
image image image

Pages déjà disponibles 🥳

Vue d'ensemble Emploi du temps Travail à faire Actualités
image image image image

Issue liée: #85

@maelgangloff maelgangloff changed the title Feat skolengo Prise en charge des ENT basés sur le CMS Skolengo Apr 5, 2023
@maelgangloff maelgangloff changed the base branch from development to services/skolengo April 5, 2023 19:31
@maelgangloff maelgangloff changed the title Prise en charge des ENT basés sur le CMS Skolengo Prise en charge des ENT basés sur le CMS Skolengo (#v1) Apr 5, 2023
@maelgangloff maelgangloff changed the title Prise en charge des ENT basés sur le CMS Skolengo (#v1) Prise en charge des ENT basés sur le CMS Skolengo (#1) Apr 5, 2023
@maelgangloff maelgangloff marked this pull request as ready for review April 5, 2023 19:35
@maelgangloff
Copy link
Contributor Author

maelgangloff commented Apr 5, 2023

Attention !
Concernant les ENT Skolengo, les requêtes se font au travers du proxy CORS de Papillon. Dans les prochains commits, il serait intéressant de se passer de ce serveur, dans le but de sécuriser davantage les données des utilisateurs finaux.

Copy link
Member

@lucas-luchack lucas-luchack left a comment

Choose a reason for hiding this comment

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

Ne peux pas être accepté en production pour le moment.

La gestion des erreurs est inapproprié, il faut que l'erreur soit prise en charge par le système de logs et que l'utilisateur final puisse savoir que la récupération a échoué...

P.S. Je ne suis actuellement pas chez moi je ne peux donc pas voir correctement le code de cette PR, il faudra donc être patient !

@maelgangloff
Copy link
Contributor Author

Ne peux pas être accepté en production pour le moment.

La gestion des erreurs est inapproprié, il faut que l'erreur soit prise en charge par le système de logs et que l'utilisateur final puisse savoir que la récupération a échoué...

P.S. Je ne suis actuellement pas chez moi je ne peux donc pas voir correctement le code de cette PR, il faudra donc être patient !

Les erreurs du calendrier s'affichent normalement maintenant.

@moeenio
Copy link
Contributor

moeenio commented Apr 6, 2023

Peux tu fournir des screenshots :

  • de la page avant la liste des ENT skolengo
  • du formulaire de connexion

?

@maelgangloff
Copy link
Contributor Author

Peux tu fournir des screenshots :

* de la page avant la liste des ENT skolengo

* du formulaire de connexion

?

Hello,
J'ai mis à jour la description de la PR avec ces screenshots ;)

@ecnivtwelve
Copy link
Contributor

Faut que je regarde un peu le code mais du coup hormis les erreurs rien n'empêche de merge ? @lucas-luchack

@lucas-luchack
Copy link
Member

lucas-luchack commented Apr 6, 2023

En soit oui tant que c'est pas merge sur la branche dev, du moins pour le peut que j'ai vu. Faut verif que tout marche bien 😉

@lucas-luchack
Copy link
Member

Je merge du coup !

@lucas-luchack lucas-luchack merged commit 02cb3cb into PapillonApp:services/skolengo Apr 6, 2023
Copy link
Member

@lucas-luchack lucas-luchack left a comment

Choose a reason for hiding this comment

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

Mergé

@maelgangloff maelgangloff changed the title Prise en charge des ENT basés sur le CMS Skolengo (#1) Prise en charge des ENT basés sur le CMS Skolengo Apr 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants