Page MenuHomePhabricator
Differential D12631

Calendar items in Upcoming Events query should link to events, not calendar
ClosedPublic
Actions

Authored by lpriestley on Apr 30 2015, 8:03 PM.
Tags
None
Referenced Files
F18864926: D12631.diff
Mon, Nov 3, 9:20 AM
F18862451: D12631.id.diff
Sun, Nov 2, 7:23 PM
F18835845: D12631.diff
Sun, Oct 26, 7:25 PM
F18834813: D12631.id30325.diff
Sun, Oct 26, 11:00 AM
F18790664: D12631.id30326.diff
Wed, Oct 15, 4:30 PM
F18720693: D12631.id.diff
Sep 29 2025, 7:37 PM
F18702916: D12631.diff
Sep 28 2025, 2:07 AM
F18650877: D12631.id30325.diff
Sep 21 2025, 2:07 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T7933: In list views, Calendar events link to months but should link to event detail pages
Commits
Restricted Diffusion Commit
rPf14e0bf2efb2: Calendar items in Upcoming Events query should link to events, not calendar
Summary

Closes T7933, Calendar items in Upcoming Events query should link to events, not calendar.

Test Plan

Open Upcoming Events query, click on an event, event detail view should open.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 30325.Apr 30 2015, 8:03 PM
lpriestley retitled this revision from to Calendar items in Upcoming Events query should link to events, not calendar.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T7933: In list views, Calendar events link to months but should link to event detail pages.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 30 2015, 8:03 PM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
epriestley accepted this revision.Apr 30 2015, 8:20 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 30 2015, 8:20 PM
Closed by commit rPf14e0bf2efb2: Calendar items in Upcoming Events query should link to events, not calendar (authored by lpriestley, committed by lpriestley). · Explain WhyApr 30 2015, 8:22 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
calendar/
query/
22 lines

Diff 30326

View Options

src/applications/calendar/query/PhabricatorCalendarEventSearchEngine.php

Loading...
Phabricator · Built by Phacility