Page MenuHomePhabricator
Differential D12781

Trim transaction overload when converting events from all-day and back
ClosedPublic
Actions

Authored by lpriestley on May 8 2015, 11:40 PM.
Tags
None
Referenced Files
F19508213: D12781.diff
Sat, Jan 10, 2:14 AM
F19093271: D12781.id.diff
Dec 3 2025, 9:13 PM
F19090410: D12781.diff
Dec 3 2025, 11:44 AM
F18884979: D12781.id30723.diff
Nov 7 2025, 2:32 AM
F18843504: D12781.diff
Oct 28 2025, 10:43 PM
F18835304: D12781.diff
Oct 26 2025, 2:54 PM
F18830783: D12781.id.diff
Oct 25 2025, 8:46 AM
F18827695: D12781.diff
Oct 24 2025, 11:33 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8136: Extra transactions for "converting from all day event"
Commits
Restricted Diffusion Commit
rPdd8e4ff056af: Trim transaction overload when converting events from all-day and back
Summary

Closes T8136, Trim transaction overload when converting events from all-day and back

Test Plan

Create new event, save, edit, change to all-day, save, remove all-day flag, save. Feed should not show "end date changed" transaction

Diff Detail

Repository
rP Phabricator
Branch
calendartransactionoverload
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5874
Build 5894: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

lpriestley updated this revision to Diff 30723.May 8 2015, 11:40 PM
lpriestley retitled this revision from to Trim transaction overload when converting events from all-day and back.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T8136: Extra transactions for "converting from all day event".
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 8 2015, 11:40 PM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
epriestley accepted this revision.May 8 2015, 11:41 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 8 2015, 11:41 PM
Closed by commit rPdd8e4ff056af: Trim transaction overload when converting events from all-day and back (authored by lpriestley, committed by lpriestley). · Explain WhyMay 9 2015, 12:04 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
calendar/
editor/
5 lines
storage/
2 lines

Diff 30723

View Options

src/applications/calendar/editor/PhabricatorCalendarEventEditor.php

Loading...
View Options

src/applications/calendar/storage/PhabricatorCalendarEvent.php

Loading...
Phabricator · Built by Phacility