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

Update UI after removing project member #372

Merged

Conversation

myieye
Copy link
Contributor

@myieye myieye commented Nov 7, 2023

Resolves #368

Added [UseMutationConvention] for consistency.

Sena.3.-.Google.Chrome.2023-11-07.17-12-21.mp4

The inconsistency in the two notifications is calling my name.

Copy link

github-actions bot commented Nov 7, 2023

UI unit Tests

1 tests  ±0   1 ✔️ ±0   0s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ±0 

Results for commit b37f3e7. ± Comparison against base commit f72fc82.

Copy link

github-actions bot commented Nov 7, 2023

C# Unit Tests

34 tests   34 ✔️  11s ⏱️
  6 suites    0 💤
  1 files      0

Results for commit b37f3e7.

Copy link
Collaborator

@hahn-kev hahn-kev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice! Yeah useMutation didn't work with useProjection when this was first written so that's probably why this was wasn't done.

@myieye myieye merged commit 012dc0a into develop Nov 8, 2023
11 checks passed
@myieye myieye deleted the bug/368-removing-a-project-member-does-not-update-the-ui branch November 8, 2023 09:22
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.

removing a project member does not update the UI
2 participants