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

feat(ws): add filter to workspaces table #157

Open
wants to merge 2 commits into
base: notebooks-v2
Choose a base branch
from

Conversation

ElayAharoni
Copy link

@ElayAharoni ElayAharoni commented Dec 11, 2024

Fixes: #143.

extracted filtering logic into different generic Filter component.

implemented logic for filtering with single filter and multiple filters, including adding and removing.

cleaned the workspaces file code a little bit with inserting looping over data.

added cypress tests for the filtering.
made minor change in the cypress.config file in order to be able to see changes in the ui immidiatly on the cypress gui and make testing more easy, so we dont need to do build on each change in order to see it on cypress.

Signed-off-by: Elay Aharoni (EXT-Nokia) <[email protected]>
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign ederign for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ElayAharoni ElayAharoni force-pushed the RHOAIENG-16233-Filter-workspaces branch 4 times, most recently from 0ad2045 to fd2eb6f Compare December 11, 2024 17:59
@thesuperzapper thesuperzapper changed the title Fix for RHOAIENG-16233: Notebooks-v2 Filter workspaces feat(ws): add filter to workspaces table Dec 11, 2024
@ElayAharoni ElayAharoni force-pushed the RHOAIENG-16233-Filter-workspaces branch from fd2eb6f to 47b4fac Compare December 12, 2024 08:35
@ElayAharoni ElayAharoni force-pushed the RHOAIENG-16233-Filter-workspaces branch 3 times, most recently from 5764a02 to cdd8587 Compare December 18, 2024 14:26
Signed-off-by: Elay Aharoni (EXT-Nokia) <[email protected]>
@ElayAharoni ElayAharoni force-pushed the RHOAIENG-16233-Filter-workspaces branch from cdd8587 to 7a34836 Compare December 19, 2024 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Needs Triage
Development

Successfully merging this pull request may close these issues.

1 participant