Display relevant OAuth information on accounts page #499
Conversation
![codeclimate[bot]](https://avatars.githubusercontent.com/in/9403?s=60&v=4){kind=small}
| </> | ||
| ), | ||
| }, | ||
| { |
There was a problem hiding this comment.
Similar blocks of code found in 2 locations. Consider refactoring.
| label: 'Profile', | ||
| body: ( | ||
| <> | ||
| Profile - potentially visible to other users. |
There was a problem hiding this comment.
please remove these two lines
| <br></br> | ||
| See more details on{' '} | ||
| <a href={auth.user?.profile.profile}>SSO OAuth provider.</a> | ||
| </> |
There was a problem hiding this comment.
Please add the groups text and code here. The exact text is available in issue #478
There was a problem hiding this comment.
Have added it in the conditional when the user is part of group/groups, this is the accounTabs constant content when he is not part of any of them
client/yarn.lock
Outdated
There was a problem hiding this comment.
There aren't any changes in the package.json dependencies. Is there any particular reason for updating this file?
anshhhhhhh
force-pushed
the
feature/distributed-demo
branch
from
e92186d
to
375d3a9
Compare
…rify the working of the same
| body: <>Account settings - private to a user.</>, | ||
| }, | ||
| ]; | ||
| function AccountTabs() { |
There was a problem hiding this comment.
Function AccountTabs has 38 lines of code (exceeds 25 allowed). Consider refactoring.
…rify the working of the same
Codecov Report
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## feature/distributed-demo #499 +/- ##
============================================================
+ Coverage 63.38% 65.67% +2.29%
============================================================
Files 31 31
Lines 396 405 +9
Branches 26 31 +5
============================================================
+ Hits 251 266 +15
+ Misses 130 124 -6
Partials 15 15
|
|
Code Climate has analyzed commit 5093940 and detected 0 issues on this pull request. View more on Code Climate. |
|
@anshhhhhhh you added the number of groups a user belongs to. Listing of these user groups is needed instead. Thanks. |
|
@anshhhhhhh |
No description provided.