Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NHibernate] Update Setup #61

Closed
alecwhittington opened this issue Oct 5, 2012 · 6 comments
Closed

[NHibernate] Update Setup #61

alecwhittington opened this issue Oct 5, 2012 · 6 comments
Assignees
Milestone

Comments

@alecwhittington
Copy link
Member

Get rid of NHibernateInitializer in favor of a Windsor Facility. This will eliminate Session Storage amongst other things, making initialization simpler.

@ghost ghost assigned alecwhittington Oct 5, 2012
@cd21h
Copy link

cd21h commented Mar 1, 2015

Hi, any updates on this?

@cd21h
Copy link

cd21h commented Apr 13, 2015

@alecwhittington I like the idea.
This would be quite useful for when using SharpArch with other framework supporting nested scopes (e.g. MassTransit, Quartz..Net).

have you done any work on this?

@seif
Copy link
Member

seif commented Apr 13, 2015

Don't think any work has been done on this. Alec is out, will need to
figure out a way to handle this.

I'm on holiday flying back tomorrow, maybe we should schedule a hangout for
the weekend and discuss things? I'll email in the next 2 days.

Apologies in advance for any auto correction or spelling mistakes, sent
from my phone.
On 14 Apr 2015 01:14, "Vitaliy K" notifications@github.com wrote:

@alecwhittington https://github.com/alecwhittington I like the idea.
This would be quite useful for when using SharpArch with other framework
supporting nested scopes (e.g. MassTransit, Quartz..Net).

have you done any work on this?


Reply to this email directly or view it on GitHub
#61 (comment)
.

@cd21h
Copy link

cd21h commented Apr 16, 2015

Yep, sounds good.
I'm in EST time zone

Regards,
Vitaliy

On Mon, Apr 13, 2015 at 7:01 PM, Seif Attar notifications@github.com
wrote:

Don't think any work has been done on this. Alec is out, will need to
figure out a way to handle this.

I'm on holiday flying back tomorrow, maybe we should schedule a hangout for
the weekend and discuss things? I'll email in the next 2 days.

Apologies in advance for any auto correction or spelling mistakes, sent
from my phone.
On 14 Apr 2015 01:14, "Vitaliy K" notifications@github.com wrote:

@alecwhittington https://github.com/alecwhittington I like the idea.
This would be quite useful for when using SharpArch with other framework
supporting nested scopes (e.g. MassTransit, Quartz..Net).

have you done any work on this?


Reply to this email directly or view it on GitHub
<
#61 (comment)

.


Reply to this email directly or view it on GitHub
#61 (comment)
.

@seif seif modified the milestone: 3.0 Apr 26, 2015
@seif
Copy link
Member

seif commented Apr 26, 2015

Talking with Vitaliy now, agreed to strip down to basic.

Remove the initialiser, change the way repositories work to have an ISession injected in.

The multi-database functionality (curently implemented via the SessionFactoryKeyAttribute) will be out of scope initially.

Once the changes are made, we will need to come up with a way to inject the correct the ISession from the IoC registration.

THis related to #68 #69

@seif seif assigned cd21h and unassigned alecwhittington Apr 26, 2015
@seif seif added this to the 4.0 milestone Apr 26, 2015
@seif
Copy link
Member

seif commented Apr 26, 2015

Also #85

cd21h added a commit to alphacloud/Sharp-Architecture that referenced this issue May 31, 2015
cd21h added a commit to alphacloud/Sharp-Architecture that referenced this issue May 31, 2015
cd21h added a commit to alphacloud/Sharp-Architecture that referenced this issue May 31, 2015
…harparchitecture#61

* ValidatableObject now requires ValidationContext;
* HasUniqueDomainSignatureAttribute now requires ValidationContext;
* DataAnnotationsEventListener how provides IServiceProvider with ISession;

* TardisBank - use CastleWindsor to register ISession and ISessionFactory;
* TardisBank - update CastleWindsor installers.
cd21h added a commit to alphacloud/Sharp-Architecture that referenced this issue Jun 3, 2015
cd21h added a commit to alphacloud/Sharp-Architecture that referenced this issue Jun 3, 2015
cd21h added a commit to alphacloud/Sharp-Architecture that referenced this issue Jun 3, 2015
cd21h added a commit to alphacloud/Sharp-Architecture that referenced this issue Jun 29, 2015
cd21h added a commit to alphacloud/Sharp-Architecture that referenced this issue Jul 24, 2015
* Add NHibernateSessionFactoryBuilder, also fixes sharparchitecture#54
* Replace NHibernateSession with NHibernateSessionFactoryBuilder, see sharparchitecture#61
* ValidatableObject now requires ValidationContext;
* HasUniqueDomainSignatureAttribute now requires ValidationContext;
* DataAnnotationsEventListener how provides IServiceProvider with ISession;
* TardisBank - use CastleWindsor to register ISession and ISessionFactory;
* TardisBank - update CastleWindsor installers.
* Use DependencyResolver in ASP.NET MVC
* Inherit ILinqRepository from IRepository
seif added a commit that referenced this issue Jul 29, 2015
@cd21h cd21h added 3 - Done and removed 1 - Ready labels Nov 27, 2015
@cd21h cd21h closed this as completed Nov 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants