Skip to content

Conversation

@richieb21
Copy link
Collaborator

@richieb21 richieb21 commented Oct 4, 2025

Notion ticket link

Bug on Intake Submission

Implementation description

  • Changed all string fields to text to bypass too many characters database validation (validation should be done on frontend or backend level)

Steps to test

  1. Run the migrations
  2. Verify the type of the columns are TEXT instead of STRING

What should reviewers focus on?

  • N/A

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

@YashK2005 YashK2005 merged commit 0a3673a into main Oct 4, 2025
5 checks passed
@YashK2005 YashK2005 deleted the string-to-text-migration branch October 4, 2025 01:46
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