Skip to content

feat: make company employees searchable 🔎#835

Merged
ramiAbdou merged 8 commits intomainfrom
rami/fix
Jun 26, 2025
Merged

feat: make company employees searchable 🔎#835
ramiAbdou merged 8 commits intomainfrom
rami/fix

Conversation

@ramiAbdou
Copy link
Member

@ramiAbdou ramiAbdou commented Jun 26, 2025

Description ✏️

This PR:

  • Updates the listCompanyEmployees to accept a search parameter. It also does some extra calculation in the query which sums up the total time someone has spent at the company, then sorts in descending order based off of that.
  • Updates the UI of the company employees page:
    • Shows one big list of employees instead of separating current/past.
    • Shows a Current | Past pill for the employee.
    • Shows the time spent at the company.
  • Fixes other visual bugs.

Type of Change 🐞

  • Feature - A non-breaking change which adds functionality.
  • Fix - A non-breaking change which fixes an issue.
  • Refactor - A change that neither fixes a bug nor adds a feature.
  • Documentation - A change only to in-code or markdown documentation.
  • Tests - A change that adds missing unit/integration tests.
  • Chore - A change that is likely none of the above.

Checklist ✅

  • I have done a self-review of my code.
  • I have manually tested my code (if applicable).
  • I have added/updated any relevant documentation (if applicable).

@ramiAbdou ramiAbdou self-assigned this Jun 26, 2025
@ramiAbdou ramiAbdou marked this pull request as ready for review June 26, 2025 01:00
@ramiAbdou ramiAbdou merged commit b992a65 into main Jun 26, 2025
1 check passed
@ramiAbdou ramiAbdou deleted the rami/fix branch June 26, 2025 01:03
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.

1 participant