Skip to content

Northwind Example Fixes/Updates #28

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

Merged
merged 1 commit into from
Aug 29, 2013
Merged

Northwind Example Fixes/Updates #28

merged 1 commit into from
Aug 29, 2013

Conversation

ChrisMcKee
Copy link
Contributor

  • Switched dependency to nuget from lib folder to simplify future updates.
  • Updated obsolete usages to use newer syntax.
  • Optimized images (lossless)
  • Removed contents of pacakages folder
  • Various minor tweaks

--Chris McKee

* Switched dependency to nuget from lib folder to simplify future updates.
* Updated obsolete usages to use newer syntax.
* Optimized images (lossless)
* Removed contents of pacakages folder
* Various minor tweaks

--Chris McKee
@ChrisMcKee
Copy link
Contributor Author

Totally cocked up trying to merge that last night; git-fucked myself.

It's in a branch in my fork if you want to run it; Before I move on to another one of the examples, do you want them updating to .net 4.5 while I'm in there? Anything else you'd advise.

Tar

mythz added a commit that referenced this pull request Aug 29, 2013
@mythz mythz merged commit 7c11122 into ServiceStack:master Aug 29, 2013
@mythz
Copy link
Member

mythz commented Aug 29, 2013

Awesome, thx Chris, looks like it turned out being a fair bit of work.
Was the return of [DataContract] attrs for SOAP support or something else?

@mythz
Copy link
Member

mythz commented Aug 29, 2013

Nah don't upgrade the projs, we need to save the current state for the v3 snapshot.
We can upgrade the projs once I get v4 out the door.

@ChrisMcKee
Copy link
Contributor Author

Slaaaaaaaaaaave to the SOAP
Formalised anal-retentiveness ;)

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