backend.ai-webui icon indicating copy to clipboard operation
backend.ai-webui copied to clipboard

feat: refactor UserList at `credential` page using react

Open gee05053 opened this issue 1 year ago • 3 comments

This PR resolves https://github.com/lablup/backend.ai-webui/issues/2216

I refactor UserList to react and UserList get data using user_nodes GraphQL query that is applied Relay.

Add features

  • Use GQL Relay user_nodes query
  • Filter by email, name, main_access_key
  • Pagination (limit 10, offset 0)
  • Refactoring UserGenerationModal and UserSignoutModal

If you want to test, you must use main branch manager and modify backend.ai-client-esm.ts file's 678line 24.03.0 to 24.03.0.*.

Before 스크린샷 2024-02-22 오후 6 54 57

After(manager ver. 24.03.0.*) 스크린샷 2024-02-22 오후 6 50 45

Checklist: (if applicable)

  • [ ] Mention to the original issue
  • [ ] Documentation
  • [ ] Minium required manager version
  • [ ] Specific setting for review (eg., KB link, endpoint or how to setup)
  • [ ] Minimum requirements to check during review
  • [ ] Test case(s) to demonstrate the difference of before/after

gee05053 avatar Feb 16 '24 12:02 gee05053

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-2219.d3g9cs6u59b8lw.amplifyapp.com

Coverage report for ./react

St.:grey_question:
Category Percentage Covered / Total
🔴 Statements
2.69% (-0.05% 🔻)
141/5246
🔴 Branches
2.9% (-0.04% 🔻)
101/3479
🔴 Functions
1.55% (-0.04% 🔻)
27/1745
🔴 Lines
2.56% (-0.05% 🔻)
132/5147
Show new covered files 🐣
St.:grey_question:
File Statements Branches Functions Lines
🔴
... / UserSignoutModal.tsx
0% 0% 0% 0%
🔴
... / UserGenerationModal.tsx
0% 0% 0% 0%
🔴
... / UserList.tsx
0% 0% 0% 0%

Test suite run success

51 tests passing in 5 suites.

Report generated by 🧪jest coverage report action from 3515ac5d54fe2aefe05fb76f1fdab51e13a7c7cc

github-actions[bot] avatar Feb 19 '24 23:02 github-actions[bot]

From now I'll be in charge of this PR.

lizable avatar Apr 05 '24 04:04 lizable