starred/documenso

Fork 0

mirror of https://github.com/documenso/documenso.git synced 2026-04-25 05:46:01 +03:00

[PR #2261] [MERGED] refactor: extract image-helpers #2163

New issue

Closed

opened 2026-02-26 20:32:42 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 20:32:42 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/2261
Author: @tedliang
Created: 11/29/2025
Status: ✅ Merged
Merged: 12/8/2025
Merged by: @Mythie

Base: main ← Head: extract-image-helpers

📝 Commits (3)

30e7002 refactor: extract image-helpers
07349b8 Merge branch 'main' into extract-image-helpers
a72460a fix: use a folder for grouping, add gemini images

📊 Changes

12 files changed (+69 additions, -56 deletions)

View changed files

📝 apps/remix/app/routes/_share+/share.$slug.opengraph.tsx (+2 -3)
📝 apps/remix/app/routes/api+/branding.logo.organisation.$orgId.ts (+5 -10)
📝 apps/remix/app/routes/api+/branding.logo.team.$teamId.ts (+5 -10)
📝 apps/remix/package.json (+0 -1)
📝 package-lock.json (+0 -1)
📝 packages/lib/server-only/ai/envelope/detect-fields/index.ts (+2 -17)
📝 packages/lib/server-only/profile/get-avatar-image.ts (+3 -8)
📝 packages/lib/server-only/profile/set-avatar-image.ts (+2 -6)
➕ packages/lib/utils/images/avatar.ts (+16 -0)
➕ packages/lib/utils/images/logo.ts (+10 -0)
➕ packages/lib/utils/images/resize-image-to-gemini-image.ts (+19 -0)
➕ packages/lib/utils/images/svg-to-png.ts (+5 -0)

📄 Description

Description

Move image related operations to image-helpers. Simplify dependency management.

N/A

Changes Made

Consolidate image related operations to image-helpers.ts

Testing Performed

Tested on browsers Chrome.

Checklist

I have tested these changes locally and they work as expected.
I have added/updated tests that prove the effectiveness of these changes.
I have updated the documentation to reflect these changes, if applicable.
I have followed the project's coding style guidelines.
I have addressed the code review feedback from the previous submission, if applicable.

Additional Notes

This change will make it easier to switch to different image processing library.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/documenso/documenso/pull/2261 **Author:** [@tedliang](https://github.com/tedliang) **Created:** 11/29/2025 **Status:** ✅ Merged **Merged:** 12/8/2025 **Merged by:** [@Mythie](https://github.com/Mythie) **Base:** `main` ← **Head:** `extract-image-helpers` --- ### 📝 Commits (3) - [`30e7002`](https://github.com/documenso/documenso/commit/30e70020aa9dca932b41ca9a95ae5ebf2c74e625) refactor: extract image-helpers - [`07349b8`](https://github.com/documenso/documenso/commit/07349b85baf15d06c49d8752de6470dad0389d90) Merge branch 'main' into extract-image-helpers - [`a72460a`](https://github.com/documenso/documenso/commit/a72460a02d812e437d0fad1a09ddd8177788e74d) fix: use a folder for grouping, add gemini images ### 📊 Changes **12 files changed** (+69 additions, -56 deletions) <details> <summary>View changed files</summary> 📝 `apps/remix/app/routes/_share+/share.$slug.opengraph.tsx` (+2 -3) 📝 `apps/remix/app/routes/api+/branding.logo.organisation.$orgId.ts` (+5 -10) 📝 `apps/remix/app/routes/api+/branding.logo.team.$teamId.ts` (+5 -10) 📝 `apps/remix/package.json` (+0 -1) 📝 `package-lock.json` (+0 -1) 📝 `packages/lib/server-only/ai/envelope/detect-fields/index.ts` (+2 -17) 📝 `packages/lib/server-only/profile/get-avatar-image.ts` (+3 -8) 📝 `packages/lib/server-only/profile/set-avatar-image.ts` (+2 -6) ➕ `packages/lib/utils/images/avatar.ts` (+16 -0) ➕ `packages/lib/utils/images/logo.ts` (+10 -0) ➕ `packages/lib/utils/images/resize-image-to-gemini-image.ts` (+19 -0) ➕ `packages/lib/utils/images/svg-to-png.ts` (+5 -0) </details> ### 📄 Description ## Description Move image related operations to image-helpers. Simplify dependency management. ## Related Issue N/A ## Changes Made Consolidate image related operations to image-helpers.ts ## Testing Performed - Tested on browsers Chrome. ## Checklist   - [X] I have tested these changes locally and they work as expected. - [X] I have added/updated tests that prove the effectiveness of these changes. - [X] I have updated the documentation to reflect these changes, if applicable. - [X] I have followed the project's coding style guidelines. - [X] I have addressed the code review feedback from the previous submission, if applicable. ## Additional Notes This change will make it easier to switch to different image processing library. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-26 20:32:42 +03:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/documenso#2163

No description provided.

Rows
Columns