create account

RE: SteemData Notify 代码学习二: Confirmation Worker / Code Study of SteemData Notify: Part two by oflyhigh

View this thread on: hive.blogpeakd.comecency.com

Viewing a response to: @furion/re-oflyhigh-steemdata-notify-confirmation-worker-code-study-of-steemdata-notify-part-two-20170709t175656325z

· @oflyhigh ·
Hi @furion, Thank you very much for providing the `SteemData Notify` service.

I had already explained the reasons for the different memo lengths in this article, Thank you again for the clarification.

As for Bug,

```
    settings = db.settings.find_one({'_id': _id})
    if settings:
        db.settings.update_one(
            {'_id': _id, 'username': op['from']},
            {'$set': {'confirmed': True}}
        )
```

I think it would be better to change above code to:
```
    settings = db.settings.find_one({'_id': _id, 'username': op['from']})
    if settings:
        db.settings.update_one(
            {'_id': _id, 'username': op['from']},
            {'$set': {'confirmed': True}}
        )
```
Otherwise, It may cause minor problems in some extreme cases . 
I'll submit this issues on Github when I finished the study of your great code.
properties (22)
authoroflyhigh
permlinkre-furion-re-oflyhigh-steemdata-notify-confirmation-worker-code-study-of-steemdata-notify-part-two-20170710t004455280z
categorycn
json_metadata{"tags":["cn"],"users":["furion"],"app":"steemit/0.1"}
created2017-07-10 00:45:00
last_update2017-07-10 00:45:00
depth2
children0
last_payout2017-07-17 00:45:00
cashout_time1969-12-31 23:59:59
total_payout_value0.000 HBD
curator_payout_value0.000 HBD
pending_payout_value0.000 HBD
promoted0.000 HBD
body_length854
author_reputation6,360,684,687,598,887
root_title"SteemData Notify 代码学习二: Confirmation Worker / Code Study of SteemData Notify: Part two"
beneficiaries[]
max_accepted_payout1,000,000.000 HBD
percent_hbd10,000
post_id7,928,904
net_rshares0