search

pjvds / ncqrs

Ncqrs Framework - The CQRS Framework for .NET
Other
539 stars 164 forks source link

Two Minor Changes #94

Closed theBoringCoder closed 11 years ago

theBoringCoder commented 11 years ago

First, I updated Ncqrs.Eventing.Storage.SQL.TableCreationScript.sql so that [dbo] does not appear on the table names. This allows the eventing tables to be created and used in schemas other than [dbo]--the trick being that the user account used to connect to SQL Server must have it's default schema set to the schema where you want the tables created, and the user account must be in the db_owner role for the database.

Second, I changed the "SaveShapshot" method of the ISnapshotStore interface to "SaveSnapshot", because, based on the name of the interface and it's one other method, "Shapshot" was obviously just a misspelling of "Snapshot".

pjvds commented 11 years ago

Both make definitely sense. Thank you very much for your contribution!

theBoringCoder commented 11 years ago

My pleasure to contribute. Will the NuGet package be updated?