Page MenuHomePhabricator

Respect "End Date" for recurring Calendar events in detail UI
ClosedPublic

Authored by epriestley on Sep 8 2016, 4:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 3:01 AM
Unknown Object (File)
Wed, Apr 17, 5:47 PM
Unknown Object (File)
Tue, Apr 16, 6:44 PM
Unknown Object (File)
Tue, Apr 16, 12:56 PM
Unknown Object (File)
Sun, Apr 14, 2:26 AM
Unknown Object (File)
Sat, Apr 13, 2:08 AM
Unknown Object (File)
Sun, Mar 31, 11:15 AM
Unknown Object (File)
Feb 6 2024, 5:39 AM
Subscribers
None
Tokens
"Love" token, awarded by 20after4.

Details

Summary

Fixes T11396. Currently, you can keep clicking "Next >" forever to generate infinite instances of an event, even if it has a set end date.

Likewise, you can visit /E123/999999 or whatever to stub out the 999999th instance of an event.

Instead:

  • Before creating a new stub, make sure it happens before any end date.
  • 404 stubs if we can't create them.
  • Disable the "Next >" button if it isn't valid.
Test Plan
  • Visited /E123/9999 for an event with a recurrence end date, got 404.
  • Clicked "Next >" on an event with an end date, got new events until I hit the end date.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Respect "End Date" for recurring Calendar events in detail UI.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 8 2016, 4:47 PM
This revision was automatically updated to reflect the committed changes.