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

Update functional_requirements.md #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KewbitXMR
Copy link

Just a couple things to be clearer on

@KewbitXMR KewbitXMR requested a review from a team as a code owner August 29, 2024 01:28
Copy link
Contributor

@woodser woodser left a comment

Choose a reason for hiding this comment

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

Provided clarification as comments. If anything, this PR should update the text to clarify we're referring to Monero nodes, so the requests for clarification aren't added to the text.

@@ -4,7 +4,7 @@ This document maintains functional requirements for the Haveno user interface.

### Startup
- On first startup, the user must configure a password.
- On first startup, the user can use a local node or configure a remote node (url, username, password).
- On first startup, the user can use a local node or configure a remote node (url, username, password). (Need specificity, what kind of node, haveno daemon, monero node?)
Copy link
Contributor

Choose a reason for hiding this comment

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

This was referring to a Monero node.

But actually maybe it's better to simply default to the provided set, and allow them to customize the node afterwards.

Copy link
Author

Choose a reason for hiding this comment

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

We should keep it simple perhaps, have some trusted community nodes, have it randomly select among them, and encourage the user after onboarding to either host their own with instructions or give them resources to find one. We could also create a pool of all nodes by scanning popular sites like Monero.fail etc to build a bootstrapped listed which is load balanced over. It could be implemented as an optional part of the stack on the installation process to the desktop, just to make it super easy and have it self-manage, self-heal, auto restart etc.

@@ -50,13 +50,13 @@ This document maintains functional requirements for the Haveno user interface.
### Settings
- The user can export a backup of their Haveno instance to a zip file.
- The user can view the mnemonic phrase of their Haveno wallet for manual backup (excludes funds locked in multisig).
- The user can configure a local or remote node.
- The user can configure a local or remote node. (Needs specificity)
Copy link
Contributor

Choose a reason for hiding this comment

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

Again referring to a Monero node.

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.

2 participants