Skip to content
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

Show notification and filter users after creating a new user #820

Merged
merged 2 commits into from
May 28, 2024

Conversation

myieye
Copy link
Contributor

@myieye myieye commented May 24, 2024

The notification and text were a bit of an oversite and filtering the table so the admin can see the user they just created is a little bonus.

So, that's what it looks like after the admin creates the user:
image

And there's a login-specific "already exists" message:
image

@myieye myieye requested a review from rmunn May 24, 2024 09:59
Copy link

UI unit Tests

11 tests  ±0   11 ✅ ±0   0s ⏱️ ±0s
 3 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit b431b4f. ± Comparison against base commit 5c40148.

@myieye myieye merged commit b2af1d4 into develop May 28, 2024
9 checks passed
@myieye myieye deleted the feat/create-user-ux-touchups branch May 28, 2024 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants