Page MenuHomePhabricator

Replace "loadDraftAndPublishedComments()" with a Query
ClosedPublic

Authored by epriestley on May 7 2020, 6:56 PM.
Tags
None
Referenced Files
F15426133: D21228.id50564.diff
Sun, Mar 23, 7:22 AM
F15332455: D21228.diff
Fri, Mar 7, 5:31 PM
Unknown Object (File)
Feb 21 2025, 1:11 PM
Unknown Object (File)
Feb 9 2025, 9:14 AM
Unknown Object (File)
Feb 9 2025, 9:14 AM
Unknown Object (File)
Feb 9 2025, 9:14 AM
Unknown Object (File)
Feb 9 2025, 9:14 AM
Unknown Object (File)
Jan 25 2025, 1:02 PM
Subscribers
None

Details

Summary

Ref T13513. Continue marching toward coherent query pathways for all access to inline comments.

Test Plan
  • Viewed a commit and a path within that commit, as a user with unpublished inlines and a different user.
  • Saw appropriate inlines in all cases (published inlines, plus undeleted unpublished inlines authored by the current viewer).
  • Grepped for "loadDraftAndPublishedComments()".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.May 7 2020, 11:02 PM
This revision was automatically updated to reflect the committed changes.