-
Notifications
You must be signed in to change notification settings - Fork 108
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
Bugs in Dashboard #2931
Bugs in Dashboard #2931
Conversation
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.
I have a handful of comments/questions for you.
BTW, has this code been run through Prettier, yet?
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.
Please let's not link PRs to Epics or Features in Jira! We should be linking PRs to stories (or bugs) where they're simple, or to sub-tasks where a story/bug requires multiple PRs.
This seems to be several bug/story issues under the PBENCH-86 epic.
It's not necessarily bad to link a PR to multiple simple issues, as long as they're closely related, but in general small focused PRs dealing with a single issue are easier to review.
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.
Looks good. (Thanks for straightening out constructToast()
for me! 😊)
I just have one (pointed) question.
PBENCH-86
User management in Dashboard
Bugs