Intégration Skolengo - Installation + Auth #84

NonozgYtb · 2024-03-18T00:55:39Z

Warning

La review de @maelgangloff est très souhaitable.

Checklist d'avant pull request

Explications des changements

Différents rajouts liés à la future intégration de Skolengo NG (Nouvelle génération - avec scolengo-api@3)
Certains fichiers du "squelette" de l'application ont été modifiés afin de remplacer l'ancienne intégration (sans scolengo-api).
Enfin d'autres modifications mineurs ont été apportés (renommage et ajout de fonctionnalités mineures (activation conditionnelle de Skolengo) notamment).

Changelogs proposés

Remplacement de l'ancienne page de recherche d'établissement Skolengo par une mise à jour
Ajout de scolengo-api à la page de recherche d'étab.
La page SelectService est d'avantage modulaire
Rajout d'un blocage à la fin de l'obtention du token d'authentification Skolengo, puisque le reste n'est pas encore correctement implémenté (réutilisation jusqu'à cette étape de l'ancienne intégration)
Ajout de SkolengoCommonCache, une nouvelle intégration d'AsyncStorage (nouveau nom de SkolengoCache, maintenant typesafe). Elle intègre la gestion des default values, de l'expiration des données ainsi que celle des collections. Même si elle sera surement utilisé dans l'intégration SkolengoNG, elle est également un POC (proof of concept) puisque cela pourra être utilisé plus généralement sur l'application pour une meilleur homogénéité et garantir un seul moyen efficace et "++" (du fait des fonctionnalités supplémentaires) pour accéder aux données stuquées sur le téléphone.

Note

Pour que votre pull-request soit fusionnée, vous devez obtenir l'approbation de au moins deux membres de l'équipe dont un coordinateur.
Soyez donc patient :)

Review de @maelgangloff recommandée

Informations supplémentaires

Plus d'information dans le titre (et occasionnellement la description) des commits.

Ce merge n'est pas urgent, mais personnellement, je me baserai sur celui-ci pour le PR N°2 sur SkolengoNG.

Tip

N'hésitez pas à revenir vers moi par ce PR ou sur Discord concernant ce Pull Request ou plus globalement sur l'intégration SkolengoNG

Info : Skolengo est activé uniquement en version dev pour l'instant à des fins de test (__DEV__ || false)

Version typesafe de SkolengoCache + ajout de skoapp-prod dans app.json

Renommage pour avoir un seul nom au sein du projet

Car je suis fatigué donc je fais pas gaffe !

A continuer avec scolengo-api

LeGeek01 · 2024-03-18T18:30:02Z

y'a un conflit sur GradeView.js, il faudrait que tu le résoude

Ne sert actuellement à rien et sert de fichier de base pour créer l'intégration future de skolengo

+ suppression de l'ancienne implémentation Skolengo dans les Settings

ecnivtwelve · 2024-03-19T06:45:05Z

GradeView n'est plus utilisé de toute manière, faudrait le virer complètement même

NonozgYtb · 2024-03-20T21:32:08Z

Copie du message relayé sur le serveur discord :

Mise à jour - Intégration Skolengo "Next Gen" :

Je vous annonce que la PR Skolengo (partie 1 - Installation et Authentification) me semble finalisé.

Où en est-on ?

L'application utilise désormais scolengo-api de notre coupaing @maelgangloff , permettant une réactivité augmenté en cas de changement de la part de Kosmos.

La recherche d'établissement est fonctionnelle et utilise dès lors une interface similaire à celle de pronote (version de il y a quelques jours seulement)

L'authentification via deeplink marche complètement et on récupère bien le token, notre sésame pour continuer

Skolengo est dès lors activé pour toutes les builds dev uniquement (!__DEV__), ceci afin de simplifier le test de fonctions sans le rendre disponible à tous puisque loin d'être encore finalisé

Plusieurs petites améliorations mineures se sont aussi immiscés dans mes commits

Et maintenant ?
J'attend pour ma part les différentes reviews donc celle de @maelgangloff.
Je prévois de réaliser la structure générale du SkolengoDataProvider dans la partie 2 (nom pas officiel mais je les numérote quand même car cela n'est pas indépendant, car oui : essayer d'utiliser des données avant de pouvoir se connecter est stupide)
Autre petite news : J'ai màj SkolengoCommonCache (ex-SkolengoCache) donc je propose l'utilisation généralisée à moyen terme au sein de l'application, mais ce n'est pas le débat actuel !

D'accord et pour Pronote ?
Très bonne question : l'appli a été publié et marche pour Pronote :clown: (oui j'essaie de faire des blagues à 22h30, pas la meilleure idée que j'ai eu je dois l'avouer)

A tout bientôt les amis et merci pour votre investissement sur le projet 👍

