-
Notifications
You must be signed in to change notification settings - Fork 24
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
Issue/270 Documented the /randomizer
command
#293
Issue/270 Documented the /randomizer
command
#293
Conversation
docs/randomizer.md
Outdated
@@ -0,0 +1,61 @@ | |||
# /randomizer | |||
|
|||
Provide FEN data based on requested parameters to start a random position. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please note: As per my understanding I found, this command originally only provides the FEN data for position to start a new game contrary to the Controller description: "Starts a random position.", as actually, we require to make an additional call to start the game: ws.send('/start classical stockfish <options>')
So I wrote the document description accordingly, kindly provide me suggestions for improvement. And if so, do we need any update for the RandomizerCommand
description @programarivm
Hi @KartikWatts, Thanks for sending this PR! The description is the same one used in the src/Command folder.
Thus, the commands are also described when starting the WebSocket server:
The JavaScript code snippet to be used as an example can be written as per the docs at Getting Started.
Figure 1. Click on Training > Checkmate and select King and bishop and knight vs. king Data:
The thing is, in order to interact with the web app, a ChesslaBlab platform needs to be up and running locally on your computer, which might not be obvious without previous practical experiments with Public Key Infrastructure (PKI).
See: I hope this helps. Keep it up, and happy learning! |
docs/randomizer.md
Outdated
@@ -0,0 +1,61 @@ | |||
# /randomizer | |||
|
|||
Provide FEN data based on requested parameters to start a random position. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The description is the same one used in the src/Command folder: "Starts a random position."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it, so for the sake of simplicity, I'll keep the description as: "Starts a random position.", kindly let me know if there are any other content updates required.
I have documented the API after going through the usage and code implementation myself, so I believe I have documented the command without missing any details.
However, it would be great to receive your thoughts. Regards.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @KartikWatts for the help on documentation!
I checked out the PR and made some changes before merging.
See:
Keep it up,
Closes #270
This PR provides documentation for the
/randomizer
command