Provide serverNames when available and fix issue around analytics #2843
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Type of change
Content
The
serverNames
(AKA as via parameter) was not provided to the join room API.This PR ensure that it is provided by using the new API
joinRoomByIdOrAlias
.This is centralized in
DefaultJoinRoom
, which is also taking care of the analytics. PreviouslyAcceptDeclineInvitePresenter
was always used to join a Room and soJoinedRoom.Trigger.Invite
was always sent to analytics, which was not correct.Motivation and context
Give more chance for the join room to work, if the homerver does not know the RoomId.
Screenshots / GIFs
NA
Tests
Tested devices
Checklist