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

DataStore のレコード削除メソッドのサンプル実装 #76

Merged
merged 11 commits into from Aug 15, 2021

Conversation

kaznak
Copy link
Contributor

@kaznak kaznak commented Jul 20, 2021

issue #75 の DataStore のレコード削除メソッドのサンプル実装です。変更点は以下の通りです。

  1. get に対応する delete メソッドとそのテストコードを追加
  2. find に対応する findel メソッドとそのテストコードを追加

onmessage の挙動は変えていません。

@kaznak kaznak marked this pull request as ready for review July 20, 2021 11:36
@MtkN1
Copy link
Member

MtkN1 commented Aug 15, 2021

こちら対応が遅くなってしまって申し訳ないです🙇‍♂️

  • メソッド名を以下のように変更しました
    • delete -> _pop (標準の dict.pop より)
    • findel -> _find_and_delete (ライブラリpymongopymongo.collection.find_one_and_delete より)
  • 試験的なメソッドと言う意味合いでアンダースコア(_)を付けました
    • (シンプルなインターフェースを保ちたい為、ユースケースを見定めたら公開メソッドにします)

こちらの内容でマージさせて頂きます。

(追記)
onmessageでの非アクティブオーダー削除は取りやめない。#75#77 の実装で対応する。

@MtkN1 MtkN1 merged commit f25934c into pybotters:develop Aug 15, 2021
@MtkN1 MtkN1 mentioned this pull request Aug 15, 2021
MtkN1 added a commit that referenced this pull request Aug 15, 2021
✨v0.6.0リリース

## Issues
✅ 対応取引所の追加(Phemex、Coincheck) #52
✅ GMO コインの DataStore を実装する #60
✅ リリース表記をPreview版にする #73
✅ FTXDataStore の orders チャネルにおける close されたオーダーの取り扱いについて #75
✅ DataStoreInterfaceのクラス名を変更する #79
## Pull requests
✅ GMO コイン用の DataStore を一部実装する #74
✅ DataStore のレコード削除メソッドのサンプル実装 #76
✅ DataStore の wait メソッドが変更されたデータを返す実装 #77
✅ Sphinxのドキュメントを追加 #78
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.

None yet

2 participants