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

[Inviter à rejoindre Tchap] Erreur avec le mainlining 1.83 #837

Closed
NicolasBuquet opened this issue May 23, 2023 · 3 comments · Fixed by #891
Closed

[Inviter à rejoindre Tchap] Erreur avec le mainlining 1.83 #837

NicolasBuquet opened this issue May 23, 2023 · 3 comments · Fixed by #891
Labels
bug Something isn't working iOS

Comments

@NicolasBuquet
Copy link
Contributor

NicolasBuquet commented May 23, 2023

En test sur le mainlining 1.83, on constate le souci suivant.

Il existe 2 manières d'inviter un agent ou un externe sur Tchap.
tchap-invite - 2023-05-23 at 18 01 15

Inviter un agent ou un externe se fait par la création d'un salon Direct Message entre l'inviteur et l'invité.

Le endpoint /createroom est appelé sur le back-end.

Avant le mainlining 1.59, seuls 3 paramètres étaient obligatoires. Lors d'une évolution entre la 1.59 et la 1.83, un nouveau paramètre est devenu obligatoire, le id_access_token.

Sur Tchap iOS, si l'on invite un agent ou un externe par l'option "Nouveau message direct", tout se passe correctement.

Mais si l'on invite un agent ou un externe par l'option "Inviter à rejoindre Tchap", le back-end renvoie une erreur :

Simulator Screen Shot - iPhone 14 - 2023-05-23 at 18 09 20

Les 2 chemins d'invitation ne sont pas les mêmes dans le code iOS :

  • l'un appelle MatrixSDK/Core/MXRestClient/createRoomWithParameters:(MXRoomCreationParameters*)parameters success:(void (^)(MXCreateRoomResponse *response))success failure:(void (^)(NSError *error))failure qui ajoute correctement le 4e paramètre.

  • l'autre appelle Tchap/Managers/Room/RoomService/createRoom(with roomServiceCreationParameters: RoomServiceCreationParameters, completion: @escaping (MXResponse<String>) qui n'ajoute pas le 4e paramètre.

Question

Faut-il corriger l'appel spécifique Tchap RoomService/createRoom ou faut-il supprimer cette option spécifique à Tchap ?

@NicolasBuquet NicolasBuquet added bug Something isn't working iOS labels May 23, 2023
@odelcroi
Copy link
Member

Inviter un externe se fait par la création d'un salon Direct Message entre l'inviteur et l'externe.

Si je comprend bien cette option n'est pas uniquement propre aux externes, je peux inviter par email un agent ?

Faut-il corriger l'appel spécifique Tchap RoomService/createRoom ou faut-il supprimer cette option spécifique à Tchap ?

Je pense qu'il faut garder le bouton "Inviter à rejoindre Tchap"

Arrive-t-on au meme resultat avec les deux fonctions mentionnées? ie : créer une room privée avec le membre invité?

@NicolasBuquet
Copy link
Contributor Author

En effet, l'invitation permet d'inviter un agent ou un externe. Dans les 2 cas, le problème se présente si l'invité n'est pas déjà présent sur Tchap.

Et c'est en effet la création de la room privée résultante qui pose problème en passant pas le menu Paramètres.

@odelcroi
Copy link
Member

est ce que ce ticket est résolu?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working iOS
Projects
None yet
2 participants