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

CDE-61 Filter input text in the mapping step to be connected to the data #27

Merged
merged 5 commits into from
Mar 4, 2024

Conversation

Aiga115
Copy link
Contributor

@Aiga115 Aiga115 commented Feb 29, 2024

Issue #CDE-61
Problem: Filter input text in the mapping step to be connected to the data
Solution:
Created a seperate hook that debounces the search input, then when debounce value changes we handle filtering

CDE.Demo.App.-.Google.Chrome.2024-02-29.15-15-45.mp4


const handleFiltersClose = () => {
setAnchorEl(null);
onChange()
// onChange(searchString)
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Aiga115 are you sure you want to comment this out? If so then please remove it from the code. The history is in Git

@Aiga115 Aiga115 requested a review from zsinnema March 4, 2024 14:46
@@ -0,0 +1,15 @@
import { useEffect, useState } from 'react';

export const useDebounce = <T>(value: T, delay = 500) => {
Copy link
Member

Choose a reason for hiding this comment

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

lib/components/steps/mapping/MappingTab.tsx Show resolved Hide resolved
@zsinnema zsinnema merged commit 18c8091 into develop Mar 4, 2024
1 check passed
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.

3 participants