-
Notifications
You must be signed in to change notification settings - Fork 283
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
refactor(cacti-cmd-gui-app): rename to cacti-ledger-browser #2572
Comments
Fixes hyperledger-cacti#2572 Signed-off-by: Peter Somogyvari <[email protected]>
Well, the GUI was meant to be common and extensible, I have a branch somewhere where each app is even separated from each other but I don't have time to push this forward. I see the point of making room for more GUI apps, but I don't like the name |
I see the points made by both of @petermetz and @outSH |
how about |
Fixes hyperledger-cacti#2572 Signed-off-by: Peter Somogyvari <[email protected]>
Fixes hyperledger-cacti#2572 Signed-off-by: Peter Somogyvari <[email protected]>
Fixes #2572 Signed-off-by: Peter Somogyvari <[email protected]>
Fixes hyperledger-cacti#2572 Signed-off-by: Peter Somogyvari <[email protected]>
Rename the package to something more specific so that it doesn't squat on the entire
namespace of all GUI applications we ever wrote/will write.
This would be a breaking change but luckily we haven't released a new version of
it to npm yet so renaming it now is easy.
The text was updated successfully, but these errors were encountered: