Skip to content
This repository was archived by the owner on Nov 5, 2025. It is now read-only.

Comments

feat: Introduce new msg interface and correct the tests#767

Merged
d-gubert merged 7 commits intoalphafrom
new/msg_raw_type
May 28, 2024
Merged

feat: Introduce new msg interface and correct the tests#767
d-gubert merged 7 commits intoalphafrom
new/msg_raw_type

Conversation

@Dnouv
Copy link
Member

@Dnouv Dnouv commented May 24, 2024

What? ⛵

  • Added a new interface, IMessageRaw for the getMessages method
  • Correct the tests for getMessages with the new type
  • Add a common createAttachments method

Why? 🤔

  • Since the IMessage required making additional DB trips
  • The createAttachments method avoids duplicating the same attachment code chunk across two methods.

Links 🌎

PS 👀

@Dnouv Dnouv requested review from d-gubert and removed request for d-gubert May 24, 2024 15:04
Dnouv and others added 3 commits May 24, 2024 20:35
Co-authored-by: Douglas Gubert <douglas.gubert@gmail.com>
Co-authored-by: Douglas Gubert <douglas.gubert@gmail.com>
@d-gubert d-gubert merged commit 5284a5f into alpha May 28, 2024
@d-gubert d-gubert deleted the new/msg_raw_type branch May 28, 2024 17:54
Dnouv added a commit that referenced this pull request Jun 20, 2024
Co-authored-by: Douglas Gubert <douglas.gubert@gmail.com>
@d-gubert d-gubert mentioned this pull request Jul 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants