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

[Settings] Hide (previously Adapt) General/Discovery settings #251

Closed
3 tasks done
giomfo opened this issue Oct 25, 2021 · 8 comments
Closed
3 tasks done

[Settings] Hide (previously Adapt) General/Discovery settings #251

giomfo opened this issue Oct 25, 2021 · 8 comments
Assignees

Comments

@giomfo
Copy link
Contributor

giomfo commented Oct 25, 2021

Can you please check first why we get the following exception on this screen (this may be due to the missing terms from id server)
image

-> this exception is because the endpoint _matrix/identity/v2/terms is blocked server side currently

  • Hide the first section "Serveur d'identité"
  • Update the second section by removing phone number references
    image
  • Hide the last section, phone number are not available
    image
@giomfo
Copy link
Contributor Author

giomfo commented Nov 19, 2021

I will mark this task as Done because the DINUM decided to hide for the moment the contact book access in Tchap
This section named "Discovery settings" will be hidden for the moment (we don't need to ask consent about id server to the user for the moment)

@giomfo giomfo assigned giomfo and unassigned Florian14 Nov 19, 2021
@giomfo giomfo added P1 and removed P3 labels Nov 19, 2021
@giomfo
Copy link
Contributor Author

giomfo commented Nov 19, 2021

I realized Element-Android requires this consent to trigger any lookup requests (see #246)
Without this consent, Element-Android doesn't reveal existing user while you fill a complete their email address in the new direct screen
image

In Tchap, I don't think we need this additional consent because the id server is forced by the Tchap infra. The consent to send email addresses to discover other Tchap users is granted when the user accept the global Tchap TAC (https://www.tchap.gouv.fr/tac/) , see the last section

@Anna-Livia @jdauphant what do you think? can we hide this additional consent request in Tchap?
if not we have to define tchap id server terms and open _matrix/identity/v2/terms endpoint on each id server

@Anna-Livia
Copy link

I'm having trouble understanding what the feature does.

What I understand the use case is:
While using Element, if I want to start a conversation with someone whom I never had a conversation with, I can use their email address to look them up and start a chat room.

Is that right ?

Then, in Tchap, once you do that, the app will show the display name of the user that have that email adresse and allow you to send them an invite.

However, and this is where I get confused, what happens in element android ?

@giomfo
Copy link
Contributor Author

giomfo commented Nov 19, 2021

Element and Tchap are currently doing the same: they are waiting for user's consent before sending the email to the server and eventually help you to discover an existing user, this prevent you from inviting them by email. You will directly start a chat in Tchap.

My question here is should we keep the requirement about the user's consent before sending email to the server.
It was not the case in TchapV1 because we considered accepted TAC was enough.

The Element-Android UI provide us a way to prompt the user before sending email to the server. We may keep it if you think this is useful/required in Tchap

@jdauphant
Copy link
Contributor

For me the user is ok to send the email currently typed.
(but not the contacts repository)

This will simplify the implementation?

@giomfo
Copy link
Contributor Author

giomfo commented Nov 25, 2021

@jdauphant yes, we will just hide this section from the general settings, so we will not have to customise the screen.

I will unblock this issue by changing the demand

  • hide section "Discovery settings" from General settings

@Florian14 can you please drop your changes suggested in #PR 267, and prepare a new PR for the above request

@giomfo giomfo changed the title [Settings] Adjust General/Discovery settings [Settings] ~~Adjust~~ Hide General/Discovery settings Nov 25, 2021
@giomfo giomfo assigned Florian14 and unassigned jdauphant, giomfo and Anna-Livia Nov 25, 2021
@giomfo giomfo removed the blocked label Nov 25, 2021
@giomfo giomfo changed the title [Settings] ~~Adjust~~ Hide General/Discovery settings [Settings] Hide (previously Adapt) General/Discovery settings Nov 25, 2021
@giomfo giomfo closed this as completed Dec 13, 2021
@jdauphant
Copy link
Contributor

In Element Android FOSS, is that possible to auto-hide the menu if the matrix server return 403 ?

@giomfo
Copy link
Contributor Author

giomfo commented Dec 14, 2021

@jdauphant no, this menu doesn't depend only on the terms of the id server.
We may discuss in direct about the discovery feature which is enabled by default in Tchap now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants