Skip to content

31 delete order by#388

Open
alekseyTask wants to merge 5 commits intoAlexander-Shein:masterfrom
alekseyTask:31_DeleteOrderById
Open

31 delete order by#388
alekseyTask wants to merge 5 commits intoAlexander-Shein:masterfrom
alekseyTask:31_DeleteOrderById

Conversation

@alekseyTask
Copy link

pr31_1

Copy link
Owner

@Alexander-Shein Alexander-Shein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you run this SP? See foreign keys constraint.

Copy link
Owner

@Alexander-Shein Alexander-Shein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a transaction :)

Copy link
Owner

@Alexander-Shein Alexander-Shein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And try catch and rollback :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants