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: Add Empty State Display for Tables #2841

Merged

Conversation

tmahabir
Copy link
Contributor

@tmahabir tmahabir commented Dec 5, 2023

Resolves #1991

Added empty state display for when there are no records for Standard objects and Custom objects.
Here's a visual of the change:

twenty-empty-state-display.mov

Copy link

github-actions bot commented Dec 5, 2023

CLA

Hello there and welcome to our project!
By submitting your Pull Request, you acknowledge that you agree with the terms of our Contributor License Agreement.
Although we don't have a dedicated legal counsel, having this kind of agreement can protect us from potential legal issues or patent trolls.
Thank you for your understanding.

Generated by 🚫 dangerJS against 2ca0358

@thaisguigon thaisguigon force-pushed the feature-empty-state-display-for-tables branch from 7dfe081 to 72a4a8b Compare December 7, 2023 15:35
Copy link
Contributor

@thaisguigon thaisguigon left a comment

Choose a reason for hiding this comment

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

Hi @tmahabir, thank you for your contribution!

I rebased your PR as some important fixes were merged recently. Left a few minor comments, otherwise LGTM!

@tmahabir
Copy link
Contributor Author

tmahabir commented Dec 7, 2023

Hi @thaisguigon ! Thank you for the comments, I've addressed them in my most recent commits!

Copy link
Contributor

@thaisguigon thaisguigon left a comment

Choose a reason for hiding this comment

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

Approved, thank you!

@charlesBochet
Copy link
Member

Merging it!

@charlesBochet charlesBochet merged commit 921366f into twentyhq:main Dec 8, 2023
7 of 11 checks passed
@charlesBochet charlesBochet mentioned this pull request Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Empty state V1 for tables
3 participants