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

Update .NET sample code #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mconnew
Copy link

@mconnew mconnew commented Mar 22, 2024

Updated to use minimal API style so no Main method, and no need to wrap an async Main method.
Updated to use new endpoint
Used new naming convention for folder
Added .gitignore suitable for VS use in dotnet folder
Added Dockerfile for Docker support to .NET sample
Added README.md file with Docker instructions
Updated website to drop the Core from .NET Core
Updated website sample to use newer C# syntax to match sample

@vicziani
Copy link
Owner

Hello Matt,

Thank you very much for your contribution! 🎉

I tried to build a docker image as I found in the README.md file, but I get an error:

image

Could you help me what is wrong with it?

You can increase the version number of the site to 2.1.0 and the date of the last update in index.md and index-hu.md. Please create a new item in the CHANGELOG file.

Don't forget to update the the last, Contributors section in index.md and index-hu.md. 😀

Thank you for your efforts!

@@ -518,18 +518,16 @@ soap.createClient(url, function(err, client) {
</div>
<div class="tab-pane fade" id="dotnet" role="tabpanel" aria-labelledby="dotnet-tab" markdown="1">

Az alábbi forráskód bemutatja, hogyan hívhat meg egy webszolgáltatást .NET használatával C# nyelv használatával.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The correct phrasing in Hungarian: Az alábbi forráskód bemutatja, hogyan hívhat meg egy webszolgáltatást .NET és C# nyelv használatával.

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