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

Direct Message: manage encrypted DM in case of invite by email #7396

Merged
merged 6 commits into from
Mar 6, 2023

Conversation

nimau
Copy link
Contributor

@nimau nimau commented Feb 27, 2023

This PR fixes #6612

Here is a summary of the changes:

  • Don’t allow to invite more than one contact by email
  • The DM will be created by enabling the encryption when the HS promotes the encryption
  • The chat composer is disabled until a matrix account is created by using the invited email

This PR is related to matrix-ios-sdk PR #1727

- Don’t allow to invite more than one contact by email
- The DM will be created by enabling the encryption when the HS promotes the encryption
- The chat composer is disabled until a matrix account is created by using the invited email
@codecov
Copy link

codecov bot commented Feb 27, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.02 ⚠️

Comparison is base (081615f) 12.15% compared to head (363dc00) 12.14%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #7396      +/-   ##
===========================================
- Coverage    12.15%   12.14%   -0.02%     
===========================================
  Files         1643     1644       +1     
  Lines       162265   162550     +285     
  Branches     66630    66766     +136     
===========================================
+ Hits         19729    19741      +12     
- Misses      141890   142164     +274     
+ Partials       646      645       -1     
Flag Coverage Δ
uitests 55.23% <0.00%> (-0.09%) ⬇️
unittests 6.01% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ot/Modules/Room/RoomInfo/RoomInfoCoordinator.swift 0.00% <0.00%> (ø)
.../Room/RoomInfo/RoomInfoCoordinatorParameters.swift 0.00% <0.00%> (ø)
Riot/Modules/Room/RoomViewController.m 0.00% <0.00%> (ø)
Riot/Modules/Room/RoomViewController.swift 0.00% <0.00%> (ø)
Riot/Modules/StartChat/StartChatViewController.m 0.00% <0.00%> (ø)
...WaitingForMembers/View/RoomWaitingForMembers.swift 0.00% <0.00%> (ø)
...g/StartLocationSharing/LocationSharingModels.swift 66.66% <0.00%> (-13.34%) ⬇️
...ingViewer/LiveLocationSharingViewerViewModel.swift 47.87% <0.00%> (-2.13%) ⬇️
...s/MatrixKit/Models/Room/MXKRoomDataSourceManager.m 28.24% <0.00%> (-1.76%) ⬇️
Riot/Modules/Room/CellData/RoomBubbleCellData.m 0.00% <0.00%> (ø)
... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@giomfo
Copy link
Member

giomfo commented Feb 28, 2023

@nimau I'm currently testing your PR, I observed a pb:

  • If I granted the phone book access, the list of my local contacts are displayed in the first screen opened when I want to start a new DM: OK
    The pb is these contacts are not selectable : I should be able to select one of them until a first email is selected

image

  • Because we can invite only one email at a time, we should adapt the wording of the following banner. @gaelledel don't you think so?:

image

Riot/Modules/StartChat/StartChatViewController.m Outdated Show resolved Hide resolved
Riot/Modules/StartChat/StartChatViewController.m Outdated Show resolved Hide resolved
Riot/Modules/StartChat/StartChatViewController.m Outdated Show resolved Hide resolved
Riot/Modules/StartChat/StartChatViewController.m Outdated Show resolved Hide resolved
Riot/Modules/StartChat/StartChatViewController.m Outdated Show resolved Hide resolved
@nimau
Copy link
Contributor Author

nimau commented Mar 1, 2023

@giom I have updated this PR to reflect your comments.

@nimau nimau marked this pull request as ready for review March 1, 2023 16:47
@nimau nimau requested a review from pixlwave March 3, 2023 09:15
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Member

@giomfo giomfo left a comment

Choose a reason for hiding this comment

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

SGTM , all tests were ok -> approved
Please wait before merging, we want to merge it cross-platform

@nimau nimau merged commit c568b5e into develop Mar 6, 2023
@nimau nimau deleted the nimau/6612-dm-email branch March 6, 2023 11:04
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.

Direct Message: Manage encrypted DM in case of invite by email
2 participants