Skip to content

Create user#2

Open
MartinDelille wants to merge 22 commits intomasterfrom
1-implement-discourse-api
Open

Create user#2
MartinDelille wants to merge 22 commits intomasterfrom
1-implement-discourse-api

Conversation

@MartinDelille
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Member Author

@MartinDelille MartinDelille left a comment

Choose a reason for hiding this comment

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

Ok c'est réglé

Copy link
Copy Markdown
Member Author

@MartinDelille MartinDelille left a comment

Choose a reason for hiding this comment

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

Pour l'instant, j'essaye juste de faire une requête sur l'api publique de github.

Comment thread pom.xml
Comment thread pom.xml Outdated

<!-- The repository where to get XWiki dependencies -->
<repositories>
<repository>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Il n'y as aucune raison pour que tu ai besoin de ça. Maven central est activé de base dans Maven.

Comment thread pom.xml
@Role
public interface UserSyncConnector
{
void getUser(String userId);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Cette methode n'as pas de sens dans cette API. Si c'est juste pour ton test unitaire il te suffit de caster this.mocker.getComponentUnderTest() en DiscourseUserSyncConnector pour appeller la methode temporaire. Ça évitera d'oublier de la virer à la fin ;)

@MartinDelille
Copy link
Copy Markdown
Member Author

@tmortagne C'est bon cette première étape est terminée de mon côté. J'ai un problème indépendant mais je vais créer une issue spécifique.

Je te laisse me faire un retour et merger si ça te parait bon!

@MartinDelille
Copy link
Copy Markdown
Member Author

@tmortagne je viens de pousser ce qu'on a fait en début d'aprèm.

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.

2 participants