fetch/Skolengo/SkolengoAuthWorkflow.ts

fetch/Skolengo/SkolengoCommonCache.ts

ecnivtwelve

A voir si @maelgangloff à quelque chose à ajouter et si l'application fonctionne pour les utilisateurs de Skolengo, mais au moins le code me parait OK !

tom-theret

J'ai cette erreur

node:events:492
      throw er; // Unhandled 'error' event
      ^

Error: EMFILE: too many open files, watch
    at FSWatcher._handle.onchange (node:internal/fs/watchers:207:21)
Emitted 'error' event on NodeWatcher instance at:
    at FSWatcher._checkedEmitError (/Users/tom.theret/Documents/git_project/Papillon/node_modules/metro-file-map/src/watchers/NodeWatcher.js:134:12)
    at FSWatcher.emit (node:events:514:28)
    at FSWatcher._handle.onchange (node:internal/fs/watchers:213:12) {
  errno: -24,
  syscall: 'watch',
  code: 'EMFILE',
  filename: null
}

NonozgYtb · 2024-03-21T16:52:28Z

Au vu de ton erreur, c'est ton pc qui doit avoir un probleme ou qui atteint la limite. Je te conseille de supprimer le dossier node_modules et de refaire un npm i ensuite quand tu fais fais npm start -- -c permettant de supprimer le cache d'expo. En tour cas, je ne pense pas que ce soit lié au PR que tu as ces problèmes. Mael corrige moi si j'ai faux.

tom-theret · 2024-03-21T17:32:09Z

Au vu de ton erreur, c'est ton pc qui doit avoir un probleme ou qui atteint la limite. Je te conseille de supprimer le dossier node_modules et de refaire un npm i ensuite quand tu fais fais npm start -- -c permettant de supprimer le cache d'expo. En tour cas, je ne pense pas que ce soit lié au PR que tu as ces problèmes. Mael corrige moi si j'ai faux.

Je viens de :

Supprimer node_module
Redémarrer le Mac
npm i
npm start -- -c

ça fonctionne toujours pas 😭

EDIT : Il fallait Watchman sur Mac

tom-theret

Supprime de app.tsx :

    {
      name: 'Grade',
      component: require('../views/Grades/GradeView').default,
      options: {
        presentation: 'modal',
        headerTintColor: '#fff',
      }
    },

et fait hyper attention à ta syntaxe

NonozgYtb · 2024-03-26T06:37:52Z

Sans message de votre part, est-ce que la PR est bonne, pouvez-vous la review et pouvez-vous peut être la merge ?

maelgangloff · 2024-03-26T06:59:00Z

Sans message de votre part, est-ce que la PR est bonne, pouvez-vous la review et pouvez-vous peut être la merge ?

Pour être validée, une PR doit obtenir deux review favorables. Cette PR est assez longue, ce qui implique d'attendre plus longtemps pour obtenir ces review.
Personnellement, je n'ai pas encore eu le temps de faire assez de tests.
Bonne journée

vinceh121

pensez à faire un kanban des tickets qui vont découler de cette PR (refresh de token, introspection, vue de notes, etc)

voir un kanban de toutes les tâches liées a Skolengo

vinceh121 · 2024-04-01T01:18:02Z

fetch/Skolengo/SkolengoAuthWorkflow.ts

+    disco
+  ).catch(skolengoErrorHandler);
+  if (!token) return skolengoErrorHandler();
+  //console.log({token});


Suggested change

//console.log({token});

vinceh121 · 2024-04-01T01:22:04Z

fetch/Skolengo/SkolengoAuthWorkflow.ts

+  return null;
+};
+
+export const loginSkolengoWorkflow = async (


faut reformatter, ça manque de lignes vide pour séparer les unités logique

vinceh121 · 2024-04-01T01:23:46Z

fetch/Skolengo/SkolengoAuthWorkflow.ts

+    'Skolengo : intégration en cours',
+    'Veuillez patienter, le processus de connexion à Skolengo à fonctionné.\nMais l\'intégration de Skolengo (NG) n\'est pas encode terminé.\n\nRevenez plus tard.'
+  );
+  /* // TODO : Créer l'intégration via scolengo-api


faudrait finir au moins l'enregistrement du token et de l'école

vinceh121 · 2024-04-01T01:24:50Z

fetch/Skolengo/SkolengoCommonCache.ts

@@ -1,36 +1,50 @@
 import AsyncStorage from '@react-native-async-storage/async-storage';

-class SkolengoPrivateCache {
+class CommonCacheUtils {


"Common" mais prefixé de SkolengoCache_ et sous un namespace Skolengo ?

vinceh121 · 2024-04-01T01:26:57Z

fetch/Skolengo/SkolengoCommonCache.ts

  static msToTomorrow() {
    const now = Date.now();
-    const timePassedToday = now % this.DAY;
+    const timePassedToday = (now % this.DAY) + this.TIMEZONE_OFFSET;


C'est du différentiel, laisse tout en UTC0 au lieu de te faire chier avec les fuseaux horaires

vinceh121 · 2024-04-01T01:43:26Z

.yarnrc

@@ -0,0 +1 @@
+--install.ignore-optional true


ne pas mélanger les package managers

vinceh121 · 2024-04-01T01:46:50Z

fetch/Skolengo/SkolengoAuthWorkflow.ts

+  ).catch(skolengoErrorHandler);
+  if (!token) return skolengoErrorHandler();
+  //console.log({token});
+  Alert.alert(


pas besoin de faire de joli toasts pour ce qui restera en develop

vinceh121 · 2024-04-01T01:47:28Z

fetch/Skolengo/SkolengoDataProvider.ts

+export class SkolengoDataProvider {
+  constructor() {
+    if (!SkolengoNotImplementedWarn) {
+      console.warn('SkolengoDataProvider is not implemented');


pas besoin de faire de joli warnings pour ce qui restera en develop

vinceh121 · 2024-04-01T01:54:44Z

views/NewAuthStack/Skolengo/LocateSkolengoEtab.tsx

+    >
+      <StatusBar
+        animated
+        barStyle={UIColors.dark ? 'light-content' : 'dark-content'}


peut être à généraliser dans le composant

vinceh121 · 2024-04-01T01:57:16Z

views/NewAuthStack/SelectService.tsx


-  const selectOption = (index) => {
+  const selectOption = (index:number) => {


Suggested change

const selectOption = (index:number) => {

const selectOption = (index: number) => {

Vilerio · 2024-04-08T18:33:05Z

Salut, moyen de régler les conflits et de fermer vos discussions si elles sont résolues svp ?

Vilerio · 2024-04-08T18:33:28Z

mais visiblement y a des trucs à modif

maelgangloff · 2024-04-08T18:45:46Z

La résolution des conversations est prioritaire sur les conflits qui pourront être gérés dans un second temps afin d'avoir une bonne base.
Pour rappel, les conditions pour qu'une PR soit acceptée sont :

L'ensemble des discussions sont résolues
La PR reçoit au moins deux review favorables

Cette PR est assez conséquente, il ne faut pas négliger les avis de chacun.
Si @NonozgYtb tu considère que certaines remarques ne sont pas pertinentes, n'hésite pas à en débattre ;)

camarm-dev

Je suis utilisateur de skolengo et l'authentification fonctionne bien de mon côté.

LeGeek01 · 2024-05-06T14:48:35Z

@NonozgYtb il va falloir fixer les conflits et regarder les remarques de vinceh21, les passer en résolu si possible

NonozgYtb added 9 commits March 17, 2024 16:10

refactor: ♻️ amélioration de la modularité de SelectService

097cfe3

feat: 🔧 ajout de skolengo (skoapp-prod)

4df3f29

feat: ✨ maj du login Skolengo

5970d12

Info : Skolengo est activé uniquement en version dev pour l'instant à des fins de test (__DEV__ || false)

feat: ✨ ajout de SkolengoCommonCache

b72ea05

Version typesafe de SkolengoCache + ajout de skoapp-prod dans app.json

fix: 🔀 merge development > feat/scolengo-ng

f500e84

style: 🎨 renommage appctx > appContext

542d766

Renommage pour avoir un seul nom au sein du projet

fix: 🐛 activation conditionnelle de Skolengo

4aa5e08

fix: 💩 rajout d'un point (appContext . dataProvider)

115376d

Car je suis fatigué donc je fais pas gaffe !

feat: 🚧 blocage de la connexion par Skolengo

f584d40

A continuer avec scolengo-api

NonozgYtb added 5 commits March 18, 2024 22:33

feat: 🚧 ajout d'une classe SkolengoDataProvider (en construction)

02f3e29

Ne sert actuellement à rien et sert de fichier de base pour créer l'intégration future de skolengo

feat: 🚧 ajout du Workflow d'authentification Skolengo (WIP)

7eb44aa

style: 🎨 lint - LocateSkolengoEtab

77c090b

feat: 🎨 restructuration SettingsScreen pour une meilleure lisibilité

05b4ff1

+ suppression de l'ancienne implémentation Skolengo dans les Settings

feat: ⚰️ suppression de l'ancienne intégration de Skolengo

1fca312

NonozgYtb added 2 commits March 20, 2024 22:07

feat: ✨ finalisation authentification Skolengo

e26fe86

feat: ⚰️ suppression GradeView

ea4e250

NonozgYtb changed the title ~~Intégration Skolengo (part 1)~~ Intégration Skolengo - Installation + Auth Mar 20, 2024

NonozgYtb added 2 commits March 20, 2024 23:04

fix: ⏪ revert d'un deeplink de test

c8dd934

feat: 🚚 renommage dossier fetch/NewSkolengo > fetch/Skolengo

402742e

maelgangloff reviewed Mar 21, 2024

View reviewed changes

fetch/Skolengo/SkolengoAuthWorkflow.ts Outdated Show resolved Hide resolved

fetch/Skolengo/SkolengoCommonCache.ts Outdated Show resolved Hide resolved

fix: 🚑 fix des caractères manquants

2d5fa07

ecnivtwelve previously approved these changes Mar 21, 2024

View reviewed changes

test: 🔇 - log du token Skolengo

68d3c2d

tom-theret reviewed Mar 21, 2024

View reviewed changes

fix: syntax LocateSkolengoEtab.tsx

f68bd25

tom-theret added 2 commits March 21, 2024 23:28

inset LocateSkolengoEtab.tsx

754765b

fix: syntax SettingsScreen.tsx

e196bf9

tom-theret suggested changes Mar 21, 2024

View reviewed changes

fix: 🔥 suppr. GraveView dans AppStack

162ff96

NonozgYtb requested a review from tom-theret March 23, 2024 20:42

NonozgYtb added 2 commits March 23, 2024 22:25

feat: 🎨 simplification du SelectService.tsx + typage

48ff4a9

Merge branch 'development' into feat/scolengo-ng

c9e8af6

NonozgYtb dismissed ecnivtwelve’s stale review via c9e8af6 March 24, 2024 13:31

maelgangloff added the enhancement New feature or request label Mar 24, 2024

vinceh121 suggested changes Apr 1, 2024

View reviewed changes

camarm-dev approved these changes Apr 13, 2024

View reviewed changes

tom-theret approved these changes May 18, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Intégration Skolengo - Installation + Auth #84

Intégration Skolengo - Installation + Auth #84

NonozgYtb commented Mar 18, 2024 •

edited by anaelchevillard

LeGeek01 commented Mar 18, 2024

ecnivtwelve commented Mar 19, 2024

NonozgYtb commented Mar 20, 2024 •

edited by maelgangloff

ecnivtwelve left a comment

tom-theret left a comment

NonozgYtb commented Mar 21, 2024

tom-theret commented Mar 21, 2024 •

edited

tom-theret left a comment •

edited

NonozgYtb commented Mar 26, 2024 •

edited

maelgangloff commented Mar 26, 2024

vinceh121 left a comment

vinceh121 Apr 1, 2024

vinceh121 Apr 1, 2024

vinceh121 Apr 1, 2024

vinceh121 Apr 1, 2024

vinceh121 Apr 1, 2024

vinceh121 Apr 1, 2024

vinceh121 Apr 1, 2024

vinceh121 Apr 1, 2024

vinceh121 Apr 1, 2024

vinceh121 Apr 1, 2024

Vilerio commented Apr 8, 2024

Vilerio commented Apr 8, 2024

maelgangloff commented Apr 8, 2024

camarm-dev left a comment

LeGeek01 commented May 6, 2024


		const selectOption = (index) => {
		const selectOption = (index:number) => {

	const selectOption = (index:number) => {
	const selectOption = (index: number) => {

Intégration Skolengo - Installation + Auth #84

Are you sure you want to change the base?

Intégration Skolengo - Installation + Auth #84

Conversation

NonozgYtb commented Mar 18, 2024 • edited by anaelchevillard

Checklist d'avant pull request

Explications des changements

Changelogs proposés

Note

Informations supplémentaires

LeGeek01 commented Mar 18, 2024

ecnivtwelve commented Mar 19, 2024

NonozgYtb commented Mar 20, 2024 • edited by maelgangloff

Mise à jour - Intégration Skolengo "Next Gen" :

ecnivtwelve left a comment

Choose a reason for hiding this comment

tom-theret left a comment

Choose a reason for hiding this comment

NonozgYtb commented Mar 21, 2024

tom-theret commented Mar 21, 2024 • edited

tom-theret left a comment • edited

Choose a reason for hiding this comment

NonozgYtb commented Mar 26, 2024 • edited

maelgangloff commented Mar 26, 2024

vinceh121 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Vilerio commented Apr 8, 2024

Vilerio commented Apr 8, 2024

maelgangloff commented Apr 8, 2024

camarm-dev left a comment

Choose a reason for hiding this comment

LeGeek01 commented May 6, 2024

NonozgYtb commented Mar 18, 2024 •

edited by anaelchevillard

NonozgYtb commented Mar 20, 2024 •

edited by maelgangloff

tom-theret commented Mar 21, 2024 •

edited

tom-theret left a comment •

edited

NonozgYtb commented Mar 26, 2024 •

edited