[PR #244] [MERGED] feat: add ability to upload a profile picture #896

New Issue

OVERLORD · 2025-10-07T00:24:15+03:00

OVERLORD commented

2025-10-07 00:24:15 +03:00

📋 Pull Request Information

Original PR: https://github.com/pocket-id/pocket-id/pull/244
Author: @stonith404
Created: 2/17/2025
Status: ✅ Merged
Merged: 2/19/2025
Merged by: @stonith404

Base: main ← Head: feat/profile-picture

📝 Commits (10+)

0d12169 add first version of profile picture
48da5eb move uuid validation to service
b7d5c97 fix existing tests
7d097d9 add ldap support
855303c remove save button
6d55d5d Revert "fix existing tests"
1ccb8d8 show upload icon instead of "update"
357387a Merge branch 'main' into feat/profile-picture
162245c validate user id in profile picture update
513ce04 improve profile picture attribute description

📊 Changes

38 files changed (+500 additions, -73 deletions)

View changed files

📝 backend/go.mod (+4 -2)
📝 backend/go.sum (+9 -2)
📝 backend/internal/common/errors.go (+8 -0)
📝 backend/internal/controller/user_controller.go (+73 -0)
📝 backend/internal/controller/well_known_controller.go (+1 -1)
📝 backend/internal/dto/app_config_dto.go (+1 -0)
📝 backend/internal/model/app_config.go (+1 -0)
📝 backend/internal/service/app_config_service.go (+4 -0)
📝 backend/internal/service/ldap_service.go (+43 -1)
📝 backend/internal/service/oidc_service.go (+1 -0)
📝 backend/internal/service/user_service.go (+69 -0)
➕ backend/internal/utils/image/profile_picture.go (+96 -0)
📝 backend/resources/files.go (+1 -1)
➕ backend/resources/fonts/PlayfairDisplay-Bold.ttf (+0 -0)
📝 frontend/src/lib/components/form/auto-complete-input.svelte (+0 -0)
📝 frontend/src/lib/components/form/checkbox-with-label.svelte (+3 -3)
📝 frontend/src/lib/components/form/custom-claims-input.svelte (+1 -1)
📝 frontend/src/lib/components/form/file-input.svelte (+1 -1)
📝 frontend/src/lib/components/form/form-input.svelte (+1 -1)
➕ frontend/src/lib/components/form/profile-picture-settings.svelte (+83 -0)

...and 18 more files

📄 Description

Fixes #58

_{🔄 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/pocket-id/pocket-id/pull/244 **Author:** [@stonith404](https://github.com/stonith404) **Created:** 2/17/2025 **Status:** ✅ Merged **Merged:** 2/19/2025 **Merged by:** [@stonith404](https://github.com/stonith404) **Base:** `main` ← **Head:** `feat/profile-picture` --- ### 📝 Commits (10+) - [`0d12169`](https://github.com/pocket-id/pocket-id/commit/0d1216912a75c083cdb391919054218c4490989c) add first version of profile picture - [`48da5eb`](https://github.com/pocket-id/pocket-id/commit/48da5eb8325100dbf3407a908497e35990ec159a) move uuid validation to service - [`b7d5c97`](https://github.com/pocket-id/pocket-id/commit/b7d5c97ce5620210e0b352655ec1d068c71a3221) fix existing tests - [`7d097d9`](https://github.com/pocket-id/pocket-id/commit/7d097d91d683839058335db5f99ac574624fb8cb) add ldap support - [`855303c`](https://github.com/pocket-id/pocket-id/commit/855303c4a68c296189bb8ed0dc0ddcf603cafe18) remove save button - [`6d55d5d`](https://github.com/pocket-id/pocket-id/commit/6d55d5dd7b94b08615255eb71f3ceab909e4da5a) Revert "fix existing tests" - [`1ccb8d8`](https://github.com/pocket-id/pocket-id/commit/1ccb8d83235f4d7237f74418b15a8ae43a8e4fb0) show upload icon instead of "update" - [`357387a`](https://github.com/pocket-id/pocket-id/commit/357387a3c4efe75770c7168a1a90d986a767e452) Merge branch 'main' into feat/profile-picture - [`162245c`](https://github.com/pocket-id/pocket-id/commit/162245ce93834b91d90c4a98d001b08d8e3e6eff) validate user id in profile picture update - [`513ce04`](https://github.com/pocket-id/pocket-id/commit/513ce0401079ee56f93b9c67a2441a620a145185) improve profile picture attribute description ### 📊 Changes **38 files changed** (+500 additions, -73 deletions) <details> <summary>View changed files</summary> 📝 `backend/go.mod` (+4 -2) 📝 `backend/go.sum` (+9 -2) 📝 `backend/internal/common/errors.go` (+8 -0) 📝 `backend/internal/controller/user_controller.go` (+73 -0) 📝 `backend/internal/controller/well_known_controller.go` (+1 -1) 📝 `backend/internal/dto/app_config_dto.go` (+1 -0) 📝 `backend/internal/model/app_config.go` (+1 -0) 📝 `backend/internal/service/app_config_service.go` (+4 -0) 📝 `backend/internal/service/ldap_service.go` (+43 -1) 📝 `backend/internal/service/oidc_service.go` (+1 -0) 📝 `backend/internal/service/user_service.go` (+69 -0) ➕ `backend/internal/utils/image/profile_picture.go` (+96 -0) 📝 `backend/resources/files.go` (+1 -1) ➕ `backend/resources/fonts/PlayfairDisplay-Bold.ttf` (+0 -0) 📝 `frontend/src/lib/components/form/auto-complete-input.svelte` (+0 -0) 📝 `frontend/src/lib/components/form/checkbox-with-label.svelte` (+3 -3) 📝 `frontend/src/lib/components/form/custom-claims-input.svelte` (+1 -1) 📝 `frontend/src/lib/components/form/file-input.svelte` (+1 -1) 📝 `frontend/src/lib/components/form/form-input.svelte` (+1 -1) ➕ `frontend/src/lib/components/form/profile-picture-settings.svelte` (+83 -0) _...and 18 more files_ </details> ### 📄 Description Fixes #58 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2025-10-07 00:24:15 +03:00

OVERLORD closed this issue

2025-10-07 00:24:16 +03:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/pocket-id#896