Paths

Table of Contentst

Differential D12591

Calendar events should now have a 'Name' field.
ClosedPublic
Actions

Authored by lpriestley on Apr 28 2015, 3:23 PM.

Tags

None

Referenced Files

	F14053623: D12591.diff
	Fri, Nov 15, 5:57 PM

	F14035753: D12591.id30231.diff
	Sun, Nov 10, 7:24 AM

	F14021261: D12591.diff
	Wed, Nov 6, 6:52 AM

	F13974829: D12591.diff
	Oct 18 2024, 7:34 AM

	Unknown Object (File)
	Oct 4 2024, 4:39 AM

	Unknown Object (File)
	Sep 25 2024, 6:27 PM

	Unknown Object (File)
	Sep 20 2024, 3:49 AM

	Unknown Object (File)
	Sep 17 2024, 8:08 AM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T7953: Give calendar events names

Commits

Restricted Diffusion Commit
rP43ab0e879fd9: Calendar events should now have a 'Name' field.

Summary

Closes T7953, Calendar events should now have a 'Name' field.

Test Plan

Create or edit event with no title, save event, should get error requiring name, event detail view timeline should reflect name changes.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 30230.Apr 28 2015, 3:23 PM

lpriestley retitled this revision from to Calendar events should now have a 'Name' field..

lpriestley updated this object.

lpriestley edited the test plan for this revision. (Show Details)

lpriestley added a reviewer: epriestley.

lpriestley added a task: T7953: Give calendar events names.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 28 2015, 3:23 PM

Herald added subscribers: epriestley, Korvin. · View Herald Transcript

Minor inlines.

src/applications/calendar/controller/PhabricatorCalendarEventViewController.php
105–107	Nuke.
src/applications/calendar/storage/PhabricatorCalendarEventTransaction.php
78	Here or in a future diff, maybe render: %s created this event. ...if `$old === null`. You'd also have to remove TYPE_NAME from shouldHide() if you do this.

This revision is now accepted and ready to land.Apr 28 2015, 3:29 PM

Closed by commit rP43ab0e879fd9: Calendar events should now have a 'Name' field. (authored by lpriestley, committed by lpriestley). · Explain WhyApr 28 2015, 3:34 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20150428.calendar.1.name.sql

2 lines

src/

applications/

calendar/

controller/

PhabricatorCalendarEventEditController.php

109 lines

PhabricatorCalendarEventViewController.php

9 lines

editor/

PhabricatorCalendarEventEditor.php

37 lines

storage/

PhabricatorCalendarEvent.php

2 lines

PhabricatorCalendarEventTransaction.php

27 lines

Diff 30231

resources/sql/autopatches/20150428.calendar.1.name.sql

Loading...

src/applications/calendar/controller/PhabricatorCalendarEventEditController.php

Loading...

src/applications/calendar/controller/PhabricatorCalendarEventViewController.php

Loading...

src/applications/calendar/editor/PhabricatorCalendarEventEditor.php

Loading...

src/applications/calendar/storage/PhabricatorCalendarEvent.php

Loading...

src/applications/calendar/storage/PhabricatorCalendarEventTransaction.php

Loading...