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 usage of unknown shield in room summary (PSG-1019) #7710

Merged
merged 3 commits into from
Dec 6, 2022

Conversation

onurays
Copy link
Contributor

@onurays onurays commented Dec 5, 2022

Type of change

  • Feature
  • Bugfix
  • Technical
  • Other :

Content

While we are developing new Session Management screens we introduced a new shield for devices which has an unknown e2e status. But this also accidentally effected the shield of rooms. This PR will revert these changes.

Motivation and context

Screenshots / GIFs

Tests

  • Login with an unverified session
  • Enable new session management in Labs settings
  • Navigate to Settings > Security & Privacy > Show All Sessions
  • See unknown shield ("?") for the current session
  • Navigate to an encrypted room details
  • See black shield next to the room name instead of an unknown shield

Tested devices

  • Physical
  • Emulator
  • OS version(s):

Checklist

@onurays onurays requested review from a team, Florian14 and mnaturel and removed request for a team December 5, 2022 15:18
@ElementBot
Copy link

Fails
🚫

node failed.

Log

Error:  Error: Could not find the Dangerfile at tools/danger/dangerfile-lint.js - does a file exist at dangerfile-lint.js in tools/danger?.
    at /usr/src/danger/distribution/platforms/GitHub.js:166:27
    at step (/usr/src/danger/distribution/platforms/GitHub.js:44:23)
    at Object.next (/usr/src/danger/distribution/platforms/GitHub.js:25:53)
    at fulfilled (/usr/src/danger/distribution/platforms/GitHub.js:16:58)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
danger-results://tmp/danger-results-77a114bb.json

Generated by 🚫 dangerJS against 32ded28

@onurays onurays changed the title Fix usage of unknown shield in room summary Fix usage of unknown shield in room summary (PSG-1019) Dec 5, 2022
Copy link
Contributor

@mnaturel mnaturel left a comment

Choose a reason for hiding this comment

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

There is a missing View to update with the render method for device: SessionInfoView. Otherwise LGTM, thanks for the fix!

@onurays onurays requested a review from mnaturel December 6, 2022 11:06
@onurays
Copy link
Contributor Author

onurays commented Dec 6, 2022

There is a missing View to update with the render method for device: SessionInfoView. Otherwise LGTM, thanks for the fix!

Nice catch, done, thank you.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 6, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

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

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@onurays onurays merged commit f76a6de into develop Dec 6, 2022
@onurays onurays deleted the feature/ons/fix_unknown_shield_icon_in_room branch December 6, 2022 14:51
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.

3 participants