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

C# Guess The Word Workshop conversion to .Net Fiddle #512

Merged

Conversation

veronicagg
Copy link
Contributor

Updating media and content files to refer to .Net Fiddle instead of Replit

… and content files to refer to .Net Fiddle instead of Replit
@ozhang22 ozhang22 changed the title C# Guess The Word Workship conversion to .Net Fiddle C# Guess The Word Workshop conversion to .Net Fiddle Sep 18, 2024

Click the button below to sign up for a Replit account - you will need an email account in order to sign up. If you already have a Replit account, you can login to your account using the below button as well.
Copy link
Member

Choose a reason for hiding this comment

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

Let's mark the sign up process as optional by surrounding with a {{ % notice tip % }} and {{ % /notice % }}

@@ -35,6 +35,6 @@ Try guessing the answers to the following expressions. Use `Console.WriteLine` t
- `(9 < 10) && (12 => 11)`
- `(15 - 2 == 11) || (4 % 3 != 2)`

<a class="my-2 mx-4 btn btn-info" href="https://replit.com/@nuevofoundation/NF-CSharp-blank" target="_blank">Launch Replit</a>
<a class="my-2 mx-4 btn btn-info" href="https://dotnetfiddle.net/ULv0JH" target="_blank">Launch Fiddle</a>
Copy link
Member

Choose a reason for hiding this comment

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

For all of these fiddles that are not found in the starting up page - these were actually previously embedded directly into the workshop before replit stopped supporting embeds - is this something that we can reenable?

Copy link
Member

Choose a reason for hiding this comment

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

This is weird... this file is duplicative of the _index file... Let's delete this.

@@ -7,8 +7,8 @@ weight: 4

## Instructions

Go to the code containing the [GuessTheWord](https://aka.ms/nuevoguesstheword) game. Press **run**. Notice that `Press 'run' on the top of the screen to play again.` is the only thing that is printed to the console. Let's add a welcome message so that the player knows they playing the Guess the Word game.
Go to the code containing the [GuessTheWord](https://dotnetfiddle.net/lMl7j4) game. Press **run**. Notice that `Press 'run' on the top of the screen to play again.` is the only thing that is printed to the console. Let's add a welcome message so that the player knows they playing the Guess the Word game.
Copy link
Member

Choose a reason for hiding this comment

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

Could you change the link to be of one that opens in a new page?

<a href="https://dotnetfiddle.net/lMl7j4" target="_blank">Guess The Word </a>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

clicking on the link does open it in a new window when I try it, so I think we could leave it as is.

@@ -16,27 +16,36 @@ The goal of today's activity is to learn C# skills and incorporate your new-foun

The goal of today's activity is to learn C# skills and incorporate your new-found knowledge to create a Guess The Word game! Let's get the game set up.

Today you will be using Replit. Before starting, you will need to sign up for a Replit account, if you do not yet have one.
Today you will be using .NET Fiddle.
Copy link
Member

Choose a reason for hiding this comment

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

Maybe add a small explanation of what .NET Fiddle is?

Deleted starting-up.md since it was the same content as index.md.
@ozhang22 ozhang22 merged commit cbb7d74 into NuevoFoundation:master Sep 20, 2024
1 of 2 checks passed
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