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

Allow HubConnections to be used in Crank #3099

Merged
merged 1 commit into from Oct 10, 2014
Merged

Allow HubConnections to be used in Crank #3099

merged 1 commit into from Oct 10, 2014

Conversation

ermontgo
Copy link
Contributor

  • Created IConnectionFactory as injection point for developers to
    customize how a Connection is created
  • Created DefaultConnectionFactory to retain behavior of creating a
    regular Connection
  • Modified Crank client to compose the IConnectionFactory dependency
    using MEF. If no candidate is supplied by an external assembly, the
    client will automatically use the DefaultConnectionFactory
    Crank: Update tool to use Hubs connection instead of just PersistentConnection #3000

- Created IConnectionFactory as injection point for developers to
customize how a Connection is created
- Created DefaultConnectionFactory to retain behavior of creating a
regular Connection
- Modified Crank client to compose the IConnectionFactory dependency
using MEF. If no candidate is supplied by an external assembly, the
client will automatically use the DefaultConnectionFactory

SignalR#3000
@davidfowl
Copy link
Member

@ermontgo Did you sign a CLA?

@ermontgo
Copy link
Contributor Author

ermontgo commented Jul 3, 2014

Yes I have. I have the confirmation email and info if you need anything.

@Eilon
Copy link

Eilon commented Jul 16, 2014

@davidfowl @pfletcher CLA is confirmed.

@DamianEdwards DamianEdwards assigned mikary and unassigned Eilon Oct 10, 2014
@DamianEdwards DamianEdwards added this to the 2.2.0 milestone Oct 10, 2014
@mikary
Copy link
Contributor

mikary commented Oct 10, 2014

Verified that factory composition works with both default and custom factories.

mikary added a commit that referenced this pull request Oct 10, 2014
Allow HubConnections to be used in Crank
@mikary mikary merged commit 5ba3e78 into SignalR:dev Oct 10, 2014
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.

None yet

5 participants