Regenerate application migration with pandadoc-matching fields#117
Closed
SamNie2027 wants to merge 254 commits intomainfrom
Closed
Regenerate application migration with pandadoc-matching fields#117SamNie2027 wants to merge 254 commits intomainfrom
SamNie2027 wants to merge 254 commits intomainfrom
Conversation
adding comments to explain type error
Create application entity
[Issue #16] - create discipline entity in TypeORM
…le and application types
added testing removed deprecated on update adding user authentication by adding provider deleted old migration file and used just the main one
151 update name admin
migrations for creating learner and volunteer info tables
…plications-by-discipline implemented endpoints
…ine dto, added tests and comments for new dto
Removed updateApplicationInterest dto, added updateApplicationDiscipl…
…lication Adds all pandadoc fields that were missing
implemented changes
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ℹ️ Issue
Closes
📝 Description
Regenerates application migration with the bare minimum new enums needed for it to work. Also removed the old migration tables
✔️ Verification
\dt and \d application show the correct output
🏕️ (Optional) Future Work / Notes
The old enums are still at the old values: that will be worked on in a future ticket.
The seeding for applications is still commented out because the enums are operating with new values - which the types are to also be updated in that same future ticket, but for now they are still commented out.