Skip to content

Subharthi#1027

Closed
splusb wants to merge 3 commits intoswapnilsparsh:masterfrom
splusb:master
Closed

Subharthi#1027
splusb wants to merge 3 commits intoswapnilsparsh:masterfrom
splusb:master

Conversation

@splusb
Copy link
Copy Markdown

@splusb splusb commented May 16, 2023

I have modified the dice rolling problem which shows two players and also declares who wins after one shuffle.

@netlify
Copy link
Copy Markdown

netlify bot commented May 16, 2023

Deploy Preview for 30daysofjs ready!

Name Link
🔨 Latest commit 7c1d4ba
🔍 Latest deploy log https://app.netlify.com/sites/30daysofjs/deploys/6463cbdb253bc700088ebfc6
😎 Deploy Preview [https://deploy-preview-1027--30daysofjs.netlify.app/09 - dice roll](https://deploy-preview-1027--30daysofjs.netlify.app/09 - dice roll)
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@swapnilsparsh
Copy link
Copy Markdown
Owner

@splusb Great idea, create a seperate folder for this project and name it as it is a new project whereas current dice roller is only for 1 dice and 1 player

@splusb
Copy link
Copy Markdown
Author

splusb commented May 16, 2023

As per the suggestion, a new file is created for the game with two players, and the previous game with one player is remaining as it was.

@swapnilsparsh
Copy link
Copy Markdown
Owner

@splusb Please create a sepearte folder naming a different project and there add your file

@splusb
Copy link
Copy Markdown
Author

splusb commented May 24, 2023 via email

@swapnilsparsh
Copy link
Copy Markdown
Owner

I have modified the dice rolling problem which shows two players and also declares who wins after one shuffle.

You don't have to make changes in the current Dice Roll folder , create a new folder there you have to upload the files of your project.

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