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
F14395622: D12781.diff
Sun, Dec 22, 4:48 AM
Unknown Object (File)
Sat, Dec 14, 10:09 PM
Unknown Object (File)
Sat, Dec 7, 11:52 AM
Unknown Object (File)
Wed, Nov 27, 3:28 PM
Unknown Object (File)
Nov 19 2024, 11:56 AM
Unknown Object (File)
Oct 23 2024, 12:08 PM
Unknown Object (File)
Oct 21 2024, 8:26 AM
Unknown Object (File)
Oct 16 2024, 6:00 PM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 30724

View Options

src/applications/calendar/editor/PhabricatorCalendarEventEditor.php

Loading...
View Options

src/applications/calendar/storage/PhabricatorCalendarEvent.php

Loading...
Phabricator · Built by Phacility