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

Add Ente #8230

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Add Ente #8230

wants to merge 7 commits into from

Conversation

fleish
Copy link

@fleish fleish commented Oct 23, 2024

This pull request adds ente.io, which offers end-to-end encrypted file/photo sharing as well as an TOTP Authenticator app

Signed-off-by: fleish <[email protected]>
Signed-off-by: fleish <[email protected]>
Remove "Unnecessary img element"

Signed-off-by: fleish <[email protected]>
Ran file through svgo

Signed-off-by: fleish <[email protected]>
Minimizing file to one line

Signed-off-by: fleish <[email protected]>
@hkamran80 hkamran80 added the add site Issue/PR adds a site to the repo. label Oct 23, 2024
"tfa": [
"totp"
],
"documentation": "https://ente.io/faq/security-and-privacy/two-factor-support/",
Copy link
Member

Choose a reason for hiding this comment

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

This URL doesn't explain how to set up MFA.

Suggested change
"documentation": "https://ente.io/faq/security-and-privacy/two-factor-support/",

Copy link
Author

Choose a reason for hiding this comment

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

It doesn't go step by step, but it does provide a pointer on where to go to set it up. I've already set it up on my client, but you can see it here on the desktop client

Screenshot 2024-10-24 at 9 12 58 AM Screenshot 2024-10-24 at 9 13 07 AM

Copy link
Member

Choose a reason for hiding this comment

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

If it provided step-by-step instructions like Calendly, for example, then the documentation URL could be added.

Plus, a good assumption for where MFA configuration would be is in settings.

Comment on lines +9 to +11
"backup",
"cloud",
"security"
Copy link
Member

Choose a reason for hiding this comment

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

The cloud category is for serverless cloud computing and security is for "online security and anti-malware services". For more information, check out the categories list.

Suggested change
"backup",
"cloud",
"security"
"backup"

Copy link
Author

Choose a reason for hiding this comment

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

I agree cloud is a mis-tag, but backup also mentions "cross-device file synchronization" which I believe applies to the Ente app

Copy link
Member

Choose a reason for hiding this comment

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

Backup is still in the suggestion. I removed cloud and security.

img/e/ente.io.svg Outdated Show resolved Hide resolved
entries/e/ente.io.json Show resolved Hide resolved
@fletcherdodd fletcherdodd changed the title Adding Ente.io Add Ente Oct 24, 2024
fleish and others added 2 commits October 24, 2024 09:18
Co-authored-by: H. Kamran <[email protected]>
Signed-off-by: fleish <[email protected]>
Co-authored-by: H. Kamran <[email protected]>
Signed-off-by: fleish <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add site Issue/PR adds a site to the repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants