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

Skip to content

merge: v1.0 alpha test #44

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 48 commits into from
Mar 31, 2024
Merged

merge: v1.0 alpha test #44

jinyoungbang merged 48 commits into from
Mar 31, 2024

Conversation

@jinyoungbang
Copy link
Contributor

@jinyoungbang jinyoungbang commented Mar 31, 2024

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

wderocco8 and others added 30 commits January 17, 2024 17:07
@wderocco8
added initial support for insights api in backend
7d0aa19
@wderocco8
added more support for InsightsService
4ac3395
@wderocco8
updated InsightsService
4f02491
@wderocco8
merged dev/0.2 into insights
2cca314
@wderocco8
fudged up frontend commit message, this should be: updated InsightsSe…
8e8b368 
…rvice to send dashboard analytics
@wderocco8
completed InsightsService functionality
2243f84
@wderocco8
renamed variables
bf39512
@wderocco8
fixed bug -> handled metric=colleges case with continue statement
4720a6e
@wderocco8
removed case for website/linkedin -> not necessary
22b30a4
@wderocco8
added title case for major/minor and fixed bugs for avg gpa
b7ed429
@wderocco8
fixed bug -> checking for missing website/linkedin
6700896
@wderocco8
handled N/A gpa case
1c7fc19
@wderocco8
created unit tests for insights service -> no clue what is breaking
54e3003
@wderocco8
found bug --> accidentally commented out InsightsService driver funct…
3576562 
…ion SMH bruh bruh bruh
@wderocco8
completed test_insights_service
02cfc8b
@wderocco8
created JSON files to store sample return values and integrated into …
7f8b441 
…test_insights_service
@wderocco8
added API pytests for insights
a33b5d5
@wderocco8
slight refactor -> used snake case
6223061
@wderocco8
refactor -> switched to most common gradYear (instead of average) and…
e4835c3 
… updated unit-tests
@wderocco8
bug fix -> display insights template for 0 applicants
655c2d4
@wderocco8
bug fix -> switched to 'hasURL' instead of 'True' (clears up confusio…
9839ab2 
…n on unit tests
@wderocco8
used Jinyoung's lambda shortcut
a29b9db
@wderocco8
Merge pull request #39 from whyphi/feat/data-analytics
c6fbfee 
Feat/data analytics (code refactor)
@wderocco8
added more error handling (0 gpas, gradYears, majors)
2d4d682
@wderocco8
added more unit tests for edge cases (missing applicants, gpas...)
b765ba8
@wderocco8
Merge pull request #40 from whyphi/fix/insights-edge-cases
5558864 
Fix/insights edge cases
@wderocco8
push
45c50f3
@wderocco8
Merge remote-tracking branch 'origin/staging' into dev/v0.2
e070512
@jinyoungbang
feat: implement route protection through JWT
0d746ca
@jinyoungbang
chore: add docstring comments
a522aa9
jinyoungbang and others added 18 commits March 3, 2024 20:36
@jinyoungbang
chore: edit docstring
e5144a2
@jinyoungbang
refactor: utilize chalice built-in exception
73a09f8
@jinyoungbang
refactor: implement auth on unit tests
5fb3370
@jinyoungbang
refactor: use enums on role-based auth
6ecab03
@jinyoungbang
chore: fix spelling err
37294b8
@jinyoungbang
Merge pull request #41 from whyphi/feat/protected-routes
c77878b 
feat: Implement Route Protection through JWT
@jinyoungbang
fix: remove jwt auth on get_listing
31d9b88
@jinyoungbang
feat: implement member onboarding API
f68bfe6
@jinyoungbang
Merge pull request #42 from whyphi/feat/onboarding-api
f549632 
feat: Onboarding API
@jinyoungbang
feat: create boilerplate APIs for events
1785085
@jinyoungbang
feat: create timeframe-related apis
5e188e5
@jinyoungbang
feat: implement create_event api
d350f99
@jinyoungbang
feat: implement get_event API
70de161
@jinyoungbang
feat: implement checkin API
e6e4af4
@jinyoungbang
feat: implement delete event api
06ce572
@jinyoungbang
refactor: change logic and fields of event creation
4aa69d2
@jinyoungbang
chore: rename function
dd2d805
@jinyoungbang
Merge pull request #43 from whyphi/feat/events-api
071e48b 
feat: Create Events related API endpoints
@jinyoungbang jinyoungbang marked this pull request as ready for review March 31, 2024 14:50
@jinyoungbang jinyoungbang merged commit eacd765 into staging Mar 31, 2024
@github-actions
Copy link

github-actions bot commented Mar 31, 2024

☂️ Python Coverage

current status: ❌

Overall Coverage

Lines Covered Coverage Threshold Status
1233 982 80% 80% 🔴

New Files

File Coverage Status
chalicelib/api/events.py 71% 🟢
chalicelib/api/insights.py 100% 🟢
chalicelib/models/roles.py 100% 🟢
chalicelib/services/EventService.py 33% 🟢
chalicelib/services/InsightsService.py 94% 🟢
tests/api/test_insights.py 100% 🟢
tests/services/test_insights_service.py 100% 🟢
TOTAL 86% 🟢

Modified Files

File Coverage Status
app.py 100% 🟢
chalicelib/api/applicants.py 100% 🟢
chalicelib/api/listings.py 88% 🟢
chalicelib/api/members.py 63% 🟢
chalicelib/decorators.py 81% 🟢
chalicelib/modules/mongo.py 30% 🟢
chalicelib/services/MemberService.py 65% 🟢
tests/api/test_applicants.py 100% 🟢
tests/api/test_listings.py 100% 🟢
TOTAL 81% 🟢

updated for commit: 071e48b by action🐍

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.

3 participants

@jinyoungbang @wderocco8