feature/clean-public-urls #1

Merged

admin merged 6 commits from feature/clean-public-urls into main 2025-11-21 12:14:34 +00:00

Conversation 0 Commits 6 Files Changed 6 +575 -6

admin commented 2025-11-21 12:14:12 +00:00

Owner

Copy Link

No description provided.

admin added 6 commits 2025-11-21 12:14:12 +00:00

feat: implement clean public URLs and slug-based image storage ... d5b78d7449

- Update Firebase service to use clean GCS URLs instead of MediaLink
- Set ACL to make uploaded files publicly accessible
- Use slug as image filename to prevent overwrites
- Add unique constraints on slug columns in books and stationery tables
- Add slug existence checks before upload (409 Conflict if duplicate)
- Update storage paths: /jd-bookshop/books and /jd-bookshop/stationery
- Remove year/month from storage paths as requested
- Construct URLs: https://storage.googleapis.com/{bucket}/{encoded-path}

Changes:
- firebase.go: Set ACL, construct clean URLs using url.PathEscape
- book.go & stationery.go: Check slug before upload, use correct paths
- book_service.go & stationery_service.go: Add slug existence check functions
- migration: Add UNIQUE constraints on slug columns

chore: update .gitignore to include Gitea workflow files ... 9a5508ffe8

- Added .gitea/workflows/deploy.yml and .gitea/workflows/build.yml to .gitignore to prevent tracking of workflow configuration files.

chore: remove unnecessary blank line in Gitea workflow configuration ... 27f6629308

- Cleaned up the .gitea/workflows/test.yml file by removing an extra blank line for better readability.

fix: correct heading in Gitea workflow documentation ... 528ae0072e

- Updated the heading from "Related Documentation" to "Related Documentations" for consistency in the .gitea/workflows/README.md file.

Add Gitea workflows for CI/CD pipeline a52b2d13f0

chore: update .gitignore to include additional Gitea workflow files ... ca62651c3c

- Added .gitea/workflows/deploy.yml and .gitea/workflows/build.yml to .gitignore to prevent tracking of these workflow configuration files.

admin merged commit a43658af84 into main 2025-11-21 12:14:34 +00:00

admin referenced this issue from a commit 2025-11-21 12:14:35 +00:00

Merge pull request 'feature/clean-public-urls' (#1) from feature/clean-public-urls into main

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

admin

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: admin/jd-book-uploader-backend#1