openactive / OpenActive.Server.NET

.NET server library, including an OpenActive Reference Implementation
MIT License
0 stars 7 forks source link

Move the throwing of `OrderAlreadyExistsError` into the `StoreBookingEngine` #208

Open nickevansuk opened 1 year ago

nickevansuk commented 1 year ago

Moving the throwing of OrderAlreadyExistsError into the StoreBookingEngine, as per https://github.com/openactive/OpenActive.Server.NET/pull/145#discussion_r1284583361, is a much better abstraction.

It is a breaking change, however worthwhile if there are other breaking changes being released.