search

OpenNetworkingFoundation / cla-manager

Automated CLA management and verification for ONF projects on Github and Gerrit
https://cla.opennetworking.org
3 stars 6 forks source link

fix linked accounts in admin table #272

Closed nitinsubz closed 3 years ago

nitinsubz commented 3 years ago

Fixes query to show the linked accounts of all users under the admin 'linked accounts' tab.

Fixes #185

teone commented 3 years ago

@nitinsubz the fix looks good, but the test needs to be updated as well

codecov[bot] commented 3 years ago

Codecov Report

Merging #272 (b7adf2b) into master (48442b2) will increase coverage by 0.92%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #272      +/-   ##
==========================================
+ Coverage   88.84%   89.77%   +0.92%     
==========================================
  Files           8        9       +1     
  Lines         278      313      +35     
  Branches       21       22       +1     
==========================================
+ Hits          247      281      +34     
- Misses         31       32       +1
Impacted Files Coverage Δ
common/model/appUser.js 96.29% <100.00%> (ø)
common/test_helpers/firestore.mock.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 48442b2...b7adf2b. Read the comment docs.