refactor: Improve Omnichannel queries

[KevLehman]

Proposed changes (including videos or screenshots)

• Move queries to models
• Correct fields for projection
• Use correct projection for places where we aren't using the full obj

Issue(s)

Steps to test or reproduce

Further comments

[changeset-bot]

⚠️ No Changeset found

Latest commit: aa4b412

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[codecov]

Codecov Report

Merging #29711 (aa4b412) into develop (fe6f5cd) will decrease coverage by 0.01%.
Report is 6 commits behind head on develop.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop   #29711      +/-   ##
===========================================
- Coverage    50.37%   50.37%   -0.01%     
===========================================
  Files          747      745       -2     
  Lines        14431    14400      -31     
  Branches      2581     2578       -3     
===========================================
- Hits          7270     7254      -16     
+ Misses        6773     6758      -15     
  Partials       388      388              

Flag	Coverage Δ
e2e	47.81% <ø> (-0.01%)	⬇️
unit	67.75% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

[sampaiodiego]

this is great! 👏 thx a lot! 🤗

just one overall comment, I think when using projection it is important to provide the correct returning type using generics.. otherwise TS will always return the full document, which can cause even more issues.

I know the generic is already there for findOneById<T>(...), so for places using it it would be just a matter of providing the returning type.. but for the new methods you'd need to add the generics too.

stale