-
Notifications
You must be signed in to change notification settings - Fork 580
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
chore(site): add e2e tests for groups #12866
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.
Nicely done!
One note, the linked issues mentions testing the everyone group which I am not seeing in here, does that need to be added?
One other note actually, since this also includes a bug fix should this be prefixed with |
Good catch, I will add the everyone group test.
That is true, since most of this PR is chore related, I think we should keep it |
…-for-create-and-remove-groups
True, just worried it will be missed in the changelog for anyone tracking the bug. For users the fix is important, the chore may just be glossed over. |
Closes #12506