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

Skip to content

refactor: modify search logic on sheets #57

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 1 commit into from
Apr 17, 2024
Merged

refactor: modify search logic on sheets #57

jinyoungbang merged 1 commit into from
Apr 17, 2024

Conversation

@jinyoungbang
Copy link
Contributor

@jinyoungbang jinyoungbang commented Apr 17, 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

@github-actions
Copy link

github-actions bot commented Apr 17, 2024

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1431 1050 73% 70% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
chalicelib/modules/google_sheets.py 31% 🟢
chalicelib/services/EventService.py 21% 🟢
TOTAL 26% 🟢

updated for commit: b999a78 by action🐍

@jinyoungbang jinyoungbang merged commit 0eda041 into dev/v1.0 Apr 17, 2024
@jinyoungbang jinyoungbang deleted the refactor/sheets-search-user branch April 17, 2024 23:21
@jinyoungbang jinyoungbang added the improvement Enhancements label Jul 9, 2024
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

improvement Enhancements

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jinyoungbang