search

JDNdeveloper / Off-By-One

CMPS 115 Project: Jayden Navarro, John Gemignani, Alex Gonzalez, Kevin Cheng, Josh Innis
0 stars 0 forks source link

Set proper container keys when event is created inside of an event #104

Closed JDNdeveloper closed 9 years ago

JDNdeveloper commented 9 years ago

Notifications need their Reminder container key and their lifestyle container key set when their created inside an event i.e. not unsorted. Reminder needs its Lifestyle container key set.

JoshInnis commented 9 years ago

Doesn't this do this already. Whats going wrong with it?

JDNdeveloper commented 9 years ago

It looks like it does. Just thoroughly test that it does it for all event types and then close it. On Mar 1, 2015 2:09 PM, "JoetheSoda" notifications@github.com wrote:

Doesn't this do this already. Whats going wrong with it?

— Reply to this email directly or view it on GitHub https://github.com/JDNdeveloper/Off-By-One/issues/104#issuecomment-76634125 .

JDNdeveloper commented 9 years ago

When you add a notification inside of a reminder does it set both container keys? The parent lifestyle and parent reminder? On Mar 1, 2015 2:09 PM, "JoetheSoda" notifications@github.com wrote:

Doesn't this do this already. Whats going wrong with it?

— Reply to this email directly or view it on GitHub https://github.com/JDNdeveloper/Off-By-One/issues/104#issuecomment-76634125 .

JoshInnis commented 9 years ago

yes it does