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

Feature: add E2EE Url Preview Passthrough in the room settings #5818

Closed
wants to merge 2,247 commits into from

Conversation

Baconxrp
Copy link

@Baconxrp Baconxrp commented Apr 22, 2022

Type of change

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

Content

add E2EE Url Preview Passthrough that was recently added to the web/desktop version of element in the room settings to both android and ios.

It was a change that allowed the user on desktop/web to be able to go into an ENCRYPTED room and have a choice to be able to see url previews if they wish to.

Motivation and context

The motivation is simple. I currently have EE2E URL preview passthrough on desktop/web when i talk to my friends at home and they send me links over dms and it is nice to see the preview on them that I chose to change the room setting on. I want to be able to also do this on my phone / mobile device.

Thank you!

Screenshots / GIFs

Screenshot_36_20220326100057_20220327064415~2

@github-actions
Copy link

github-actions bot commented Apr 22, 2022

Unit Test Results

122 files  +  46  122 suites  +46   2m 13s ⏱️ + 1m 15s
205 tests +  61  205 ✔️ +  61  0 💤 ±0  0 ±0 
690 runs  +238  690 ✔️ +238  0 💤 ±0  0 ±0 

Results for commit 0b00f97. ± Comparison against base commit 4ce1ab2.

This pull request removes 9 and adds 70 tests. Note that renamed tests count towards both.
org.matrix.android.sdk.api.pushrules.PushRuleActionsTest ‑ test_action_parsing
org.matrix.android.sdk.api.pushrules.PushRulesConditionTest ‑ test_displayName_condition
org.matrix.android.sdk.api.pushrules.PushRulesConditionTest ‑ test_eventmatch_cake_condition
org.matrix.android.sdk.api.pushrules.PushRulesConditionTest ‑ test_eventmatch_cakelie_condition
org.matrix.android.sdk.api.pushrules.PushRulesConditionTest ‑ test_eventmatch_path_condition
org.matrix.android.sdk.api.pushrules.PushRulesConditionTest ‑ test_eventmatch_type_condition
org.matrix.android.sdk.api.pushrules.PushRulesConditionTest ‑ test_eventmatch_words_only_condition
org.matrix.android.sdk.api.pushrules.PushRulesConditionTest ‑ test_notice_condition
org.matrix.android.sdk.api.pushrules.PushRulesConditionTest ‑ test_roommember_condition
im.vector.app.features.analytics.impl.DefaultVectorAnalyticsTest ‑ given lateinit user properties when valid analytics id updates then identify with lateinit properties
im.vector.app.features.analytics.impl.DefaultVectorAnalyticsTest ‑ given user consent when tracking events then submits to posthog
im.vector.app.features.analytics.impl.DefaultVectorAnalyticsTest ‑ given user consent when tracking screen events then submits to posthog
im.vector.app.features.analytics.impl.DefaultVectorAnalyticsTest ‑ given user has not consented when tracking events then does not track
im.vector.app.features.analytics.impl.DefaultVectorAnalyticsTest ‑ given user has not consented when tracking screen events then does not track
im.vector.app.features.analytics.impl.DefaultVectorAnalyticsTest ‑ when consenting to analytics then updates posthog opt out to false
im.vector.app.features.analytics.impl.DefaultVectorAnalyticsTest ‑ when revoking consent to analytics then updates posthog opt out to true
im.vector.app.features.analytics.impl.DefaultVectorAnalyticsTest ‑ when setting the analytics id then updates analytics store
im.vector.app.features.analytics.impl.DefaultVectorAnalyticsTest ‑ when setting user consent then updates analytics store
im.vector.app.features.analytics.impl.DefaultVectorAnalyticsTest ‑ when signing out then resets posthog
…

♻️ This comment has been updated with latest results.

@bmarty bmarty added the Z-Community-PR Issue is solved by a community member's PR label Apr 22, 2022
@bmarty
Copy link
Member

bmarty commented Apr 22, 2022

Hey @Baconxrp , thanks for your PR!

This diff is quite strange, can you ensure that the base branch is set to our develop branch?

Then we will be able to review it.

Thanks

abidin251 and others added 7 commits April 22, 2022 09:59
Currently translated at 40.2% (893 of 2217 strings)

Translation: Element Android/Element Android App
Translate-URL: https://translate.element.io/projects/element-android/element-app/ar/
Currently translated at 99.8% (2213 of 2217 strings)

Translation: Element Android/Element Android App
Translate-URL: https://translate.element.io/projects/element-android/element-app/et/
Currently translated at 100.0% (2217 of 2217 strings)

Translation: Element Android/Element Android App
Translate-URL: https://translate.element.io/projects/element-android/element-app/it/
Currently translated at 96.8% (2148 of 2217 strings)

Translation: Element Android/Element Android App
Translate-URL: https://translate.element.io/projects/element-android/element-app/ja/
Currently translated at 100.0% (2217 of 2217 strings)

Translation: Element Android/Element Android App
Translate-URL: https://translate.element.io/projects/element-android/element-app/zh_Hant/
Currently translated at 100.0% (57 of 57 strings)

Translation: Element Android/Element Android Store
Translate-URL: https://translate.element.io/projects/element-android/element-store/et/
Currently translated at 100.0% (57 of 57 strings)

Translation: Element Android/Element Android Store
Translate-URL: https://translate.element.io/projects/element-android/element-store/zh_Hant/
@Baconxrp
Copy link
Author

Hey @Baconxrp , thanks for your PR!

This diff is quite strange, can you ensure that the base branch is set to our develop branch?

Then we will be able to review it.

Thanks

Sorry, im unsure how to do that. Im just putting this out there as an individual who runs a matrix server and hasn't federated yet because this feature hasn't been fully implemented across all platforms. Been running our server for close to two years now. Hopefully it comes out so we can federate.

@Baconxrp Baconxrp changed the base branch from main to feature/bca/e2e_message_info April 22, 2022 11:26
Claire1817 and others added 6 commits April 22, 2022 16:19
Currently translated at 42.4% (942 of 2217 strings)

Translation: Element Android/Element Android App
Translate-URL: https://translate.element.io/projects/element-android/element-app/ar/
Currently translated at 100.0% (2217 of 2217 strings)

Translation: Element Android/Element Android App
Translate-URL: https://translate.element.io/projects/element-android/element-app/fa/
ericdecanini and others added 27 commits May 5, 2022 19:33
Bumps [libphonenumber](https://github.com/google/libphonenumber) from 8.12.47 to 8.12.48.
- [Release notes](https://github.com/google/libphonenumber/releases)
- [Changelog](https://github.com/google/libphonenumber/blob/master/making-metadata-changes.md)
- [Commits](google/libphonenumber@v8.12.47...v8.12.48)

---
updated-dependencies:
- dependency-name: com.googlecode.libphonenumber:libphonenumber
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
…ode.libphonenumber-libphonenumber-8.12.48

Bump libphonenumber from 8.12.47 to 8.12.48
Reformats project based on editorconfig
…rendering_issue_5151

Feature/aris/threads improve rendering issue 5151
* Fork sonarqube run into a nightly build, report failures back to channel.
* Each PR triggers a build after merge, report failures back to channel.
…st-post-pr

noop change to trigger another PR
…_to_test_post_pr

noop change to test Post PR merging
…eck_post_pr_check

noop change to README to test post-pr merging
@bmarty
Copy link
Member

bmarty commented May 9, 2022

Sorry, there is a pb with this PR, please create it again

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.