Skip to content

Comments

NSC-events-fullstack_31_140_Create-Media-Entity-for-File-Tracking#157

Draft
LiteWilliamNguyen wants to merge 1 commit intomainfrom
origin/140-Create-Media-Entity-for-File-Tracking
Draft

NSC-events-fullstack_31_140_Create-Media-Entity-for-File-Tracking#157
LiteWilliamNguyen wants to merge 1 commit intomainfrom
origin/140-Create-Media-Entity-for-File-Tracking

Conversation

@LiteWilliamNguyen
Copy link

Summary & Changes 📃

  • Resolves: Feature Request: Create Media Entity for File Tracking #140

  • Summary: (Briefly describe what this PR does)
    Creating a dedicated Media entity to track all uploaded files with full S3 metadata. Refactoring the file upload system to store file metadata in dedicated Media entity instead of simple URL strings.

  • Changes:

  • Creating Media Module

  • Drops eventCoverPhoto and evenDocument columns from Activity Table

  • Add coverPhoto and document relations (foreign key) to Activity Table

Screenshots / Visual Aids 🔎

Expand ⬇️

How to Test 🧪

  1. Steps to Reproduce:
    • Step 1: ...
    • Step 2: ...
  2. Expected Behavior: (Describe what should happen)
  3. Actual Behavior (if bug): (Describe what happens instead)

Checklist ✅

  • I have tested this PR locally and it works as expected.
  • This PR resolves an issue (Resolves #issue-number).
  • Reviewers, assignees(self), tags, and labels are correctly assigned.
  • Squash commits and enable auto-merge if approved.

@LiteWilliamNguyen LiteWilliamNguyen self-assigned this Feb 7, 2026
@LiteWilliamNguyen
Copy link
Author

Still a working in progress.

Copy link
Contributor

@NahomAlemu NahomAlemu left a comment

Choose a reason for hiding this comment

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

You've made great progress on the Media Entity implementation here.

However, it looks like this issue was actually resolved and merged recently. The reason you likely didn't see it is that the existing implementation was placed in a different directory (nsc-events-nestjs/src/media/entities/media.entity.ts) than this PR.

Since the work is already in main, we should probably close this draft PR to avoid any merge conflicts or duplicate logic. Sorry about the overlap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants