search

stratum-mining / stratumprotocol.org

Website for the Stratum V2 bitcoin mining protocol
https://stratumprotocol.org
MIT License
13 stars 20 forks source link

replace JN graphics with JD #184

Closed pavlenex closed 1 year ago

pavlenex commented 1 year ago

This PR replaces the outdated JN graphics.

One thing remaining here is how to handle the config D. IMO Config D should be the one displayed prominently in the tabs, so config D should be in front of config A at least until we have fw. Thoughts?

netlify[bot] commented 1 year ago

Deploy Preview for startling-capybara-2231ea ready!

Name Link
Latest commit 46ef0940209e4acf82d8d6659c0a2e43db0d9662
Latest deploy log https://app.netlify.com/sites/startling-capybara-2231ea/deploys/651af4684066c900073d80e3
Deploy Preview https://deploy-preview-184--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.

GitGab19 commented 1 year ago

This PR replaces the outdated JN graphics.

One thing remaining here is how to handle the config D. IMO Config D should be the one displayed prominently in the tabs, so config D should be in front of config A at least until we have fw. Thoughts?

I agree with you. I think we could also simply renaming configs since they're just mentioned into the getting-started guide which will be updated in the next days. Also because it sounds strange to have a first look at the guide and start with Config C/D, instead of A/B

pavlenex commented 1 year ago

I will merge this one and do a follow up PR as I may want to modify text slightly.