FOSSologyUI icon indicating copy to clipboard operation
FOSSologyUI copied to clipboard

feat(UI): added manage group-users page

Open dushimsam opened this issue 2 years ago • 7 comments

Description

Added Manage Group Users Page.

Changes

  1. Added a new file [src/pages/Admin/Group/Manage/index.jsx]
  2. Introduced a new route in src/Routes.jsx
  3. Introduced new API callers for the UI integration.

How to test

  1. Go to the project's main directory.
  2. run command npm start
  3. Visit the page at http://localhost:PORT/admin/group/manage
  4. You will see a list of group members for the default selected group.
  5. You can change the permission of any use in that group.

https://user-images.githubusercontent.com/66276301/183613559-6a682fe1-303c-46a1-83f2-d710092a3606.mp4

Related Issue

Closing #242

cc: @shaheemazmalmmd @GMishx

dushimsam avatar Aug 09 '22 09:08 dushimsam

Changes LGTM @dushimsam. Needs testing

Shruti3004 avatar Aug 15 '22 13:08 Shruti3004

Alright , have added tests now.

dushimsam avatar Aug 15 '22 17:08 dushimsam

hello @dushimsam

While testing this branch i have encountered the below error please check.

Warning: Failed prop type: Invalid prop countof typestringsupplied toForwardRef(Pagination), expected number. Pagination@http://localhost:3000/static/js/13.chunk.js:4327:23 WithStyles@http://localhost:3000/static/js/13.chunk.js:6745:25 div div td tr tbody table div div div Browse@http://localhost:3000/static/js/20.chunk.js:926:93 div Route@http://localhost:3000/static/js/vendors~main.chunk.js:35004:29 PrivateLayout@http://localhost:3000/static/js/61.chunk.js:60:7 C@http://localhost:3000/static/js/vendors~main.chunk.js:35261:31 Switch@http://localhost:3000/static/js/vendors~main.chunk.js:35206:29 Router@http://localhost:3000/static/js/vendors~main.chunk.js:34623:30 BrowserRouter@http://localhost:3000/static/js/vendors~main.chunk.js:34244:35 Routes Fe@http://localhost:3000/static/js/vendors~main.chunk.js:42702:67 App@http://localhost:3000/static/js/main.chunk.js:111:63 GlobalProvider@http://localhost:3000/static/js/main.chunk.js:1209:7 AppWrapper Suspense index.js:1 e index.js:1 React 5 WithStyles withStyles.js:79 React 9 unstable_runWithPriority scheduler.development.js:468 React 5 Browse index.jsx:99

shaheemazmalmmd avatar Aug 25 '22 10:08 shaheemazmalmmd

Tested Looks fine. but manage users should also have a functionality to add users to particular group. can you work on it @dushimsam

shaheemazmalmmd avatar Aug 25 '22 11:08 shaheemazmalmmd

Tested Looks fine. but manage users should also have a functionality to add users to particular group. can you work on it @dushimsam

sure sir, I am going to work on it as well.

dushimsam avatar Aug 25 '22 11:08 dushimsam

This pull request has conflicts, please rebase to resolve those before we can evaluate the pull request.

github-actions[bot] avatar Aug 25 '22 11:08 github-actions[bot]

This pull request has conflicts, please rebase to resolve those before we can evaluate the pull request.

github-actions[bot] avatar Sep 05 '22 19:09 github-actions[bot]