Skip to content

Refactored Area of Interest to be an array of enum #116

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
bodhiYG wants to merge 221 commits into
base: main
Choose a base branch
from
Open

Refactored Area of Interest to be an array of enum #116

bodhiYG wants to merge 221 commits into from

Conversation

@bodhiYG
Copy link

@bodhiYG bodhiYG commented Feb 11, 2026

ℹ️ Issue

Closes

📝 Description

Refactored Area of Interest to be an array of enums to better match the Pandadoc, where you can multi-select areas of interest

Briefly list the changes made to the code:

  1. Updated column to be an array of enum
  2. Updated all references accordingly
  3. Added a migration to convert the interest column to an array.

✔️ Verification

🏕️ (Optional) Future Work / Notes

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
@chnnick
delete learner route + tests
539ce73
SamNie2027 and others added 30 commits January 27, 2026 17:59
@SamNie2027
Using local cloud
5dc7280
@rayyanmridha
fix discipline to use enum instead of string
d31c33d
@SamNie2027
Merge pull request #140 from Code-4-Community/backend-refactoring-pt2
b44bb1d 
(new) backend refactor pt 2
@chnnick
test case for when mockLearnersRepository throws error
6133d28
@chnnick
Merge branch 'main' of https://github.com/Code-4-Community/proj-bhchp …
e03fff6 
…into delete-learner-route
@chnnick
finished merging, changed references to learner in controller and ser…
415523b 
…vice to be applicant
@chnnick
fixed testing typos
bf3e1e3
@SamNie2027
Adding validation that error is emitted at the controller level
e462c3c
@SamNie2027
Merge pull request #111 from Code-4-Community/delete-learner-route
bf5bf6a 
Delete learner route
@SamNie2027
Revise README for BHCHP Application setup
de24def 
Updated README to reflect new application name and setup instructions.
@SamNie2027
Improve formatting of commands in README
41ff4e4 
Formatted commands in the README for better readability.
@SamNie2027
Improve README setup instructions for nx
e9daaf4 
Updated instructions for installing nx globally.
@SamNie2027
Update project title casing in README.md
ca4bd63
@SamNie2027
Enhance README with local setup and database instructions
d317115 
Updated README to include setup instructions for local development and database configuration.
@SamNie2027
Revise README for PostgreSQL setup instructions
327ffb7 
Updated PostgreSQL installation instructions for Windows and MacOS.
@rayyanmridha
replace test endpoint with a generic one
2a6d530
@SamNie2027
Update README.md
c87f838
@rayyanmridha
Merge pull request #146 from Code-4-Community/SamNie2027-patch-1
f73e38c 
Adds local development setup instructions as readme
@SamNie2027
Changing the name of the method
5ddd8e0
@SamNie2027
Merge pull request #149 from Code-4-Community/125-aws-ses-troubleshoot
c22d1c4 
Replace test email endpoint with a generic one
@chnnick
migrations for creating learner and volunteer info tables
9962925
@chnnick
Admin changes
d148622
@chnnick
user changes id to appId.
022121f
@chnnick
updated controller tests and delete-user-DTO to use appId
176746a
@SamNie2027
Merge pull request #159 from Code-4-Community/151-update-name-admin
48b0b8f 
151 update name admin
@chnnick
seed data in txt file for volunteer and learner
a3faa69
@SamNie2027
Merge pull request #158 from Code-4-Community/128-refactoring-part-6
d673497 
migrations for creating learner and volunteer info tables
@ostepan8
implemented endpoints
23f0146
@SamNie2027
Merge pull request #166 from Code-4-Community/endpoint-to-retrieve-ap…
abe70e8 
…plications-by-discipline

implemented endpoints
@bodhiYG
Refactored Area of Interest to be an array of enum
d4ee67c
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

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