Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/Infinite-scrolling-feature-and-user-search-fix-on-user-management-page #860

Merged

Conversation

prashant67690
Copy link
Contributor

@prashant67690 prashant67690 commented Sep 28, 2024

Date: 28-09-2024

Developer Name: Prashant Kumar


Issue Ticket Number

closes : #833

Description

The User Management page currently features infinite scroll loading of users and the users in the list view are now rendered as the list rather than in the form of cards / tiles , due these changes the search functionality was having issue , that thing is fixed and running as expected down below.

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshots

Screenshot 1

Before:
prev
oldlist
371653915-742a5a61-42d0-48d9-a1a1-058c21169ad7

Screenshot 2

After:
loader
afterlist

Test Coverage

Screenshot 1

tests

Additional Notes

here we can see the search functionality behavior after the fixes

2024-09-28.03-02-01.mp4

users/script.js Outdated Show resolved Hide resolved
users/script.js Show resolved Hide resolved
users/script.js Outdated Show resolved Hide resolved
users/script.js Outdated Show resolved Hide resolved
users/script.js Outdated Show resolved Hide resolved
users/script.js Show resolved Hide resolved
@prashant67690
Copy link
Contributor Author

yash sir fixed the previous problems and also wrote the additional test as you requested please review

@prashant67690 prashant67690 deleted the fix/user-management-fix branch September 30, 2024 18:25
@prashant67690 prashant67690 restored the fix/user-management-fix branch September 30, 2024 18:27
@prashant67690 prashant67690 reopened this Sep 30, 2024
@prashant67690 prashant67690 changed the title Fix/user management fix Feature/Infinite-scrolling-feature-and-user-search-fix-on-user-management-page Sep 30, 2024
users/style.css Show resolved Hide resolved
Copy link

@Ayushsanjdev Ayushsanjdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vinit717 vinit717 mentioned this pull request Oct 2, 2024
10 tasks
@iamitprakash iamitprakash merged commit a246e51 into Real-Dev-Squad:develop Oct 2, 2024
4 checks passed
@yesyash yesyash mentioned this pull request Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enhancement: Implement Infinite Scroll Loading For Users and Make User List Render As List
5 participants