Skip to content

Update Filters to not allow filtering on all columns - #654

Merged
TorstenDittmann merged 8 commits into
feat-messaging-push-previewfrom
feat-filters
Jan 2, 2024
Merged

Update Filters to not allow filtering on all columns#654
TorstenDittmann merged 8 commits into
feat-messaging-push-previewfrom
feat-filters

Conversation

@stnguyen90

@stnguyen90stnguyen90 commented Dec 14, 2023

Copy link
Copy Markdown
Contributor

What does this PR do?

  • Create a component to handle 0 results from filtering
  • Update filterable columns on subscribers table

Test Plan

Manual

Related PRs and Issues

Parent:

Child:

Have you read the Contributing Guidelines on issues?

Yes

@vercel

vercelBot commented Dec 14, 2023

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

NameStatusPreviewCommentsUpdated (UTC)
console-cloud✅ Ready (Inspect)Visit Preview💬 Add feedbackJan 2, 2024 2:31pm
console-preview✅ Ready (Inspect)Visit Preview💬 Add feedbackJan 2, 2024 2:31pm
console-preview-cloud✅ Ready (Inspect)Visit Preview💬 Add feedbackJan 2, 2024 2:31pm

@stnguyen90
stnguyen90 changed the base branch from main to feat-messaging-push-previewDecember 14, 2023 21:06
@stnguyen90stnguyen90 mentioned this pull request Dec 14, 2023
@stnguyen90
stnguyen90force-pushed the feat-messaging-push-preview branch from 632fa84 to ed69b25CompareDecember 14, 2023 23:35
@stnguyen90
stnguyen90force-pushed the feat-messaging-push-preview branch from ed69b25 to 40a9ec6CompareDecember 15, 2023 20:04
@stnguyen90
stnguyen90force-pushed the feat-messaging-push-preview branch from 40a9ec6 to 5581914CompareDecember 19, 2023 22:48
@stnguyen90
stnguyen90force-pushed the feat-messaging-push-preview branch from 5581914 to 48babe9CompareDecember 20, 2023 01:59
The right side of the droplist should align with the right side of the button.
Fix messaging based on design review feedback
Sign up for freeto 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.

2 participants

@stnguyen90@TorstenDittmann