Skip to content

Comments

[WIP] create discourse connector#2

Open
MartinDelille wants to merge 1 commit intomasterfrom
discourse
Open

[WIP] create discourse connector#2
MartinDelille wants to merge 1 commit intomasterfrom
discourse

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.

3 participants