Skip to content

alveena/backend refactor pt2 #107

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
alveenahanif wants to merge 180 commits into
base: main
Choose a base branch
from
Open

alveena/backend refactor pt2 #107

alveenahanif wants to merge 180 commits into from

Conversation

@alveenahanif
Copy link

@alveenahanif alveenahanif commented Jan 27, 2026

Closes backend-refactor-pt2

This PR adds two new required fields to the Application entity to align with the updated data model.
The DTO and service tests were updated to ensure the new fields are properly validated and tested.

Changes made:

  1. Added two new fields to the Application entity with appropriate types and documentation.
  2. Updated CreateApplicationDto to include the new required fields.
  3. Updated application service tests to account for the new fields.
  • Ran the existing application service tests locally.
  • Updated failing tests to include the new required fields.
  • Verified all tests pass after changes.

piersolh and others added 30 commits September 14, 2025 14:37
@piersolh
adding migrations
50ac20e
@piersolh
adding back example env
749cdab
@piersolh
Merge pull request #1 from Code-4-Community/HP-adding-migrations
b144c4b 
Hp adding migrations - test
@piersolh
adding comments to explain type error
f3d586e
@piersolh
Merge pull request #24 from Code-4-Community/HP-adding-migrations
9dc4a94 
adding comments to explain type error
@chnnick
put interceptor decorators in auth controller and module
7cbd8a8
@ItsEricSun
added new application entity and new enums
e47cf24
@piersolh
setting up scaffolding for disciplines entity
b14ae2a
@piersolh
adding seeding and creating a disciplines enum
4cef618
@piersolh
fixing typo
683505a
@rayyanmridha
add initial aws module
d0cc629
@ostepan8
added admin controller, entity, module, and service
d990085
@ostepan8
small changes to ensure functionality
ab3e30c
@maxn990
fix s3 403 error
0d93119
@rayyanmridha
make sure test doesn't run automatically
2962fa2
@piersolh
Merge pull request #35 from Code-4-Community/create-application-entity
25719d0 
Create application entity
@piersolh
Merge pull request #36 from Code-4-Community/HP-create-discipline-entity
a3ad5b4 
[Issue #16] -  create discipline entity in TypeORM
@ItsEricSun
add get and post Application routes and some clean up of migration fi…
47af7aa 
…le and application types
@chnnick
controller and service tests for cognito
0cf8822
@ItsEricSun
add application with ID not found error
9759078
@ItsEricSun
add application not found to tests
d15bb85
@ItsEricSun
Merge branch 'main' into create-get-and-post-Application-routes
c64a30f
@ostepan8
updated pr for comments
bbdd87e 
added testing
removed deprecated on update
adding user authentication by adding provider
deleted old migration file and used just the main one
@ostepan8
changed the update to only update the email
ce1d5a7
@ostepan8
make migration dynamic
3ddea89
@chnnick
GET and POST Learner Routes + Testing
28a2dd9
@chnnick
got rid of currentUserInterceptor in controller tests, not sure if ne…
6847ad4 
…eded
@ItsEricSun
create remove and edit application routes
5ddac0d
@ItsEricSun
add tests for new service methods
d8b2c29
@rayyanmridha
install ses
20a224f
ostepan8 and others added 30 commits January 20, 2026 09:39
@ostepan8
implement validation checks
b01fa16
@SamNie2027
Merge pull request #118 from Code-4-Community/application-input-valid…
55dceec 
…ation

implement validation checks
@SamNie2027
Merge pull request #119 from Code-4-Community/edit-remove-create-disc…
5e76ddc 
…ipline-routes

added discipline CRUD routes
@SamNie2027
Additional documentation and added TODO
1d038f4
@SamNie2027
Merge pull request #112 from Code-4-Community/HP-create-discipline-ro…
66e1beb 
…utes

create basic CRUD ops for disciplines
@SamNie2027
Merge branch 'main' into application-table-columns
5191952
@chnnick
Created DaysOfTheWeek and ApplicantType enums to be used in the entit…
28c5d04 
…y + DTO instead of previous strings, refactored tests to match new entity + DTO
@chnnick
merged in old components + moved the application table to the compone…
f77e244 
…nts folder
@SamNie2027
Adding bottom margin, for some reason mb-5 class doesn't work
65532b7
@SamNie2027
Merge pull request #117 from Code-4-Community/application-table-columns
7804bbf 
chakraprovider wrapper in main.tsx + table
@SamNie2027
Adding documentation
310def9
@SamNie2027
Merge pull request #126 from Code-4-Community/114-nick-anjali-backend…
49e15dd 
…-refactor

new entities + tests
@SamNie2027
Adding Learner Info entity
d5a88fb
@SamNie2027
Create create-learner-info-request.dto.ts
04c7098
@SamNie2027
Adding entire Create pipeline
2b23639
@SamNie2027
Test setup for service
dcc7a06
@SamNie2027
Controller test setup
96954ad
@SamNie2027
Added service tests for create
0deb038
@SamNie2027
Adding controller tests for learner info create
bb76fff
@SamNie2027
Adding Update Application Interest endpoint
1167e5f
@SamNie2027
tests for create, findbyid, and update for the service
9e6b03c
@SamNie2027
Tests for create and update
4eab7b8
@SamNie2027
Adding full volunteer info component
e46acac
@SamNie2027
Adding appropriate type casts
b301fc2
@SamNie2027
Adding Learner DTO and splitting name into first name and last name
e9705c5
@SamNie2027
Merge branch '113-ui-for-profiles-part-1' into 120-work-on-backend-re…
e5dea25 
…factoring-pt-3-learner-info
@SamNie2027
Merge pull request #132 from Code-4-Community/120-work-on-backend-ref…
b0c5f3a 
…actoring-pt-3-learner-info
@rayyanmridha
refactor Learner to Applicant
8f044fd 
Renames the learner entity to Applicant and replaces id with appId as the identifier
@rayyanmridha
fix env path
ede36ab
@alveenahanif
Add new Application fields and update DTO and service tests
15b859b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

10 participants

@alveenahanif @piersolh @chnnick @ItsEricSun @rayyanmridha @ostepan8 @maxn990 @yumi520 @bodhiYG @SamNie2027