Skip to content

Implement migration to allow multiple submissions per contest while p…#105

Merged
GauriGupta21 merged 1 commit intoindictechcom:mainfrom
Aditya0545:fix/submittion
Dec 27, 2025
Merged

Implement migration to allow multiple submissions per contest while p…#105
GauriGupta21 merged 1 commit intoindictechcom:mainfrom
Aditya0545:fix/submittion

Conversation

@Aditya0545
Copy link
Collaborator

…reventing duplicate articles

  • Added Alembic migration to update the submissions table, converting article_created_at from VARCHAR to DATETIME format.
  • Updated unique constraint to allow multiple submissions per user per contest, while ensuring article uniqueness using a prefix index on article_link.
  • Modified submission logic to handle duplicate article submissions and improved error handling in the API.
  • Adjusted model and route files to reflect changes in submission constraints and error messaging.
image

…reventing duplicate articles

- Added Alembic migration to update the `submissions` table, converting `article_created_at` from VARCHAR to DATETIME format.
- Updated unique constraint to allow multiple submissions per user per contest, while ensuring article uniqueness using a prefix index on `article_link`.
- Modified submission logic to handle duplicate article submissions and improved error handling in the API.
- Adjusted model and route files to reflect changes in submission constraints and error messaging.
@Aditya0545 Aditya0545 self-assigned this Dec 26, 2025
@Aditya0545 Aditya0545 added the enhancement New feature or request label Dec 26, 2025
@GauriGupta21
Copy link
Collaborator

Just to align on process, during the last call, I had mentioned that I’d be working on this.

@Aditya0545
Copy link
Collaborator Author

This anomalies was from my side when i implemented submission logic then created this submit restrictions so i thought it's better to fix from my side> Just to align on process, during the last call, I had mentioned that I’d be working on this.

@GauriGupta21
Copy link
Collaborator

Thanks for the context. Just to clarify we had discussed this earlier and I had shared that I was already working on it. Since I wasn’t updated that parallel work was also in progress, I continued with my implementation, which led to overlapping efforts.

@GauriGupta21 GauriGupta21 merged commit 3345118 into indictechcom:main Dec 27, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants