Sprint 002 issue 150

[br2490]

SANITY CHECK, DO NOT MERGE.
First attempt to contribute to microservices.

Addressing #150 , delete all oreProxy when delete is called on resource service.

Looking for feedback, please! I will continue to work on this by wrapping in a TX, adding a return (which I'm not sure what I should be returning), and removing debug lines. Am I in the right direction? Is it okay that I'm using a pseudo queue to gather all of objects before actually deleting them? This is not how it is done in other services so if I should align myself with how other services behave I will. Anything?! Thanks :)

[br2490]

Also I'll make sure to pull from a clean branch...

[ruebot]

I think you're fine on this one. In the future, just make sure you do a pull before you start your work, and it's a good idea to do a pull before you do the pull request just to make sure you're not going to have any merge conflicts.

[br2490]

Agreed and thanks.

Sent from my phone. Forgive brevity and autocorrect.
On Apr 25, 2016 3:04 PM, "Nick Ruest" notifications@github.com wrote:

I think you're fine on this one. In the future, just make sure you do a
pull before you start your work, and it's a good idea to do a pull before
you do the pull request just to make sure you're not going to have any
merge conflicts.

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#206 (comment)

[br2490]

@whikloj when you have time could you look at a7b1fc6

[ruebot]

Superseded by #210