Deferred DMs - Locals are not deleted anymore within the new AppLayout

[Florian14]

Steps to reproduce

First bug:

1. Start creating a DM with the start DM feature flag and the new App Layout enabled
2. Abort the DM creation and go back to the home list
3. Activate the recent rooms display from the AppLayout configuration
4. Then, the local room appears in the recent rooms

Second bug:

1. Start creating a DM with the start DM feature flag and the new App Layout enabled
2. Write some text without sending it
3. Abort the DM creation and go back to the home list
4. Start creating again a DM with the same person
5. Then, the local room is opened again with the drafted message

Outcome

What did you expect?

The local room should not be persisted anymore when the user is back to the room list

Your phone model

No response

Operating system version

No response

Application version and app store

No response

Homeserver

No response

Will you send logs?

No

Are you willing to provide a PR?

Yes

[daniellekirkwood]

This feels like good default behaviour (if the user is part way thru crafting a message, wants to switch rooms and then go back to that message, they are able to do so...)

@giomfo as someone close to the behaviour of start a DM (and acting PM!) what do you think? :D

[giomfo]

This feels like good default behaviour (if the user is part way thru crafting a message, wants to switch rooms and then go back to that message, they are able to do so...)

I agree the second "bug" described here looks like an expected behavior, but it was not planned in this first implementation. We decided to fix/remove it for the moment to prevent us from unstable behavior.
We would have to work on this use case. FYI we already discussed about this point in Element-web repo here. The plan would be to be able to rescue the pending message but keeping hidden the "local"(uncreated) room