Page MenuHomePhabricator
Differential D15779

Calendar event list items 'Attending:' field should only show users who have confirmed attendance
ClosedPublic
Actions

Authored by lpriestley on Apr 21 2016, 5:21 PM.
Tags
None
Referenced Files
F15518670: D15779.id.diff
Sat, Apr 19, 3:39 PM
F15516497: D15779.diff
Fri, Apr 18, 7:13 PM
F15445837: D15779.id.diff
Thu, Mar 27, 4:11 PM
F15436140: D15779.id38017.diff
Tue, Mar 25, 12:13 PM
F15403585: D15779.diff
Mar 18 2025, 4:00 AM
F15402079: D15779.diff
Mar 17 2025, 8:30 PM
F15399760: D15779.id38017.diff
Mar 17 2025, 7:28 AM
Unknown Object (File)
Mar 1 2025, 2:10 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
chad
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8897: "Attending" in Calendar list view shows all users, not just attending users
Commits
rPbd8969a23cd9: Calendar event list items 'Attending:' field should only show users who have…
Summary

Fixes T8897

Test Plan

Open any list view of Calendar events, every event should only show "Attending: ..." with users who are attending event.

Diff Detail

Repository
rP Phabricator
Branch
calendarlistitem
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 11885
Build 14927: Run Core Tests
Build 14926: arc lint + arc unit

Event Timeline

lpriestley updated this revision to Diff 38014.Apr 21 2016, 5:21 PM
lpriestley retitled this revision from to Calendar event list items 'Attending:' field should only show users who have confirmed attendance.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added reviewers: epriestley, chad.
lpriestley added a task: T8897: "Attending" in Calendar list view shows all users, not just attending users.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 21 2016, 5:21 PM
Herald added a subscriber: Korvin. · View Herald Transcript
epriestley accepted this revision.Apr 21 2016, 5:31 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 21 2016, 5:31 PM
Closed by commit rPbd8969a23cd9: Calendar event list items 'Attending:' field should only show users who have… (authored by lpriestley, committed by lpriestley). · Explain WhyApr 21 2016, 6:11 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
calendar/
query/
5 lines

Diff 38014

View Options

src/applications/calendar/query/PhabricatorCalendarEventSearchEngine.php

Loading...
Phabricator · Built by Phacility