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.

Sign up for GitHub

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

Jump to bottom

Crash creating new DM when the new WYSIWYG is enabled #7689

Closed
giomfo opened this issue Oct 3, 2023 · 1 comment · Fixed by #7692
Closed

Crash creating new DM when the new WYSIWYG is enabled #7689

giomfo opened this issue Oct 3, 2023 · 1 comment · Fixed by #7692
Labels
A-DMs customer-retainer T-Defect Something isn't working: bugs, crashes, hangs and other reported problems

Comments

@giomfo
Copy link
Member

giomfo commented Oct 3, 2023

Steps to reproduce

  • Enable the new WYSIWYG editor in the application settings
  • Select start a new chat option
  • Select a user
  • Click on Start button

Outcome

What did you expect?

open the pending DM

What happened instead?

the app crashes.

Additional info:
Log: “Element/WysiwygInputToolbarView.swift:261: Fatal error: Unexpectedly found nil while implicitly unwrapping an Optional value”

The crash occurs here:
.environmentObject(AvatarViewModel(avatarService: AvatarService(mediaManager: toolbarViewDelegate.mediaManager())))

Your phone model

No response

Operating system version

No response

Application version

1.11.2

Homeserver

No response

Will you send logs?

No
@giomfo giomfo added T-Defect Something isn't working: bugs, crashes, hangs and other reported problems A-DMs labels Oct 3, 2023
@giomfo
Copy link
Member Author

giomfo commented Oct 3, 2023

The crash is due to the deferred creation of the room. Indeed the DM creation is deferred when the first message is sent

@alfogrillo alfogrillo mentioned this issue Oct 3, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-DMs customer-retainer T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
Issue triage
Status: Closed
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix new dm crash with RTE is enabled element-hq/element-ios
1 participant
@giomfo