search

sillsdev / languageforge-lexbox

Lexbox, SIL linguistic data hub
MIT License
7 stars 2 forks source link

Make ProjectMembersVisibilityMiddleware more robust #1199

Closed myieye closed 2 weeks ago

myieye commented 3 weeks ago

TL;DR

Fixed project member visibility to include users even when their full user object hasn't been loaded.

What changed?

Modified the project members visibility middleware to check both the User.Id and UserId properties when filtering project users. This ensures that project members are correctly shown even when the User navigation property hasn't been fully loaded.

myieye commented 3 weeks ago

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @myieye and the rest of your teammates on Graphite Graphite

github-actions[bot] commented 3 weeks ago

C# Unit Tests

75 tests  ±0   75 :white_check_mark: ±0   5s :stopwatch: ±0s 13 suites ±0    0 :zzz: ±0   1 files   ±0    0 :x: ±0 

Results for commit dbbd5794. ± Comparison against base commit 6a5a04df.

:recycle: This comment has been updated with latest results.

myieye commented 2 weeks ago

Merge activity