-
Notifications
You must be signed in to change notification settings - Fork 109
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 some todo #2360
Fix some todo #2360
Conversation
It was used at the beginning of the project, but is not useful anymore.
…, not only for the error dialog.
class SessionDirectoryNameProvider { | ||
// Rust sanitises the user ID replacing invalid characters with an _ | ||
fun provides(sessionId: SessionId): String { | ||
return sessionId.value.replace(":", "_") |
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.
Once matrix-org/matrix-rust-sdk#3098 will be done, we may iterate on this.
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
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.
LGTM, thanks!
Small cleanup on the project, may be reviewed commit per commit.