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

Refactor non-conversation list using custom type support from #8247

Open
Tracked by #5051
charlag opened this issue Jan 10, 2025 · 0 comments · May be fixed by #8308
Open
Tracked by #5051

Refactor non-conversation list using custom type support from #8247

charlag opened this issue Jan 10, 2025 · 0 comments · May be fixed by #8308

Comments

@charlag
Copy link
Contributor

charlag commented Jan 10, 2025

Depends on #8222

Estimated: 20h
Time spent: 21h

@paw-hub paw-hub self-assigned this Jan 14, 2025
@hrb-hub hrb-hub self-assigned this Jan 16, 2025
hrb-hub added a commit that referenced this issue Jan 17, 2025
This removes MailViewModel having to keep track of MailSetEntry<->Mail,
and it removes a lot of legacy code for getting Mail vs MailSetEntry, as
all of the implementation details for getting mails are now in
MailListModel. Additionally, everyone uses mailsets, so the legacy code
is no longer necessary.

Closes #8247

Co-authored-by: hrb-hub <[email protected]>
hrb-hub added a commit that referenced this issue Jan 17, 2025
This removes MailViewModel having to keep track of MailSetEntry<->Mail,
and it removes a lot of legacy code for getting Mail vs MailSetEntry, as
all of the implementation details for getting mails are now in
MailListModel. Additionally, everyone uses mailsets, so the legacy code
is no longer necessary.

Closes #8247

Co-authored-by: hrb-hub <[email protected]>
@paw-hub paw-hub linked a pull request Jan 17, 2025 that will close this issue
paw-hub added a commit that referenced this issue Jan 17, 2025
This removes MailViewModel having to keep track of MailSetEntry<->Mail,
and it removes a lot of legacy code for getting Mail vs MailSetEntry, as
all of the implementation details for getting mails are now in
MailListModel. Additionally, everyone uses mailsets, so the legacy code
is no longer necessary.

Closes #8247

Co-authored-by: hrb-hub <[email protected]>
@hrb-hub hrb-hub linked a pull request Jan 17, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants