Closed GoogleCodeExporter closed 9 years ago
Написано неплохо, но есть пара замечаний, я
сделал исправление здесь:
https://code.google.com/p/any-balance-providers/source/detail?r=4305
Этот провайдер типа финансы, поэтому в
категорию стоит поставить money, как и в ID
(рекомендуется).
провайдер не привязан к стране, поэтому не
указываем никакую страну.
Счетчики там, насколько я понимаю,
числовые. Лучше тогда их числами и
показывать, иначе по ним не будет возможно
построение графиков.
Проверьте, что изменения корректны, и тогда
можно будет опубликовать. Я дал вам доступ
на запись в репозиторий, можете
отредактировать провайдер самостоятельно.
Original comment by dmitry.k...@gmail.com
on 11 Jul 2014 at 7:26
С замечаниями согласен, и честно говоря,
сделал их текстовыми так как выудить
оттуда весь кусок html, а потом стрипнуть
вашей функцией таги, было просто,
выкидывать оттуда суффикс BTC, который мне
ничем не мешал, а даже наоборот, было лень, а
то, что parseBalance на подобном суффиксе и так не
будет падать, я не знал.
Изменения проверил, разве что того самого
суффикса немного не хватает, добавил как
единицу в манифесте. Сейчас подредактирую
тут. Спасибо!
Original comment by Boris...@gmail.com
on 11 Jul 2014 at 8:32
Пришлось дома svn поставить, которого раньше
не было :-)
Вот коммит -
https://code.google.com/p/any-balance-providers/source/detail?r=4307
ЗЫ: вслед за ним сразу еще один, так как
случайно испортил идентацию :(
Original comment by Boris...@gmail.com
on 11 Jul 2014 at 8:54
Original comment by dmitry.k...@gmail.com
on 11 Jul 2014 at 7:49
Original issue reported on code.google.com by
Boris...@gmail.com
on 10 Jul 2014 at 10:46Attachments: