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

[FEATURE] Retrouver une plateforme LTI grâce à son clientId (PIX-16940). #11609

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

VincentHardouin
Copy link
Member

@VincentHardouin VincentHardouin commented Mar 7, 2025

🥞 Problème

Actuellement, nous n'avons pas de repository pour récupérer une plateforme LTI, ce qui va être nécessaire dans les futur développement.

🥓 Proposition

Ajouter une méthode pour le faire.

🧃 Remarques

A cause de la mise en place du chiffrement de la privateKey, nous nous sommes rendus compte que le type varchar(255) n'était pas suffisant. Nous sommes donc passé en type text pour cette colonne.

😋 Pour tester

N/A

@VincentHardouin VincentHardouin added team-maddo Mise à Dispo de Données LTI labels Mar 7, 2025
@VincentHardouin VincentHardouin marked this pull request as ready for review March 7, 2025 15:19
@VincentHardouin VincentHardouin requested review from a team as code owners March 7, 2025 15:19
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@nlepage nlepage force-pushed the pix-16940-lti-platform-registration-repo branch from 653fb0c to 90d0366 Compare March 7, 2025 15:31
Co-authored-by: Nicolas Lepage <[email protected]>
Co-authored-by: Vincent Hardouin <[email protected]>
@nlepage nlepage force-pushed the pix-16940-lti-platform-registration-repo branch from 90d0366 to 09f6d90 Compare March 7, 2025 15:33
Copy link
Member

@nlepage nlepage left a comment

Choose a reason for hiding this comment

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

LGTM!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LTI team-maddo Mise à Dispo de Données
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants