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.

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

[WIP] Add remove cf data with graphQL #234

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[WIP] Add remove cf data with graphQL #234

wants to merge 1 commit into from

Conversation

BLYKIM
Copy link
Contributor

@BLYKIM BLYKIM commented Jan 6, 2023 •
edited
Loading

  • Add deleteAll query with protocols(cf names): Vec<String>
  • without protocol, delete all data in database

Close: #233

@BLYKIM BLYKIM requested a review from msk January 6, 2023 07:05
@codecov-commenter
Copy link

Codecov Report

Base: 80.04% // Head: 80.05% // Increases project coverage by +0.01% 🎉

Coverage data is based on head (abc6861) compared to base (5f71b00).
Patch coverage: 80.95% of modified lines in pull request are covered.

❗ Current head abc6861 differs from pull request most recent head e7fb278. Consider uploading reports for the commit e7fb278 to get more accurate results

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #234      +/-   ##
==========================================
+ Coverage   80.04%   80.05%   +0.01%     
==========================================
  Files          17       18       +1     
  Lines        3703     3745      +42     
==========================================
+ Hits         2964     2998      +34     
- Misses        739      747       +8
Impacted Files Coverage Δ
src/graphql.rs 66.23% <ø> (ø)
src/storage.rs 67.82% <77.14%> (+1.95%) ⬆️
src/graphql/delete.rs 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@msk msk mentioned this pull request Jan 9, 2023
Open
@BLYKIM BLYKIM force-pushed the bly/delete_cf branch 2 times, most recently from 4d98de6 to 6172c3a Compare January 27, 2023 07:42
@BLYKIM BLYKIM changed the title Add remove all data with graphQL Add remove cf data with graphQL Jan 30, 2023
@BLYKIM BLYKIM force-pushed the bly/delete_cf branch 2 times, most recently from 53ad3e3 to 8f8765c Compare February 13, 2023 02:20
@BLYKIM BLYKIM changed the title Add remove cf data with graphQL [WIP] Add remove cf data with graphQL Apr 21, 2023
@BLYKIM
Add remove all data with graphQL
45ca0d9 
- Add deleteAll query with protocol(cf name)
- without protocol, delete all data in database

Close: #233
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@msk msk Awaiting requested review from msk

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

데이터베이스 초기화 기능
2 participants
@BLYKIM @codecov-commenter