fix: correct discriminated union type for forward/backward pagination

[wschurman]

Why

When testing this downstream, I noticed that it was allowed in types to specify first & before / last & after which aren't supported in the pagination.

How

Fix the types by being explicit about the other fields.

Test Plan

yarn tsc

[wschurman]

• fix: correct pagination behavior for standard pagination with descending ordering clauses #477
• fix: correct precedence parentheses for SQLFragment joins #476
• feat: add method to get pagination cursor for single entity #475
• fix: correct discriminated union type for forward/backward pagination #474 👈 (View in Graphite)
• feat: support nullable search fields with postgres transforms #470
• fix: Apply same viewer context subclass type mechanism to knexLoader and related methods #472
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[ide]

Interesting. Makes sense.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (53b0604) to head (bf1c639).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #474   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          110       110           
  Lines        16135     16159   +24     
  Branches       869       869           
=========================================
+ Hits         16135     16159   +24     

Flag	Coverage Δ
integration	23.98% <100.00%> (+0.11%)	⬆️
unittest	95.04% <100.00%> (+<0.01%)	⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[wschurman]

Merge activity

• Feb 27, 2:15 AM UTC: A user started a stack merge that includes this pull request via Graphite.
• Feb 27, 2:21 AM UTC: Graphite rebased this pull request as part of a merge.
• Feb 27, 2:23 AM UTC: @wschurman merged this pull request with Graphite.