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

[WIP] create discourse connector #2

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

Conversation

MartinDelille
Copy link
Member

@MartinDelille MartinDelille commented Sep 22, 2018

Voici un début d'implémentation du connecteur Discourse. Pour l'instant, le code est le même que celui de OVH, j'ai juste renommé les fichiers.

Si vous voyez quelque chose qui déconne, dites moi!

Je prévois de faire des tests sur https://adm.trydiscourse.com/ en attendant qu'on choisisse un hébergeur.

@tmortagne
Copy link
Member

l'API existante est très mailing-list oriented du coup pas moyen de créer les nouveaux users et maintenir leurs infos à jour avec ça. J'imagine que le plus simple c'est d'implémenter ce module uniquement pour maintenir à jour des listes de users en fonction des groupes de l'intra et écrire un autre listener dédié à la création des nouveaux users et la synchro de leurs infos (mail, nom, prénom, image au moins).

@MartinDelille
Copy link
Member Author

@tmortagne il faut qu'on en parle en fin de semaine si tu es là

@MartinDelille
Copy link
Member Author

@tmortagne Bon on a pas eu le temps d'en parler mais en gros je préfère partir de ce code car je suppose qu'il est branché au changement de groupe de l'intra. Toutes la partie se connectant à l'API OVH est bien évidemment à réécrire. Qu'en penses-tu?

@tmortagne
Copy link
Member

Je ne t'ai pas dit que tout ce code est a jeter, je te dit simplement qu'avec cette API tu n'as pas assez d'info pour la partir création de compte Discourse lorsqu'un compte est créé sur l'intranet.

@MartinDelille
Copy link
Member Author

Ok il va falloir trouver une solution. Cela dit je pense que la question de l'authentification est prioritaire!

@tmortagne
Copy link
Member

Oui mais l’authentification est un sujet différent. J'ai déjà configuré Discourse pour qu'il se connecte sur l'intra mais il semble qu'il n'ai pas le droit d'accéder à internet actuellement...

@skylex
Copy link

skylex commented Oct 8, 2018 via email

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.

None yet

3 participants