Skip to content

Filename removed from schema#79

Open
Anirudha0702 wants to merge 7 commits intodevfrom
DNG-97-feature/upload-pdf-to-drive
Open

Filename removed from schema#79
Anirudha0702 wants to merge 7 commits intodevfrom
DNG-97-feature/upload-pdf-to-drive

Conversation

@Anirudha0702
Copy link
Copy Markdown
Collaborator

PR Description:

  • Removed the filename field from the database schema.
  • Leveraged Cloudinary's unique public IDs to store files, removing the need to manually maintain filenames with use_filename and unique_filename as True, it generates a unique name <file_name>_<unique_id_from_cloudinary>
  • Added logic to extract the original filename from the Cloudinary public ID when needed.

This change simplifies file management and ensures filenames are always consistent with Cloudinary storage.

- Updated .example.env to include Cloudinary credentials.
- Enhanced requirements files to include Cloudinary package.
- Modified config.py to add Cloudinary settings.
- Implemented FileUploadService for handling PDF uploads to Cloudinary.
- Created UploadPDFSchema for validating upload requests.
- Added new route for uploading PDFs in user_routes.py.
- Introduced PreviousYearPaperModel for managing previous year papers.
- Updated repository structure to accommodate new models and services.
- Enhanced PreviousYearPaper model to include filename, public_id, and uploaded_by fields.
- Implemented delete functionality for PDFs in the repository and service layers.
- Updated file upload service to return additional file information.
- Modified routes to handle PDF uploads and deletions, including user identification.
@vercel
Copy link
Copy Markdown

vercel bot commented Nov 23, 2025

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

Project Deployment Preview Comments Updated (UTC)
doclin Ready Ready Preview Comment Nov 23, 2025 2:11pm

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.

1 participant