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

Docs page reviewed and updated #187

Merged
merged 2 commits into from
Oct 9, 2023
Merged

Conversation

GitGab19
Copy link
Collaborator

@GitGab19 GitGab19 commented Oct 6, 2023

I updated the docs page to add more details about JDP, especially Job Declaration Server and Client.
Let me know if it's good enough to clarify those points.
Closes #185

@netlify
Copy link

netlify bot commented Oct 6, 2023

Deploy Preview for startling-capybara-2231ea ready!

Name Link
🔨 Latest commit 9654926
🔍 Latest deploy log https://app.netlify.com/sites/startling-capybara-2231ea/deploys/6520424a388fde000872e2e9
😎 Deploy Preview https://deploy-preview-187--startling-capybara-2231ea.netlify.app
📱 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 configuration.


The Sv2 Mining Proxy acts as an intermediary between the mining devices and the Sv2 Pool. It receives mining requests from multiple devices, aggregates them, and forwards them to the Sv2 Pool. It can open group/extended channels with upstream (the Sv2 Pool) and standard channels with downstream (Sv2 Mining Devices).

#### Translator Proxy
Copy link
Collaborator

Choose a reason for hiding this comment

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

Translation Proxy? Not sure we used Translator proxy terminology anywhere?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Translation Proxy? Not sure we used Translator proxy terminology anywhere?

You're right, thanks!

pavlenex added a commit to pavlenex/stratumprotocol.org that referenced this pull request Oct 6, 2023
Copy link
Collaborator

@pavlenex pavlenex left a comment

Choose a reason for hiding this comment

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

This looks good to me now, there are probably some more optimizations we can do but this now ensures the copy is technically correct and goes along well with our next update. Good work @GitGab19

@pavlenex pavlenex merged commit bb822b5 into stratum-mining:main Oct 9, 2023
4 of 5 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.

Review contents on Documentation page (docs/index.md)
2 participants