search

cyberway / golos.contracts

Smart contract for Golos dApp
http://golos.io
MIT License
5 stars 3 forks source link

Remove writing pin/block records from social #905 #906

Closed soft-bagel-93 closed 5 years ago

soft-bagel-93 commented 5 years ago

Resolve #905. What is done: See below: https://github.com/cyberway/golos.contracts/pull/906#issuecomment-536137190

gropox commented 5 years ago

А что будет вместо удаленного pinblock-а? А то я начал использовать в своих ботах эти данные. Есть уже план?

Спасибо

soft-bagel-93 commented 5 years ago

@gropox какие именно данные? Сами экшны pin, unpin, block, unblock остаются, просто больше не хранятся записи в монге, теперь их хранить будет IO.

gropox commented 5 years ago

@boostmultifruit вот как раз данные из таблицы pinblock и брал.

В смартконтракте бота 

        tables::pinblock_table blacklist_table(golos::config::social_name, get_self().value);
        auto block = blacklist_table.find(author.value);
        if(block != blacklist_table.end() && block->blocking) {
            return "Автор в черном списке @" + author.to_string();          
        }

и в другом телеграмном боте, из базы монги вытаскивал черный список, что бы не информировать людей о сообщениях от заблокированных пользователей.

Это было очень удобно. Если уберете, то окологолосинным приложениям придется туго и тоже надо будет ловить экшены и хранить все в своей базе.

Со смартом ладно, там я могу свой черный список завести, а вот с ботом чуть тяжелее. Свою базу делать + сканировать эвенты с самого начала, если ботом к блокчейну не сначала подключаться.

Все конечно некритично, но все же ))

И жалко что убираете блокировку постинга под под постом автора заблокировавшего пользователя. Было бы интересно сохранить. Может этот вопрос как раз обсудить сначала с лидерами, что бы не заводить в тупик. Ведь любой сможет тогда писать, хоть и заблокированный.

afalaleev commented 5 years ago

At the current time, block and pin lists don't contain all exist records. As result client fails on removing of pin/block records.

So, the main change is to disable fails on removing not-existing records in contacts tables.

And as @gropox says, we should keep saving of pin/block records.

soft-bagel-93 commented 5 years ago

@gropox Мы оставляем сохранение записей в таблице pin/block, но убираем ошибку, которая возникает при попытке отписаться от аккаунта, если подписан на него до переезда и записи в таблице нет. Проверку блокировки при постинге мы убираем, но временно. Позже, добавив недостающие записи, мы планируем вернуть весь этот функционал.

soft-bagel-93 commented 5 years ago

@afalaleev What is done: