Skip to content

Config Options for Wallet Optimization #1265

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

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

Conversation

corviato1
Copy link
Collaborator

This pull request is 2 parts.

1- Add simple info and a redirect to the readme for better visibility to Config-README.md.

2- Add Config-README.md. This file helps users leverage their node via personalization. File contains a variety of RPC and Network Settings.

The primary goal of Config-README.md is to provide RPC and Network Settings that reduce time needed to sync the wallet. These options allow high performance computers to sync the full chain in minutes rather than hours.

@corviato1
Copy link
Collaborator Author

corviato1 commented Apr 10, 2025

At @TronBlack and @HyperPeek you two are the only collaborators I know of off hand. Can you please upgrade my account to be able to view other collaborators for review requests?

Screenshot_20250409_173611_GitHub.jpg

@corviato1
Copy link
Collaborator Author

corviato1 commented Apr 30, 2025

Interesting, So there are two approvals, but @NiftyRaven and @rumplin approved the merge, but do not have write access to allow the merge. That's a bummer.

It would be nice if there was a list of users who have write access that I can ping.

At the very least, @NiftyRaven should have write access.
I know he would never push code to main net if there was even a chance of breaking the code, or adding an exploit.

There should be an audit for current reviewers with write access and a way to add new reviewers with write access who would only push simple fix codes that would not break the chain, or enable exploits.

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.

3 participants