Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@catalinpit
Copy link
Member

@catalinpit catalinpit commented Nov 20, 2025

No description provided.

@vercel
Copy link

vercel bot commented Nov 20, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

5 Skipped Deployments
Project Deployment Review Updated (UTC)
prd-app Ignored Ignored Dec 23, 2025 2:16pm
prd-docs Ignored Ignored Dec 23, 2025 2:16pm
prd-openpage-api Ignored Ignored Dec 23, 2025 2:16pm
stg-app Ignored Ignored Dec 23, 2025 2:16pm
stg-docs Ignored Ignored Preview Dec 23, 2025 2:16pm

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 20, 2025

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • coderabbit

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/get-documents-by-ids-endpoint

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

envelopeIds: z.array(z.string()).min(1),
});

export const ZGetEnvelopesByIdsResponseSchema = z.array(ZEnvelopeSchema);
Copy link
Member Author

Choose a reason for hiding this comment

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

used the same schema as in getEnvelopeRoute


const envelopeOrInput = envelopeWhereInput.OR!;

const envelopes = await prisma.envelope.findMany({
Copy link
Member Author

Choose a reason for hiding this comment

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

returns the same amount of information as the endpoint for retrieving one envelope

Implement getMultipleEnvelopeWhereInput for fetching multiple envelopes
with proper access control, supporting envelopeId, documentId, and
templateId types via discriminated union. Add 20 ID limit per request
and comprehensive E2E tests covering unauthorized access and teamId
manipulation attacks.
@Mythie Mythie merged commit 90fdba8 into main Dec 24, 2025
15 of 16 checks passed
@Mythie Mythie deleted the feat/get-documents-by-ids-endpoint branch December 24, 2025 00:02
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.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.