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

[OBSDEF-47922] Datagrid : Added changes to move user to first page if user selects any filters #284

Merged
merged 2 commits into from
May 22, 2024

Conversation

akate1
Copy link
Contributor

@akate1 akate1 commented May 22, 2024

OBSDEF-47922

Summary of the Change

  • When the user selects any of these filters or Show All toggle then it should redirect to the first page in datagrid
  • Updated story for datagrid full demo

Screenshot/GIF

Before

LogFilterIssueBefore

After

Storybook:

DatagridFilterIssue

Objectscale Portal UI:

LogFitlerDemo

Where to test ?

Storybook :
http://10.227.234.153:6006/?path=/story/datagrid--grid-with-pagination-and-pagesizes-dropdown

ObjectScale Portal UI:
http://10.227.234.153:3000/#

Scenarios verified/Documentation link

List all the scenarios verified for the changes or link to the manual test documentation for new features

Peer Tester Name

Add team member's name who will test your PR
@netraja-chavan

Copy link
Contributor

@netraja-chavan netraja-chavan left a comment

Choose a reason for hiding this comment

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

Tested
LGTM

Copy link
Contributor

@netraja-chavan netraja-chavan left a comment

Choose a reason for hiding this comment

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

LGTM

@akate1 akate1 merged commit 87c13bd into main May 22, 2024
1 check passed
@akate1 akate1 deleted the bugfix-OBSDEF-47922 branch May 22, 2024 11:46
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.

4 participants