-
-
Notifications
You must be signed in to change notification settings - Fork 9
Conversation
/test/components/views/dialogs/security/ @matrix-org/element-crypto-web-reviewers | ||
/src/stores/SetupEncryptionStore.ts @matrix-org/element-crypto-web-reviewers | ||
/test/stores/SetupEncryptionStore-test.ts @matrix-org/element-crypto-web-reviewers | ||
/src/SecurityManager.ts @element-hq/element-crypto-web-reviewers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this team doesn't exist on element-hq.
Should that be created or use element-crypto for now @richvdh
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should do, I just created it: https://github.com/orgs/element-hq/teams/element-crypto-web-reviewers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess it can't be a secret team - I've changed it to public but it might take a while for GH to realise and update the codeowners file check.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope, obviously it was because I forgot to add the team's access to the repo 🤦
|
Checklist
public
/exported
symbols have accurate TSDoc documentation.