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

[FIX] Disabling not supported features in federated channels #27204

Conversation

MarcosSpessatto
Copy link
Contributor

@MarcosSpessatto MarcosSpessatto commented Nov 7, 2022

Proposed changes (including videos or screenshots)

Jira: FED-51

  • Disables OTR in federated rooms
  • Disable E2E encryption in federated rooms
  • Adds a validation in the user's auto-complete, making sure only the username using @ and the matrix syntax are caught by the input.
  • Disables the possibility to update the avatar for federated rooms (on Admin)
  • Disables the edition for (almost) all federated room properties (on Admin), EXCEPT name and topic
  • Disables the possibility to delete federated rooms.
  • Disables the invite link button on the Room members tab for federated rooms.

Issue(s)

Steps to test or reproduce

Further comments

@codecov
Copy link

codecov bot commented Nov 7, 2022

Codecov Report

Merging #27204 (da32e5e) into feat/federation-feat-2 (cdc2f99) will increase coverage by 0.03%.
The diff coverage is 71.42%.

❗ Current head da32e5e differs from pull request most recent head 0382800. Consider uploading reports for the commit 0382800 to get more accurate results

Impacted file tree graph

@@                    Coverage Diff                     @@
##           feat/federation-feat-2   #27204      +/-   ##
==========================================================
+ Coverage                   40.94%   40.98%   +0.03%     
==========================================================
  Files                         838      838              
  Lines                       17677    17683       +6     
  Branches                     1983     1985       +2     
==========================================================
+ Hits                         7238     7247       +9     
+ Misses                      10201    10194       -7     
- Partials                      238      242       +4     
Flag Coverage Δ
e2e 40.98% <71.42%> (+0.03%) ⬆️

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

@MarcosSpessatto MarcosSpessatto added this to the 5.4.0 milestone Nov 16, 2022
@MarcosSpessatto MarcosSpessatto marked this pull request as ready for review November 16, 2022 18:29
@MarcosSpessatto MarcosSpessatto requested a review from a team as a code owner November 16, 2022 18:29
@MarcosSpessatto MarcosSpessatto removed this from the 5.4.0 milestone Nov 17, 2022
@RocketChat RocketChat deleted a comment from lgtm-com bot Nov 19, 2022
@RocketChat RocketChat deleted a comment from lgtm-com bot Nov 19, 2022
@MarcosSpessatto MarcosSpessatto added this to the 6.0.0 milestone Nov 23, 2022
Copy link
Member

@yash-rajpal yash-rajpal left a comment

Choose a reason for hiding this comment

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

Shouldn't we also block "Enable E2E" room action for federated rooms?

image

@MarcosSpessatto MarcosSpessatto merged commit 3cd3696 into feat/federation-feat-2 Jan 16, 2023
@MarcosSpessatto MarcosSpessatto deleted the fix/federation-fix-channel-visual-elements branch January 16, 2023 18:32
gabriellsh added a commit that referenced this pull request Jan 17, 2023
* feat/federation-search-public-rooms: (21 commits)
  [FIX] Disabling not supported features in federated channels (#27204)
  fix: missing files
  fix: add the auto-join feature back using method call
  fix: add back the auto join feature after refactoring
  fix: do not wait to publish it to directory
  test: adding unit test cases
  chore: fix lint
  fix: prefer use rest to avoid collision after merge with other PRs
  fix: do not show join button on federated rooms if CE
  fix: do not set room topic if its empty
  fix: reverting last change + unit tests
  feat: allow users auto-join user the RC auto-join button
  fix: fixing bad typing
  test: fix broken test
  feat: small tweaks
  feat: adding support to save federation server names search by user
  test: adding unit tests for search public rooms feature
  fix: make it possible with  synapse as well (dendrite and synapse behaves different)
  feat: federation public rooms search (WIP)
  feat: endpoints to search and join public rooms (WIP)
  ...
gabriellsh added a commit that referenced this pull request Jan 23, 2023
…etChat/Rocket.Chat into matrixSearch

* 'feat/federation-search-public-rooms' of github.com:RocketChat/Rocket.Chat: (23 commits)
  fix: change errors strings
  fix: final tweaks when user join the public room by himself
  [FIX] Disabling not supported features in federated channels (#27204)
  fix: missing files
  fix: add the auto-join feature back using method call
  fix: add back the auto join feature after refactoring
  fix: do not wait to publish it to directory
  test: adding unit test cases
  chore: fix lint
  fix: prefer use rest to avoid collision after merge with other PRs
  fix: do not show join button on federated rooms if CE
  fix: do not set room topic if its empty
  fix: reverting last change + unit tests
  feat: allow users auto-join user the RC auto-join button
  fix: fixing bad typing
  test: fix broken test
  feat: small tweaks
  feat: adding support to save federation server names search by user
  test: adding unit tests for search public rooms feature
  fix: make it possible with  synapse as well (dendrite and synapse behaves different)
  ...
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Feb 13, 2023
@sampaiodiego sampaiodiego mentioned this pull request Mar 9, 2023
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
squad: federation stat: QA tested stat: ready to merge PR tested and approved waiting for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants