edina / nbexchange

External exchange for nbgrader
Other
6 stars 2 forks source link

Correctly assign the users org id #42

Closed agxs closed 3 years ago

agxs commented 3 years ago

Don't need to split on the username, which didn't work anymore anyway. The result data has the org id in it already.

codecov[bot] commented 3 years ago

Codecov Report

Merging #42 (6955594) into master (d15bdd9) will increase coverage by 0.14%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
+ Coverage   76.74%   76.89%   +0.14%     
==========================================
  Files          43       43              
  Lines        2086     2082       -4     
==========================================
  Hits         1601     1601              
+ Misses        485      481       -4     
Impacted Files Coverage Δ
nbexchange/handlers/auth/naas_user_handler.py 75.00% <ø> (+12.50%) :arrow_up:

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 d15bdd9...6955594. Read the comment docs.