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

Skip to content

Conversation

@wderocco8
Copy link
Member

What does this PR do?

  • removed case sensitivity when checking in as rushee
  • popped attendees form rush events

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

Screen.Recording.2025-01-27.at.9.25.17.PM.mov

Additional Comments

@github-actions
Copy link

github-actions bot commented Jan 28, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
2118 1531 72% 70% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
chalicelib/services/EventsRushService.py 15% 🟢
TOTAL 15% 🟢

updated for commit: 37a5730 by action🐍

Copy link
Contributor

@jinyoungbang jinyoungbang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks for the quick fix!

@wderocco8 wderocco8 merged commit d148985 into staging Jan 28, 2025
1 check passed
@wderocco8 wderocco8 mentioned this pull request Jan 28, 2025
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants