storj
storj copied to clipboard
web/multinode: Show appropriate error while adding new node
While adding new nodes, if the nodeID, Public IP Address, or the API Key is incorrect no error was being shown on the webpage. This commit resolves this issue by showing an error notification on the top right corner of the screen with appropriate error text.
Fixes #4199
Please describe the performance impact:
Code Review Checklist (to be filled out by reviewer)
- [ ] NEW: Are there any Satellite database migrations? Are they forwards and backwards compatible?
- [ ] Does the PR describe what changes are being made?
- [ ] Does the PR describe why the changes are being made?
- [ ] Does the code follow our style guide?
- [ ] Does the code follow our testing guide?
- [ ] Is the PR appropriately sized? (If it could be broken into smaller PRs it should be)
- [ ] Does the new code have enough tests? (every PR should have tests or justification otherwise. Bug-fix PRs especially)
- [ ] Does the new code have enough documentation that answers "how do I use it?" and "what does it do?"? (both source documentation and higher level, diagrams?)
- [ ] Does any documentation need updating?
- [ ] Do the database access patterns make sense?
Hey! This is my first PR to this project so let me know if anything is missing! Thank you.
Hey @Pranav2612000, thank you for your pull request :)
Before I review this change, could you get the build to pass? It looks like there are currently some linting errors: https://build.dev.storj.io/blue/organizations/jenkins/storj/detail/PR-4293/1/pipeline/ By the way, you can see the outcome of future builds by clicking "Details" next to "continuous-integration/jenkins/pr-head — This commit cannot be built" at the bottom of your PR
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @Pranav2612000 on file. Once you have signed the CLA, please let us know, so we can manually review and add you to the approved contributors list.
@Pranav2612000 , one more thing: could you upgrade your npm
version to at least version 7? For reference, my npm/node versions are 7.5.1/15.8.0 (you shouldn't need those exact versions). But we think part of the reason why so many lines were removed from your package-lock.json is that you have an older version of npm.
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @Pranav2612000 on file. Once you have signed the CLA, please let us know, so we can manually review and add you to the approved contributors list.
Hey @mobyvb ! I've updated nvm and node to 7.5.1/15.8.0 as you asked.
@cla-bot check
The cla-bot has been summoned, and re-checked this pull request!
This pull request has been mentioned on Storj Community Forum (official). There might be relevant details there:
https://forum.storj.io/t/multinode-seems-to-stop-where-it-should-continue/21697/14