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
feat: add custom baseport for nodes #878
base: master
Are you sure you want to change the base?
Conversation
This commit implement the feature that let users set custom baseport for their nodes to avoid having port conflicts with other applications running on the same machine.
Hey @Abdulkbk Thanks for opening the PR. It looks like you're off to a great start. My initial feedback just from looking at your screenshots is:
|
Thanks for the feedback @jamaljsr. I'll work on the suggestions in next iteration. |
In this commit, we allow the user to set grpc base ports for nodes that support it in addition to setting rest base ports.
This is how it looks now @jamaljsr |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The UI is looking great now. I left some feedback on the code. Once those changes are made, we just need to have the unit tests updated to prevent the failures and get the coverage back up. Then this should be all set to merge.
In this commit, we refactor/move some types from shared folder to the main types. We also remove store import from utils file and modified functions that need some values from store to accept them as parameters
@jamaljsr I added a new commit addressing the feedbacks you left. |
This commit implements the feature that lets users set custom base ports for their nodes to avoid having port conflicts with other applications running on the same machine.
Closes #533
Description
I added a settings page with input fields to set the base ports and then save. The custom base ports persist in the settings.json file and are loaded whenever polar starts. If a node, for example,
LND
does not have a custom port set, the defaults from constant.ts are usedSteps to Test
Network Setting
tab and click on it.Screenshots
Node Alice
Node Bob