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

feat: get org ooo entries and filters/sort #18645

Merged
merged 3 commits into from
Jan 14, 2025
Merged

Conversation

ThyMinimalDev
Copy link
Contributor

@ThyMinimalDev ThyMinimalDev commented Jan 14, 2025

What does this PR do?

Added endpoint to get all ooo entries of users within your org, with sort and filter by email
same sorting for /users/:userId/ooo endpoint

also increased maximum size of page for get all org user endpoint to accommodate orgs with thousands of users

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

yarn test:e2e organizations-users-ooo.e2e-spec.ts

@keithwillcode keithwillcode added core area: core, team members only platform Anything related to our platform plan labels Jan 14, 2025
@graphite-app graphite-app bot requested a review from a team January 14, 2025 08:54
@dosubot dosubot bot added api area: API, enterprise API, access token, OAuth organizations area: organizations, orgs ✨ feature New feature or request labels Jan 14, 2025
Copy link

graphite-app bot commented Jan 14, 2025

Graphite Automations

"Add platform team as reviewer" took an action on this PR • (01/14/25)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add ready-for-e2e label" took an action on this PR • (01/14/25)

1 label was added to this PR based on Keith Williams's automation.

Copy link

vercel bot commented Jan 14, 2025

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

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Jan 14, 2025 3:10pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Jan 14, 2025 3:10pm

CarinaWolli
CarinaWolli previously approved these changes Jan 14, 2025
Copy link
Member

@CarinaWolli CarinaWolli left a comment

Choose a reason for hiding this comment

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

Code looks good 🙏

Copy link
Contributor

E2E results are ready!

@CarinaWolli CarinaWolli merged commit 2aa3d1d into main Jan 14, 2025
38 checks passed
@CarinaWolli CarinaWolli deleted the feat-get-org-ooo-endpoint branch January 14, 2025 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api area: API, enterprise API, access token, OAuth core area: core, team members only ✨ feature New feature or request organizations area: organizations, orgs platform Anything related to our platform plan ready-for-e2e
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants