Thanks to visit codestin.com
Credit goes to github.com

Skip to content

v3.1 Release #146

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
Merged
jinyoungbang merged 180 commits into from
Jun 7, 2025
Merged

v3.1 Release #146

jinyoungbang merged 180 commits into from
Jun 7, 2025

Conversation

@jinyoungbang
Copy link
Contributor

@jinyoungbang jinyoungbang commented Jun 1, 2025

What does this PR do?

Type of change

  • Fix: Bug fix (non-breaking change which fixes an issue)
  • Refactor: Any code refactoring
  • Chore: technical debt, workflow improvements
  • Feature: New feature (non-breaking change which adds functionality)
  • Documentation: This change requires a documentation update
  • Merge: Pushing features to an upper envrironment

Tests Performed

Screenshots

Additional Comments

vinli0921 and others added 30 commits January 21, 2025 19:13
@vinli0921
created service file for email broadcasting--job scraping from google…
96012d9 
… sheets
@vinli0921
parse_job_types function able to parse one sheet, needs internal team…
fdd5610 
… to update spreadsheet so all sheets are formatted the same
@vinli0921
began broadcast service framework
ba73fd2
@MattFromBeijing
feat: completed jobpostingservice
cb75adb
@vinli0921
Merge feat/jobposting into feat/broadcast
8dc511f
@vinli0921
updated job posting service, now pulling from online repositories. up…
d1e217a 
…dated broadcast services which utilize the job posting service to send a formatted emails with the job postings
@vinli0921
changed api endpoint from accouncement to broadcast
2e9310a
@vinli0921
added selenium and webdriver manager for job posting scraping capabil…
3a633cc 
…ities
@vinli0921
updated get jobs function to scrape from multiple repositories
75b6747
@vinli0921
wrote unit tests for for the job posting service
133fa49
@vinli0921
wrapped webdriver creation arguments in a function, updated docstrings
3573066
@vinli0921
updated the webdriver creation function to take arguments
b4bbe7a
@vinli0921
updated job posting service import
2a7aa54
@vinli0921
fixed company name not showing up for crve repository
9f8e1f1
@vinli0921
repository links were incorrect
b28bf26
@vinli0921
updated getJobs docstring
7ba2523
@vinli0921
updated unit tests to reflect job posting service edge case change
fcb471b
@vinli0921
Merge pull request #128 from whyphi/feat/broadcast
41f11e0 
Feat/broadcast
@vinli0921
two functions get_hyperlink_from_grid_data and get_sheet_with_grid_da…
89a54e3 
…ta so that we can extract hyperlinks that were used by the google sheets insert functionality
@vinli0921
updated css styling for the email
c6fcea9
@vinli0921
updated get finance jobs functionality to get all hyperlinks and adde…
3f5deab 
…d better error handling for dates
@vinli0921
updated temporary beta email list
b36405b
@vinli0921
fixed link break for job source html
d44746c
@vinli0921
updated unit test to factor in new google sheets hyperlink functionality
8e3c5f5
@vinli0921
updated job sources text
f651b34
@vinli0921
updated get_sheet_with_grid_data and get_hyperlink_from_grid_data for…
596d218 
… better code readability
@vinli0921
added function to store css styling in utils instead of BroadcastService
c0f33fd
@vinli0921
updated gitFinanceJobs function to use constands for better readabili…
e4677ee 
…ty, updated isMoreThanOneWeekAgo for more robust error handling and a TODO
@vinli0921
moved job sources to a constant, moved css styling string to utils.py
3f046b3
@vinli0921
added auth decorator to send_announcement, removed beta list form commit
40ce8de
wderocco8 added 29 commits June 4, 2025 08:39
@wderocco8
added update exceptions
da8d3ca
@wderocco8
added toggle_boolean_field exceptions
8374857
@wderocco8
added delete exceptions
473c560
@wderocco8
Merge pull request #147 from whyphi/feat/will/migration/pytests
776e315 
Feat/will/migration/pytests
@wderocco8
added env variables for pytests on GitHub actions
1189a37
@wderocco8
lazy-init for pytest on github actions
f021a4a
@wderocco8
fixed lazy-init in base repo too (might need a more robust strategy)
e712e54
@wderocco8
refetch env for each client/repo init
b81dcd4
@wderocco8
removed redundant comments
553c315
@wderocco8
moved apply to listings
8a738f7
@wderocco8
moved apply applicant tests to listings
b4af9a2
@wderocco8
Merge pull request #148 from whyphi/refactor/apply-endpoint
347d2ea 
Refactor/apply endpoint
@wderocco8
trimmed migration script (only includes dynamo data for now)
f231af5
@wderocco8
Merge branch 'dev/v3.0' into refactor/will/migrate-supabase
4e102dd
@wderocco8
uninstalled web-driver deps and temporarily removed broadcasting
782316a
@wderocco8
moved dev packages
952c9c0
@wderocco8
updated coverage flow to also include dev requirements
df42e5f
@wderocco8
synced with refactor branch
14a563c
@wderocco8
mongomock is non-dev
627f5a7
@wderocco8
Merge pull request #149 from whyphi/fix/will/migration/remove-job-lis…
e35b465 
…tings

Fix/will/migration/remove job listings
@wderocco8
added handle exceptions
0e4b1a0
@wderocco8
application model typo
fa9b8b3
@wderocco8
added supabase resources to policies
52692ca
@wderocco8
Merge pull request #136 from whyphi/refactor/will/migrate-supabase
b9e806a 
Refactor/will/migrate supabase
@wderocco8
Merge pull request #150 from whyphi/dev/v3.0
f7b638d 
dev/v3.1 release (supabase migration)
@wderocco8
dummy commit to test supabase migration ci/cd (readme)
2dd6969
@wderocco8
Merge pull request #151 from whyphi/dev/v3.0
3365cd3 
dummy commit to test supabase migration ci/cd (readme)
@wderocco8
nested workflows not allowed
5647d1b
@wderocco8
Merge pull request #152 from whyphi/dev/v3.0
abb65e0 
Fixed nested workflows
@jinyoungbang jinyoungbang merged commit a43846b into main Jun 7, 2025
6 checks passed
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

release New version released to prod

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jinyoungbang @vinli0921 @MattFromBeijing @wderocco8