fossology / FOSSologyUI

Repository to hold the new UI framework for FOSSology built with React
https://fossology.github.io/FOSSologyUI/
GNU General Public License v2.0
49 stars 88 forks source link

feat(UI): added manage group-users page #248

Open dushimsam opened 2 years ago

dushimsam commented 2 years ago

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

Shruti3004 commented 2 years ago

Changes LGTM @dushimsam. Needs testing

dushimsam commented 2 years ago

Alright , have added tests now.

shaheemazmalmmd commented 2 years ago

hello @dushimsam

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

Warning: Failed prop type: Invalid propcountof typestringsupplied toForwardRef(Pagination), expectednumber. 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 commented 2 years ago

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

dushimsam commented 2 years ago

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.

github-actions[bot] commented 2 years ago

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

github-actions[bot] commented 2 years ago

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