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.

Sign up for GitHub

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

Jump to bottom

Clean up pool to solo guide #376

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Clean up pool to solo guide #376

wants to merge 4 commits into from

Conversation

nathanielcwm
Copy link
Contributor

@nathanielcwm nathanielcwm commented Jun 27, 2023 •
edited
Loading

Closes #357

Todo:

  • Provide instructions on fixing a split CPID
  • Proof read
  • Expand instructions on removing pool account manager and projects?

@nathanielcwm
Copy link
Contributor Author

nathanielcwm commented Jun 27, 2023 •
edited
Loading

I plan to squash commits before this is merged.
My local copy of jekyll isn't working properly so I'm testing it using Github pages.

@barton2526
Copy link
Member

Can you please name your commits?

@nathanielcwm
Copy link
Contributor Author

I plan to squash them later.

@makeasnek
Copy link
Contributor

Glad to see some more work is being done on the site :).

Re: account managers, I would suggest leaving these out of the solo guide. There should be one, simple default way to go through the guide with any other complexity "branched off" for users who need it. BAM is great, but it's one more complex thing for a user to setup and it has plenty of quirks which make it difficult to use, so I don't think it should be the default method.

We already have this noted on the first page of the solo crunching guide: Note: If you are an existing BOINC user and use an account manager such as BAM, you can add projects via your account manager so long as accounts at the projects all use the same e-mail address (BAM's default).

I do think some design changes in terms of layout and presentation could make a drastic difference to readability. This is beyond my HTML/CSS knowledge, but for example, why do we need the BOINC videos to be a different column here: https://gridcoin.us/guides/boinc-install.htm. Breaking up text visually could be a major help.

@nathanielcwm nathanielcwm marked this pull request as ready for review July 10, 2023 04:39
@nathanielcwm
Copy link
Contributor Author

I think design changes is a bit out of scope for this PR.
Honestly it might be good to just redirect people to the guides on .world if aesthetics is an issue.

@makeasnek
Copy link
Contributor

I also just submitted a handful of very small PRs for usability you may want to check out and/or incorporate into yours.

@jamescowens
Copy link
Member

Can we finish this please?

@nathanielcwm
Copy link
Contributor Author

Are there any changes required?

@nathanielcwm
Clean up pool to solo guide
f2c68ea 
Add split cpid instructions

Add notice about lack of rewards with pool account

Remove account manager info in line with solo guide

Expand split cpid warning
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Pool to solo guide
4 participants
@nathanielcwm @barton2526 @makeasnek @jamescowens