-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Conversation
@alexvandesande, thanks for your PR! By analyzing the history of the files in this pull request, we identified @frozeman, @hiddentao and @evertonfraga to be potential reviewers. |
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.
👍
Dont merge yet, as discussed with @alexvandesande he wants to make it more modular and display the name of the network in the |
@frozeman I updated the code. Couldn't think of a way to include |
… into add-ropstein-support
@alexvandesande where are you setting the name of the network to the title tag? |
b116d20
to
797e4a2
Compare
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.
Works as expected
This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread. |
This adds the automatic detection of the Ropstein as “testnet” and not “privatenet”