Add default permalink to access rooms by link

[yostyle]

Type of change

• [ ] Feature
• [x] Bugfix
• [ ] Technical
• [ ] Other :

Content

Motivation and context

1040

Screenshots / GIFs

Before	After

Tests

• Step 1
• Step 2
• Step ...

Tested devices

• [x] Physical
• [ ] Emulator
• OS version(s):

Checklist

• [ ] Changes has been tested on an Android device or Android emulator with API 21
• [ ] UI change has been tested on both light and dark themes
• [ ] Accessibility has been taken into account. See https://github.com/tchapgouv/tchap-android/blob/develop/CONTRIBUTING.md#accessibility
• [x] Pull request is based on the develop branch
• [x] Pull request includes a new file under ./changelog.d. See https://github.com/tchapgouv/tchap-android/blob/develop/CONTRIBUTING.md#changelog
• [ ] Pull request includes screenshots or videos if containing UI changes
• [ ] Pull request includes a sign off
• [ ] You've made a self review of your PR
• [ ] If you have modified the screen flow, or added new screens to the application, you have updated the test UiAllScreensSanityTest.allScreensTest()

[yostyle]

Je ne vois pas où les modifications ajoutent les paramètres via dans le cas de construction de permalink a partir de roomId ?

J'appelle la méthode createRoomPermalink au lieu de createPermalink quand il s'agit d'un roomId

[NicolasBuquet]

Je ne vois pas où les modifications ajoutent les paramètres via dans le cas de construction de permalink a partir de roomId ?

J'appelle la méthode createRoomPermalink au lieu de createPermalink quand il s'agit d'un roomId

Ok, mais sans le paramètres via. Hors, si on crée le permalink à partir du roomId, les via deviennent nécessaires cross homeServers.

[yostyle]

Je ne vois pas où les modifications ajoutent les paramètres via dans le cas de construction de permalink a partir de roomId ?

J'appelle la méthode createRoomPermalink au lieu de createPermalink quand il s'agit d'un roomId

Ok, mais sans le paramètres via. Hors, si on crée le permalink à partir du roomId, les via deviennent nécessaires cross homeServers.

Comme tu peux le voir dans le screenshot le paramètre via est présent. Il est calculé et ajouté par défaut sur Android. C'est pour cette raison que je t'ai proposé d'en faire de même sur iOS quand on génére un lien avec le roomId.

https://github.com/tchapgouv/tchap-android/blob/c7b54b8d3d23e202877044107694d1e963d8f426/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/permalinks/PermalinkFactory.kt#L75 https://github.com/tchapgouv/tchap-android/blob/a4dd2793522a9060a9b03e278c65f825ed2a921c/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/permalinks/ViaParameterFinder.kt#L57

[NicolasBuquet]

Ok, d'accord. Je vois maintenant. Good.