Skip to content
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.

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

merge: v2.0 #89

Merged
merged 51 commits into from
Aug 16, 2024
Merged

merge: v2.0 #89

merged 51 commits into from
Aug 16, 2024

Conversation

jinyoungbang
Copy link
Contributor

@jinyoungbang jinyoungbang commented Jul 14, 2024

What does this PR do?

Updating staging with some significant changes, but mainly revamped rush analytics/tracking

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

@jinyoungbang jinyoungbang changed the base branch from main to staging July 14, 2024 19:11
Copy link

github-actions bot commented Jul 14, 2024

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
2034 1505 74% 70% 🟢

New Files

File Coverage Status
chalicelib/api/events_member.py 96% 🟢
chalicelib/api/events_rush.py 94% 🟢
chalicelib/services/EventsMemberService.py 40% 🟢
chalicelib/services/EventsRushService.py 15% 🟢
tests/api/test_events_members.py 100% 🟢
tests/api/test_events_rush.py 100% 🟢
tests/api/test_members.py 100% 🟢
tests/services/test_events_member_service.py 100% 🟢
TOTAL 81% 🟢

Modified Files

File Coverage Status
app.py 100% 🟢
chalicelib/api/accountability.py 62% 🟢
chalicelib/api/applicants.py 100% 🟢
chalicelib/api/insights.py 100% 🟢
chalicelib/api/listings.py 88% 🟢
chalicelib/api/members.py 98% 🟢
chalicelib/modules/mongo.py 25% 🟢
chalicelib/s3.py 41% 🟢
chalicelib/services/AccountabilityService.py 47% 🟢
chalicelib/services/InsightsService.py 94% 🟢
chalicelib/services/ListingService.py 71% 🟢
chalicelib/services/MemberService.py 33% 🟢
chalicelib/utils.py 100% 🟢
tests/api/test_applicants.py 100% 🟢
tests/api/test_insights.py 100% 🟢
tests/api/test_listings.py 100% 🟢
tests/services/test_applicant_service.py 100% 🟢
tests/services/test_insights_service.py 100% 🟢
tests/services/test_listing_service.py 100% 🟢
tests/test_decorators.py 100% 🟢
tests/test_dynamodb.py 100% 🟢
TOTAL 84% 🟢

updated for commit: 0217dbe by action🐍

@jinyoungbang jinyoungbang changed the title Merge pull request #88 from whyphi/fix/add-mongomock-unittests merge: v1.0.7 Jul 14, 2024
jinyoungbang and others added 24 commits July 25, 2024 17:35
added functionality to get all events for rush category
feat: Create Update Member API + Implement Member API unit tests
…g exceptions --> otherwise user will not get 401 error even if they're unauthenticated)
feat: implement family tree get filter api
@wderocco8 wderocco8 added the breaking change Breaking Changes label Aug 16, 2024
@wderocco8 wderocco8 changed the title merge: v1.0.7 merge: v2.0 Aug 16, 2024
@wderocco8 wderocco8 marked this pull request as ready for review August 16, 2024 01:13
@wderocco8 wderocco8 merged commit f295656 into staging Aug 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change Breaking Changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants