Skip to content

Conversation

@GabrielTozatti
Copy link
Collaborator

Description

Update the empty state to the new design, ensuring visual alignment with the updated layout and an improved user experience.

Summary

  • In the empty state, colors were updated to match the new Figma layout, a more appropriate icon was added, and spacing adjustments were made.
  • In notifications, only an icon adjustment was made.

Screenshots

Empty State:
Screenshot_1

Notifications:
Screenshot_2

  • Video example : Link

Checklist

  • I reviewed my PR code before submitting
  • I ensured that the implementation is working correctly and did not impact other parts of the app
  • I implemented error handling for all actions/requests and verified how they will be displayed in the UI (or there was no error handling needed).
  • I mentioned the PR link in the task

Copy link
Member

@guimroque guimroque left a comment

Choose a reason for hiding this comment

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

🚀 Production Promotion Checklist

This PR is being merged to main. Please verify the items below:

🔒 Security

  • No secrets/credentials exposed in code
  • No staging/development URLs hardcoded
  • No debug flags enabled

✅ Quality

  • Build passed on staging
  • Tests passed on staging
  • Code was reviewed on staging

🔄 Integrity

⚠️ Branch main has 4 commit(s) not in source. Consider syncing before merge.

⚠️ Issues Found

🔴 Critical (blocks merge)

  • Localhost URL: localhost:5173 em playwright.config.ts:23

🟡 Warnings (manual review)

  • Test domain: .test em .github/workflows/test-e2e.yml:23
  • Test domain: .test em .github/workflows/test-e2e.yml:24
  • Test domain: .test em package.json:16
  • Unresolved TODO/FIXME: TODO: em src/components/dialog/link.tsx:8

@pedroanastacio pedroanastacio changed the base branch from main to staging January 8, 2026 13:35
Copy link
Member

@guimroque guimroque left a comment

Choose a reason for hiding this comment

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

LGTM! ✅

All previous issues have been resolved. Code approved.

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